Thu, 25 Aug 2016 10:23:29 +0200fixes issue 33622: Selection lost on table/view based P&E having >100 records
Carlos Aristu <carlos.aristu@openbravo.com> [Thu, 25 Aug 2016 10:23:29 +0200] rev 29907
fixes issue 33622: Selection lost on table/view based P&E having >100 records

Selection could be lost on P&E windows based on a table or a view, having more than 100 records selected. This problem was solved on issue 31989 for those P&E windows based on HQL query tables (this kind of P&E windows make use of a ReadOnlyDataSourceService).

P&E windows based on a table or a view make use of the DefaultDataSourceService which uses DefaultJsonDataService to retrieve data. So, in order to solve the problem, a similar mechanism to the existing one on ReadOnlyDataSourceService has been implemented within DefaultJsonDataService.

Thus, those P&E which use DefaultDataSourceService to retrieve data, now increase the page size dynamically in order to get all selected records in the same request.

Thu, 25 Aug 2016 03:04:09 +0000CI: merge back from main
RM packaging bot <staff.rm@openbravo.com> [Thu, 25 Aug 2016 03:04:09 +0000] rev 29906
CI: merge back from main

Thu, 25 Aug 2016 02:50:19 +0000CI: update AD_MODULE to version 29904
RM packaging bot <staff.rm@openbravo.com> [Thu, 25 Aug 2016 02:50:19 +0000] rev 29905
CI: update AD_MODULE to version 29904

Wed, 24 Aug 2016 14:54:57 -0400Fixes issue 33636: Error should be raised when running "Modify Payment Plan"
Rafael Queralta Pozo <rqueralta@nauta.cu> [Wed, 24 Aug 2016 14:54:57 -0400] rev 29904
Fixes issue 33636: Error should be raised when running "Modify Payment Plan"

Now "The invoice is already included in a Remittance." message is shown when running Modify Payment Plan process from an invoice included in a remittance.

Wed, 24 Aug 2016 20:46:44 +0000CI: merge back from main
RM packaging bot <staff.rm@openbravo.com> [Wed, 24 Aug 2016 20:46:44 +0000] rev 29903
CI: merge back from main

Wed, 24 Aug 2016 20:32:36 +0000CI: update AD_MODULE to version 29901
RM packaging bot <staff.rm@openbravo.com> [Wed, 24 Aug 2016 20:32:36 +0000] rev 29902
CI: update AD_MODULE to version 29901

Wed, 24 Aug 2016 15:49:21 +0200related to issue 33706: code review improvements
Carlos Aristu <carlos.aristu@openbravo.com> [Wed, 24 Aug 2016 15:49:21 +0200] rev 29901
related to issue 33706: code review improvements

Wed, 24 Aug 2016 12:59:17 +0000CI: merge back from main
RM packaging bot <staff.rm@openbravo.com> [Wed, 24 Aug 2016 12:59:17 +0000] rev 29900
CI: merge back from main

Wed, 24 Aug 2016 12:45:07 +0000CI: update AD_MODULE to version 29890
RM packaging bot <staff.rm@openbravo.com> [Wed, 24 Aug 2016 12:45:07 +0000] rev 29899
CI: update AD_MODULE to version 29890

Wed, 24 Aug 2016 14:39:03 +0200Fixed 33771. Avoid double .list() call to not run query twice.
Stefan Hühner <stefan.huehner@openbravo.com> [Wed, 24 Aug 2016 14:39:03 +0200] rev 29898
Fixed 33771. Avoid double .list() call to not run query twice.

Old code ran following a OBCriteria
if (obc.list() == null || obc.list().size() == 0) {

However .list() does not return null as per Hibernate javadoc but empty or
non-empty list (of method raising an exception).

So remove the extra null check to avoid double .list() call.

Also improve .list().size() check slightly by using .list().isEmpty()

Add note that filter is on unique constraint so .list().size()<=1 also