Mon, 07 Mar 2016 11:31:44 +0100Fixes issue 32396: cannot filter grid when it is grouped by a column.
Inigo Sanchez <inigo.sanchez@openbravo.com> [Mon, 07 Mar 2016 11:31:44 +0100] rev 28791
Fixes issue 32396: cannot filter grid when it is grouped by a column.

Having a column grouped, if a filter was applied in a grid, it was not
possible to apply it.A js error was thrown:
Uncaught TypeError: this.data.willFetchData is not a function.

The problem was in:"this.data.willFetchData(....))" statement which
incorrectly assumed this.data to be an object and willFetchData a function
within it. This is not true when in this situation where this.data is a
Tree class.

It has been fixed by taking into account this case by checking that "this.data" defines the
function.

Fri, 31 Jul 2015 12:59:08 +0530Fixes Issue 30147: US Region code are wrong
Atul Gaware <atul.gaware@openbravo.com> [Fri, 31 Jul 2015 12:59:08 +0530] rev 28790
Fixes Issue 30147: US Region code are wrong

Codes for region updated as follows:
MI - Michigan
MN - Minnesota

Fri, 04 Mar 2016 14:37:26 +0100Fixed issue 32151: It wasnt possible configure text referenced cols. properly.
Naroa Iriarte <naroa.iriarte@openbravo.com> [Fri, 04 Mar 2016 14:37:26 +0100] rev 28789
Fixed issue 32151: It wasnt possible configure text referenced cols. properly.

It was not possible to configure the Allow Filtering property in colums that used Text reference.
The problem was in the UIDefinition.removeAttributeFromString.

Here the canSort and canFilter properties were being deleted and that was not correct, because in the case of
the "help" column of the Module window, for example, when the method was invoked, it was only desired to remove
the "canSort" property and not the canFilter.
The method has been changed, now, the passed String argument is the one which will be deleted instead of deleting
canSort or canFilter directly.

Fri, 04 Mar 2016 08:51:01 +0100related to bug 32375: added test case covering preference's client visibility
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 04 Mar 2016 08:51:01 +0100] rev 28788
related to bug 32375: added test case covering preference's client visibility

Fri, 04 Mar 2016 08:50:01 +0100fixed bug 32375: preferences incorrectly resolve client visibility
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 04 Mar 2016 08:50:01 +0100] rev 28787
fixed bug 32375: preferences incorrectly resolve client visibility

Reimplemented client visibility resolution to correctly handle the case of
2 preference candidates with different client visibility.

Fri, 04 Mar 2016 08:48:31 +0100related to bug 32375: make preference resolution more consistent
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 04 Mar 2016 08:48:31 +0100] rev 28786
related to bug 32375: make preference resolution more consistent

In case of several preferences defined for a given property/attribute, the
result of getting value could be undetermistic.

Solved by sorting by preference's id in these cases.

Fri, 04 Mar 2016 08:45:40 +0100related to bug 32375: preferences incorrectly resolve client visibility
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 04 Mar 2016 08:45:40 +0100] rev 28785
related to bug 32375: preferences incorrectly resolve client visibility

Small code clean up:
-isHigherPriority renamed to getHighestPriority: by convention is* are boolean
getters which was not the case of this one
-removed unneded brackets in condition which didn't help to make the code more
readable

Thu, 11 Feb 2016 13:03:59 +0100Fixes issue 32386: Set Final good C product as isquantityvariable in QA client
Alvaro Ferraz <alvaro.ferraz@openbravo.com> [Thu, 11 Feb 2016 13:03:59 +0100] rev 28784
Fixes issue 32386: Set Final good C product as isquantityvariable in QA client

Tue, 01 Mar 2016 13:24:31 +0100Fixes bug 32083: Move Service Price Rule menu entry under Pricing Setup menu entry
Unai Martirena <unai.martirena@openbravo.com> [Tue, 01 Mar 2016 13:24:31 +0100] rev 28783
Fixes bug 32083: Move Service Price Rule menu entry under Pricing Setup menu entry

Mon, 29 Feb 2016 18:07:19 +0530Fixes Issue 32337:Division by zero error when completing an invoice
Atul Gaware <atul.gaware@openbravo.com> [Mon, 29 Feb 2016 18:07:19 +0530] rev 28782
Fixes Issue 32337:Division by zero error when completing an invoice
associated to a closed order (quantity 0) with a promotion

While calculating promotion if qty ordered is non zero then only
use divide by statement else set values to zero

