Hello,
I've got a question concerning posting a pro forma sales packing slip. Our client has recently updated their invironment to CU8 and with this update there was a change in SalesFormLetter class. Specifically in the checkSales() method.
This is part of the method that has changed.
if (! this.checkHeading() ||
! this.checkBlockedLevel() ||
! this.checkLines() ||
(isConfigurationkeyEnabled(configurationkeynum(CRSEItaly)) &&
(salesParmTable.Ordering == DocumentStatus::PackingSlip ||
salesParmTable.Ordering == DocumentStatus::ProjectPackingSlip) &&
! this.checkLedgerPeriod()))
{
ok = false;
}
The red code is what seems to be new for the invironment after the update. What we've noticed is that when we try to post a pro forma packing slip it checks if the ledger period is open or close. "!this.checkLedgerPeriod()" part of the code. This seems to be a new method added in SalesFormLetter_PackingSlip class.
protected boolean checkLedgerPeriod()
{
boolean ok;
;
if(isConfigurationkeyEnabled(configurationkeynum(CRSEItaly)))
{
ok = LedgerPeriod::checkLedgerPeriodDateModule(salesParmTable.Transdate, SysModule::Sales);
}
return ok;
}
As seen above if the configurationkey CRSEItaly is enabled a check is made against the LedgerPeriod.
What this does is that since the configurationkey CRSEItaly is activated in our environment, other companies that have nothing to do with the Italian part also get this "check" when they try to post a pro forma packing slip.
My question is simple. Is this intended? Is this a bug? Or am I missing something obvious?
I've searched through partnersource and the web and I cannot seem to find anything about this issue.
Any ideas/solutions about this are appreciated.
Regards,
