Mon, 19 Sep 2016 11:33:17 -0400Fixes issue 34018: Is possible posting a Purchase Invoice with Finance role
Mark <markmm82@gmail.com> [Mon, 19 Sep 2016 11:33:17 -0400] rev 30558
Fixes issue 34018: Is possible posting a Purchase Invoice with Finance role

Modified the OrganizationStructureProvider.getPeriodControlAllowedOrganization() method to execute in Admin mode.

Mon, 19 Sep 2016 14:24:57 +0000CI: merge back from main
RM packaging bot <staff.rm@openbravo.com> [Mon, 19 Sep 2016 14:24:57 +0000] rev 30557
CI: merge back from main

Mon, 19 Sep 2016 14:10:25 +0000CI: update AD_MODULE to version 30294
RM packaging bot <staff.rm@openbravo.com> [Mon, 19 Sep 2016 14:10:25 +0000] rev 30556
CI: update AD_MODULE to version 30294

Fri, 16 Sep 2016 12:42:07 -0400Fixes issue 34003: Set New Currency process warning needs to be modified
Mark <markmm82@gmail.com> [Fri, 16 Sep 2016 12:42:07 -0400] rev 30555
Fixes issue 34003: Set New Currency process warning needs to be modified

Warning message changed to:

Detected available credit for this business partner.
Changing the Business Partner's currency will not allow you to use available credit in the old currency. If you want to convert the available credit in the old currency into credit in the new currency automatically, select a G/L Item.

Therefore last sentece has beee removed.

Mon, 19 Sep 2016 12:53:30 +0200Related to issue 33887: Code review improvements
Alvaro Ferraz <alvaro.ferraz@openbravo.com> [Mon, 19 Sep 2016 12:53:30 +0200] rev 30554
Related to issue 33887: Code review improvements

- Remove unnecessary parenthesis.
- Add paymentDetail.getFINPaymentScheduleDetailList().get(0).getOrderPaymentSchedule() != null check to avoid NPE.

Wed, 14 Sep 2016 19:39:08 -0400Fixes issue 33887: GLItem accounting dimensions maintained when posting Payment
Mark <markmm82@gmail.com> [Wed, 14 Sep 2016 19:39:08 -0400] rev 30553
Fixes issue 33887: GLItem accounting dimensions maintained when posting Payment

In case Payment document has business partner set in payment's header and GL Item line is included setting different business partner, then accounting entry is generated for line's business partner instead of header's business partner.

Mon, 19 Sep 2016 11:14:46 +0200Related to issue 33989: Code review improvements
Alvaro Ferraz <alvaro.ferraz@openbravo.com> [Mon, 19 Sep 2016 11:14:46 +0200] rev 30552
Related to issue 33989: Code review improvements

- Avoid setting admin mode in getOrgLedgerRecursive() and getClientLedger() methods as it is already done in getOrgLedger() method in OBLedgerUtils class.
- Retrieve first, general ledger of login organization in LoginUtils class. If no result is retrieved, we will retrieve general ledger of rest of context organizations.

Wed, 14 Sep 2016 13:39:12 +0200Fixed issue 33989. Removed slow query getting Organization Ledger.
Gorka Ion Damián <gorkaion.damian@openbravo.com> [Wed, 14 Sep 2016 13:39:12 +0200] rev 30551
Fixed issue 33989. Removed slow query getting Organization Ledger.

The method to get the default General Ledget was using a query that filtered
and sorted the results using ad_isorgincluded procedure. In case of having a
large number of organizations this query was slow.

The method is now using the OrganizationStructureProvider to get the parent
organization List of the required org to search the default General Ledger.

Thu, 15 Sep 2016 12:14:10 -0400Fixes issue 33838: Correct Stock Reservation when created from a closed order
Mark <markmm82@gmail.com> [Thu, 15 Sep 2016 12:14:10 -0400] rev 30550
Fixes issue 33838: Correct Stock Reservation when created from a closed order

When voiding a shipment related to a reserved order, if order is closed, new reservation will not be created (in case order was fully shipped) and existing reservation will not be updated (in case order was partially shipped).
Besides, when closing an order, related reservation will be also closed.

Thu, 15 Sep 2016 14:04:47 +0200Related to issue 33879: Code review improvements
Alvaro Ferraz <alvaro.ferraz@openbravo.com> [Thu, 15 Sep 2016 14:04:47 +0200] rev 30549
Related to issue 33879: Code review improvements