Thu, 15 Dec 2016 09:15:56 +0100fixed bug 28149: PG extension functions are tried to be deleted on update DB
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 15 Dec 2016 09:15:56 +0100] rev 31194
fixed bug 28149: PG extension functions are tried to be deleted on update DB

Having some installed any PG extension that adds functions, they are tried to
be deleted each time update.database is executed.

Fixed by excluding any function named like PG_% which should be reserved for
PG extension functions.

Wed, 14 Dec 2016 14:28:50 +0100Fixed issue 33626: Unneeded ds request when cancelling a record fixed
Naroa Iriarte <naroa.iriarte@openbravo.com> [Wed, 14 Dec 2016 14:28:50 +0100] rev 31193
Fixed issue 33626: Unneeded ds request when cancelling a record fixed

There was an uneeded ds request when there was a summary function and a new record edition was cancelled in grid view.
To fix this, the showGridSummary property has been set to false before the call to the function "handleUpdate". This avoids one flow of smartclient which was triggerin a ds request.
After that the "setShowGridsummary" function is invoked, to show again the summary function values.

Thu, 15 Dec 2016 09:03:05 +0100fixed bug 24051: Audit trail triggers are exported to core
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 15 Dec 2016 09:03:05 +0100] rev 31192
fixed bug 24051: Audit trail triggers are exported to core

Exclude them properly using exclude filter with wildcards

Thu, 15 Dec 2016 08:53:09 +0100fixed bug 33617: don't show "is fully audited" field for non db tables
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 15 Dec 2016 08:53:09 +0100] rev 31191
fixed bug 33617: don't show "is fully audited" field for non db tables

Fully audited field makes only sense for AD tables backed out by a physical DB
table, though it was visible for all of them.

Now it is hidden for those tables with origin type different to Table and for
views.

Additionally, if any non table origin is marked as audited, its audit trigger
will not be generated, so that error is not thrown.

Thu, 15 Dec 2016 00:12:10 +0100Fixes issue 34721: Error thrown in the log after invoking a JSON Rest Web Service
Martin Taal <martin.taal@openbravo.com> [Thu, 15 Dec 2016 00:12:10 +0100] rev 31190
Fixes issue 34721: Error thrown in the log after invoking a JSON Rest Web Service
Handle webservice requests for generic REST data interfaces

Wed, 14 Dec 2016 16:45:26 +0100fixes issue 34725: make jasper report params accessible in getReportData method
Carlos Aristu <carlos.aristu@openbravo.com> [Wed, 14 Dec 2016 16:45:26 +0100] rev 31189
fixes issue 34725: make jasper report params accessible in getReportData method

Now the parameter map received as an argument in getReportData() contains the HTTP request parameters together with the parameters that will be sent to the report. This second set of parameters can be retrieved from the map using the JASPER_REPORT_PARAMETERS key

Wed, 14 Dec 2016 10:27:46 -0500Fixes issue 34731: Wrong amount value when adding payment to a transaction
Alejandro <alekosmp86@gmail.com> [Wed, 14 Dec 2016 10:27:46 -0500] rev 31188
Fixes issue 34731: Wrong amount value when adding payment to a transaction

When creating a withdrawal transaction in Financial Account screen, if you add a
payment, the field Actual Payment was showing 0.00 instead of the correct amount
of the transaction. This issue is solved by calling the method convertToSafeDecimalString,
which returns a well formed string, so the correct amount can be displayed.

Mon, 12 Dec 2016 16:10:29 +0100Fixed issue 33853: There were too much DS request
Naroa Iriarte <naroa.iriarte@openbravo.com> [Mon, 12 Dec 2016 16:10:29 +0100] rev 31187
Fixed issue 33853: There were too much DS request

When a column with a summary function was set, if any of the grid columns was reordered, there were three DS requests triggered. To fix the problem two different things have been done. The first, in ob-view-grid.js, consists on setting the showGridSummary property to false before the call to the function "reorderField" of the Super- This avoids one flow of smartclient which was, at the end, triggering two DS requests.
The second approach is also in ob-view-grid, but also in ob-grid. The problem was that when the ob-grid.getSummaryRow function was invoked, the return statement was calling the "getSummaryRow" function of the Super. There, once again a DS request was triggered. To fix this, a new object has been created. This object contains three properties. One of them is a property which is true if the columns have been reordered.
Now, in the ob-grid.getSummaryRow, if that property is true and if there is a summary function, the summary row is returned.

Wed, 14 Dec 2016 12:35:05 +0100Related to issue 34713: Code review improvements
Alvaro Ferraz <alvaro.ferraz@openbravo.com> [Wed, 14 Dec 2016 12:35:05 +0100] rev 31186
Related to issue 34713: Code review improvements

Fix also getLastCumulatedCosting method.

Tue, 13 Dec 2016 10:01:21 -0500Fixes issue 34713: getProductCost() is not checking if product is production
Alejandro <alekosmp86@gmail.com> [Tue, 13 Dec 2016 10:01:21 -0500] rev 31185
Fixes issue 34713: getProductCost() is not checking if product is production

If a product is defined as "production", when costing is checked for
that product, "getProductCost" method in AverageAlgorithm.java was filtering by
"warehouse" dimension (in case costing rule is defined like this) so error "No
average cost found for given product and date" was raised, when it should not.
The proposed solution has been implemented, adding a validation when constructing
the "where" condition, so now there is no error.