Tue, 01 Mar 2016 18:02:57 +0100Related to issue 32364: Code review improvements
Alvaro Ferraz <alvaro.ferraz@openbravo.com> [Tue, 01 Mar 2016 18:02:57 +0100] rev 28781
Related to issue 32364: Code review improvements

Call line.getProductCosts only when line.transaction != null in DocInventory.java.

Mon, 29 Feb 2016 17:00:12 +0530Fixes Issue 32364:NullPointerException when posting a Physical Inventory
Atul Gaware <atul.gaware@openbravo.com> [Mon, 29 Feb 2016 17:00:12 +0530] rev 28780
Fixes Issue 32364:NullPointerException when posting a Physical Inventory
with a line without an associated transaction (not stocked product)

- Avoid insertion of zero quantity products in inventory line having
stocked flag as No.
- Code improvement to handle line with null transaction.

Tue, 01 Mar 2016 13:19:21 +0100Fixes issue 0032381: missing upload image icon
Adrián Romero <adrianromero@openbravo.com> [Tue, 01 Mar 2016 13:19:21 +0100] rev 28779
Fixes issue 0032381: missing upload image icon
* Restoring upload icon and erase icon

Mon, 29 Feb 2016 23:49:20 +0100Fixes issue 32233: Payment In and Payment Out lines are not exported to CSV.
Inigo Sanchez <inigo.sanchez@openbravo.com> [Mon, 29 Feb 2016 23:49:20 +0100] rev 28778
Fixes issue 32233: Payment In and Payment Out lines are not exported to CSV.

The problem was that export to CSV does not work properly in some subtabs without
link to parent column (e.g.Lines subtab of Payment In/Out). Before, It was used
"tabId" to obtaing data for export to CSV and the value of the "tabId" was sent
in a variable called "tab". The Criteria that is run when it is displayed the tab
uses a property named "tabId" instead of "tab".

The solution for this case (and other related cases) is to adapt the code that
exports to CSV to use the same Criteria run when displaying the tab. Now export
to CSV uses tabId property. Besides it has been refactored some piece of code
related with export data in DataSourceServlet to use "tabId" instead of "tab".

Thu, 18 Feb 2016 13:23:15 +0100Fixed issue 0032044: Ensure stable ids in docs send from WebPOS to the server (Payment API)
Miguel de Juana <miguel.dejuana@openbravo.com> [Thu, 18 Feb 2016 13:23:15 +0100] rev 28777
Fixed issue 0032044: Ensure stable ids in docs send from WebPOS to the server (Payment API)

- Set some entities ids related to order loading in Web POS to ensure stable ids

Tue, 26 Jan 2016 17:24:27 +0100Fixed bug 31419: Add parameter to control flushes in some APRM methods
Víctor Martínez Romanos <victor.martinez@openbravo.com> [Tue, 26 Jan 2016 17:24:27 +0100] rev 28776
Fixed bug 31419: Add parameter to control flushes in some APRM methods

Added a parameter doFlush to some APRM methods to control whether to flush inside the method. By default it continues to have the same behavior as before (to flush inside the method) so we should avoid any possible regression.
These flushes inside the method makes the process of several payments in a batch really slow, which is a common scenario for the remittance module. With this change (and with the necessary modifications in the remittance module), the time to process a remittance line has been decreased from 900ms to 15ms.

The idea is to create this "self-protected" methods and adapt the flows with performance problems. When the time goes by, we should clean the duplicated methods.

Fri, 26 Feb 2016 08:57:54 +0100fixed bug 32354: field edit access doesn't allow to save if tab contains audit
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 26 Feb 2016 08:57:54 +0100] rev 28775
fixed bug 32354: field edit access doesn't allow to save if tab contains audit

For tabs that explicitly define their audit (updated/updated by) fields, roles
with access to edit only some fields couldn't save unless in those fields audit
were also included.

Now audit fields are not checked for this validation as they are always updated
on any allowed modification.

Fri, 26 Feb 2016 08:20:01 +0100[ChangePassword] Merge ChangePassword development
Guillermo Gil <guillermo.gil@openbravo.com> [Fri, 26 Feb 2016 08:20:01 +0100] rev 28774
[ChangePassword] Merge ChangePassword development

Mon, 22 Feb 2016 15:30:30 +0100[ChangePassword] Merge with pi
Jonathan Bueno <jonathan.bueno@openbravo.com> [Mon, 22 Feb 2016 15:30:30 +0100] rev 28773
[ChangePassword] Merge with pi

