Mon, 19 Dec 2016 10:34:03 +0100fixed issue 33816: OBCriteria adds standard filters several time
Carlos Aristu <carlos.aristu@openbravo.com> [Mon, 19 Dec 2016 10:34:03 +0100] rev 30981
fixed issue 33816: OBCriteria adds standard filters several time

In some cases (ie. in count method), OBCriteria adds the standard filters for client, org and active several times. This problem is caused because initialize method can be invoked several times on the same criteria instance adding each time those filters.

This is fixed now by using two flags:
- initialize: is set on 1st initialization and it is used to check if the criteria instance has already been initialized.
- modified: it is used to identify if the criteria instance has been modified (by changing some filter or the order by clause) after being initialized. If it is false, we will exit the method without executing any action. Otherwise, the initialization will be done again. In this case a warning will be thrown in the log, informing that the generated qquery will have duplicated filters.

Note that still OBCriteria is not thread safe: several parallel initialization in the same instance could clash

Mon, 19 Dec 2016 10:05:00 +0100related to issue 34721: apply formatting + update copyright year
Carlos Aristu <carlos.aristu@openbravo.com> [Mon, 19 Dec 2016 10:05:00 +0100] rev 30980
related to issue 34721: apply formatting + update copyright year

Mon, 19 Dec 2016 09:06:33 +0100fixed issue 34765: incorrect license for servlet-api 3.0
Asier Lostalé <asier.lostale@openbravo.com> [Mon, 19 Dec 2016 09:06:33 +0100] rev 30979
fixed issue 34765: incorrect license for servlet-api 3.0

Updated to Apache 2.0

Sun, 18 Dec 2016 21:54:08 +0100Fixes issue 34721: Error thrown in the log after invoking a JSON Rest Web Service
Martin Taal <martin.taal@openbravo.com> [Sun, 18 Dec 2016 21:54:08 +0100] rev 30978
Fixes issue 34721: Error thrown in the log after invoking a JSON Rest Web Service
Support stateless also for rest requests

Fri, 16 Dec 2016 17:37:14 +0000CI: merge back from main
RM packaging bot <staff.rm@openbravo.com> [Fri, 16 Dec 2016 17:37:14 +0000] rev 30977
CI: merge back from main

Fri, 16 Dec 2016 17:22:08 +0000CI: update AD_MODULE to version 30975
RM packaging bot <staff.rm@openbravo.com> [Fri, 16 Dec 2016 17:22:08 +0000] rev 30976
CI: update AD_MODULE to version 30975

Fri, 16 Dec 2016 13:22:24 +0100Related with issue 34453:Added missing catch block for the commitAndClose
Inigo Sanchez <inigo.sanchez@openbravo.com> [Fri, 16 Dec 2016 13:22:24 +0100] rev 30975
Related with issue 34453:Added missing catch block for the commitAndClose

Fri, 16 Dec 2016 11:08:58 +0100related to issue 34721: apply file format and add missing @Override annotation
Carlos Aristu <carlos.aristu@openbravo.com> [Fri, 16 Dec 2016 11:08:58 +0100] rev 30974
related to issue 34721: apply file format and add missing @Override annotation

Fri, 16 Dec 2016 11:00:19 +0100Fixed issue 33626: Code review improvements
Naroa Iriarte <naroa.iriarte@openbravo.com> [Fri, 16 Dec 2016 11:00:19 +0100] rev 30973
Fixed issue 33626: Code review improvements

When a record was cancelled i grid view, without any summary row defined in any column, the summary function grid was shown with the text "no items to show" this has been fixed by adding a condition to the ob-view-grid.discardEdits function. Now, the summary grid will only be shown when there actually exists a summary function in any column.

Fri, 16 Dec 2016 10:33:06 +0100Related to issue 33853: Code review improvements
Naroa Iriarte <naroa.iriarte@openbravo.com> [Fri, 16 Dec 2016 10:33:06 +0100] rev 30972
Related to issue 33853: Code review improvements

In js variables must be declared at the top of the function.
Also there were problems when reordering if there was not a summary function declared. This has been prevented with an if statement. A new function has been created, this, handles the behavior of the summary functions' grid when it is necessary.