Mon, 18 Jul 2016 13:26:10 +0200fixes issue 33367: testChangedClasses doesn't calculate the past date correctly
Carlos Aristu <carlos.aristu@openbravo.com> [Mon, 18 Jul 2016 13:26:10 +0200] rev 29791
fixes issue 33367: testChangedClasses doesn't calculate the past date correctly
600 days in milliseconds overflows Integer.MAX_VALUE, so the resulting past Date using that value was not correct. Now the value is turned into long before substracting it to the current time.

Mon, 18 Jul 2016 13:15:07 +0200fixes issue 33355: Potential security issue in Process Definitions
Carlos Aristu <carlos.aristu@openbravo.com> [Mon, 18 Jul 2016 13:15:07 +0200] rev 29790
fixes issue 33355: Potential security issue in Process Definitions
The problem was that in SecurityChecker the checking for write access was not considering that the object being written could be an instance of Client (This class is the only DAL generated class which does not implement the ClientEnabled interface).

Mon, 18 Jul 2016 11:45:42 +0200fixed bug 33490: can't create/edit record depending on how windows are opened
Asier Lostalé <asier.lostale@openbravo.com> [Mon, 18 Jul 2016 11:45:42 +0200] rev 29789
fixed bug 33490: can't create/edit record depending on how windows are opened

Depending on the order different windows were opened/operated within the same
session, it was not possible to edit/crete records in some of these windows.

The problem was on how data was cached in ADCS. If more than one tab shared
the same table, tab.getTable was only initialized for the 1st one because
tab.getTable.getId() was used to check whether it was already in cache. This
code before made the table to be fully initialized, but now it only gets the
id from proxy.

Fixed by forcing table initialization.

Fri, 15 Jul 2016 10:12:00 +0200fixed bug 32908: Grid recordcount has a delay to be updated
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 15 Jul 2016 10:12:00 +0200] rev 29788
fixed bug 32908: Grid recordcount has a delay to be updated

Removed unneded delay

Fri, 15 Jul 2016 09:09:26 +0200fixed bug 33077: ImportEntryManager constructor called twice
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 15 Jul 2016 09:09:26 +0200] rev 29787
fixed bug 33077: ImportEntryManager constructor called twice

Constructor in ImportEntryManager was invoked twice even it is a bean with
application scope.

The problem is CDI doesn't guarantee single constructor invocation per injection
point instance as this code was asuming.

The solution is to move initialization code to a method annotated with
@PostConstruct.

Mon, 11 Jul 2016 13:25:10 +0200fixed bug 26664: useless ConnectionProvider caching in ADCS
Asier Lostalé <asier.lostale@openbravo.com> [Mon, 11 Jul 2016 13:25:10 +0200] rev 29786
fixed bug 26664: useless ConnectionProvider caching in ADCS

ConnectionProvider was cached in ADCS through cached ComboTableData instances.

This retained a significant amount of memory.

Now it is not cached anymore and obtained from DAL connection when needed.

Thu, 14 Jul 2016 15:12:51 +0200fixed bug 33484: links to Import Entry and Archive Import are not navigable
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 14 Jul 2016 15:12:51 +0200] rev 29785
fixed bug 33484: links to Import Entry and Archive Import are not navigable

The tables didn't define the windows to navigate to.

Thu, 14 Jul 2016 12:15:48 +0200fixed bug 33412: error in tree sub tab when creating record in its header
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 14 Jul 2016 12:15:48 +0200] rev 29784
fixed bug 33412: error in tree sub tab when creating record in its header

A JS error was thrown beacause tree subtab incorrectly assumed there was a
record selected in its parent tab.

Wed, 13 Jul 2016 09:45:27 -0400Fixes issue 33465: Right invoice amount when Add Payment from Match Statement
Mark <markmm82@gmail.com> [Wed, 13 Jul 2016 09:45:27 -0400] rev 29783
Fixes issue 33465: Right invoice amount when Add Payment from Match Statement

Invoice amount is populated when running Add Payment from Add Transaction from Match Statement process, and the amount field is set to invoice's amount instead of to zero when specific invoice is selected.

Wed, 13 Jul 2016 15:52:09 +0200fixed 33474: parameter shouldn't be assigned warns in WAD code
Asier Lostalé <asier.lostale@openbravo.com> [Wed, 13 Jul 2016 15:52:09 +0200] rev 29782
fixed 33474: parameter shouldn't be assigned warns in WAD code

Assign the paramter to a local variable so it can be reasigned without warn.