Mon, 22 Feb 2016 11:31:04 +0100[ChangePassword] Applied code review.
Jonathan Bueno <jonathan.bueno@openbravo.com> [Mon, 22 Feb 2016 11:31:04 +0100] rev 28772
[ChangePassword] Applied code review.

Now client messages are translated.
Used === for javascript comparison

Fri, 19 Feb 2016 13:42:53 +0100[ChangePassword] Renamed trigger to AD_USER_EXPIRATIONPASS_TRG
Jonathan Bueno <jonathan.bueno@openbravo.com> [Fri, 19 Feb 2016 13:42:53 +0100] rev 28771
[ChangePassword] Renamed trigger to AD_USER_EXPIRATIONPASS_TRG

Fri, 19 Feb 2016 12:56:49 +0100[ChangePassword] Applied code review.
Jonathan Bueno <jonathan.bueno@openbravo.com> [Fri, 19 Feb 2016 12:56:49 +0100] rev 28770
[ChangePassword] Applied code review.

Renamed trigger to be consistent, used expiration instead of expiry.
Used === to compare strings in Login.html.
Renamed getUpdatePasswordDate method to checkIfPasswordExpired in DefaultAuthenticationManager class.
Added throws in signature and javadoc for checkIfPasswordExpired and updatePassword methods.
Removed static parameter from checkIfPasswordExpired and updatePassword methods.
Removed extra brackets in expression in checkIfPasswordExpired method.
Exception AuthenticationExpirationPasswordException fixed. Removed extra variables, error is now loaded in the subclass AuthenticationException.
Removed empty line in checkIfPasswordExpired.
Removed check to support 2.50.
LoginUtils file keep unmodified.

Wed, 17 Feb 2016 09:56:42 +0100[ChangePassword] Merge with pi
Jonathan Bueno <jonathan.bueno@openbravo.com> [Wed, 17 Feb 2016 09:56:42 +0100] rev 28769
[ChangePassword] Merge with pi

Wed, 17 Feb 2016 09:52:43 +0100[ChangePassword] Applied code review.Fixed security problem.
Jonathan Bueno <jonathan.bueno@openbravo.com> [Wed, 17 Feb 2016 09:52:43 +0100] rev 28768
[ChangePassword] Applied code review.Fixed security problem.

Mon, 15 Feb 2016 13:16:35 +0100[ChangePassword] Added text interfaces from login.html
Jonathan Bueno <jonathan.bueno@openbravo.com> [Mon, 15 Feb 2016 13:16:35 +0100] rev 28767
[ChangePassword] Added text interfaces from login.html

Mon, 15 Feb 2016 12:28:37 +0100[ChangePassword] Merge with pi
Jonathan Bueno <jonathan.bueno@openbravo.com> [Mon, 15 Feb 2016 12:28:37 +0100] rev 28766
[ChangePassword] Merge with pi

Mon, 15 Feb 2016 12:11:46 +0100[ChangePassword] Code review applied.
Jonathan Bueno <jonathan.bueno@openbravo.com> [Mon, 15 Feb 2016 12:11:46 +0100] rev 28765
[ChangePassword] Code review applied.

Updated javadocs of methods.
Added check to avoid security problem when updating password.

Fri, 12 Feb 2016 13:44:48 +0100[ChangePassword] Updated year of license in some files. Updated referenced data with new name of the column
Jonathan Bueno <jonathan.bueno@openbravo.com> [Fri, 12 Feb 2016 13:44:48 +0100] rev 28764
[ChangePassword] Updated year of license in some files. Updated referenced data with new name of the column

Fri, 12 Feb 2016 11:41:37 +0100[ChangePassword] Code review applied.
Jonathan Bueno <jonathan.bueno@openbravo.com> [Fri, 12 Feb 2016 11:41:37 +0100] rev 28763
[ChangePassword] Code review applied.

Solved camel case in Days To Password Expiration column.
Labels in Login.html are now translatable.
Last Password Update column is now updated in case user didn't have a password and is set for the first time.
Messages are now set correctly from the beginning and are not overwritten.
Refactor code, now check for expiration date is made in the method, exception is thrown in the method also.
AuthenticationExpirationPasswordException now log a single line.

Tue, 09 Feb 2016 09:45:56 +0100[Change Password] Merge with pi
Alberto Santos <alberto.santos@openbravo.com> [Tue, 09 Feb 2016 09:45:56 +0100] rev 28762
[Change Password] Merge with pi