Related to issue 29168. Missing copyright update

Related to issue 28668: Pressing Cancel in Match Statement or in Process Reconciliation does not trigger a refresh.
Refactor query to use named parameters to avoid this piece to be used as template for other developers as this could be a risky practice due to injections.

Related to issue 29353: Refactor of class in FIN_Utility so it can be used in core. This is now used in Initial Client Setup

Fixed bug 29353: ICS implemented indirect dataset dependencies

When getting the module dependencies we take into account all the modules installed in the instance instead of only the one selected by the user.
We use the UpdateReferenceDataData.selectModules() method, which is used by the Enterprise Module Management process too, to get the right order.
The UpdateReferenceDataData class is now public to be called from InitialClientSetup.java.
Added missed ProcessingDataset message

Related to issue 29720: Modify TestCosting tests

Modify GM11, GM12, GM13 and GM5 tests, where cost adjustments from goods movements are created, to update accounting amounts

Fixes issue 29720: Wrong accounting in Cost Adjusment

Wrong accounting of the cost adjusment lines in case of Goods Movement From.
In this case, accounting should consider negative amounts, in order to have a net effect of the goods movement adjustment accounting as 0.

Fixed bug 28592: Added alerts and callout to manage Tax Location

Implemented 2 alerts:
+ Business Partner with no Tax Location Address
+ Business Partner with several Tax Location Addresses

Implemented callout over Location and Tax Location fields (Business Partner | Location/Address tab) that raises a warning if any of these conditions is fulfilled for the business partner:
+ More than 1 country set as Tax Location
+ Other country is declared as Tax Location and the user is trying to set a new country as Tax Location
+ The business partner has several countries and neither of them is declared as Tax Location

Fixes issue 28668: Pressing Cancel in Match Statement or in Process Reconciliation does not trigger a refresh.
Cancel button has been removed as it was considered confusing. Now there are two buttons, one for closing the window called 'OK', as it was before, and another one called 'Reconcile'.

Both triggers refresh. Messages have been added to improve user understanding on how the window works. New message to inform the user any change will be persisted. This message can be disabled once clear and will not popup again. Message to run algorithm as well improved.

Related to issue 29687: Added QUnit test cases - Fixed JSLint problem

Related to bug 28927: Fix unmatching error

An error was being raised while doing unmatch when using 'addSelectedIDsToCriteria' method. The call to this method has been removed because it is not necessary as the criteria already has the selected ids.

related to issue 29719: fixed property store test

CCU property is expected to be present

Related to issue 29687: Added QUnit test cases

Fixed issue 28513: Error message wrongly shown when processing an inventory.

The error message when processing an inventory is wrongly shown with a product
with % in its name and a negative stock.

The issue occurs when we are parsing the error message and detects the % symbol
and try to find the string format in the next character.

The proposal solution is to eliminate the String.format() method from the
code.

Now, the error message is shown properly with the correct name.

Fixes Issue 28895:Cannot process an invoice if the role does not have access
to FIN_Payment and the business partner has credit left

Changed OBContext to Admin mode when accessing Description field of Payment
and lastly restored it back again.

Fixes issue 29338: Payment navigation from Transaction tab is working wrong

Issotrx auxiliary input query in Transaction tab has been changed to get into account related payment isreceipt field, when navigating from it to Payment In/Out window
Also, SE_Payment_Transaction and SE_Trxtype_Transaction callouts have been changed to overwrite issotrx session value when updating payment or transactiontype fields

Related to bug 29091: Code Review

[1]: Fix copyright
[2]: Change != for <>

Fixed issue 29091: Fully depreciated field is not updated

The process didn't update the Fully Depreciated field when one or more
asset amortization lines were unprocessed.

The problem was in the amortization process.

Now the process changes the isFullyDepreciated field if the Depreciated
Value and the Depreciated Plan fields are differents.

fixes issue 29667: NPE Error when a child tab is based on an HQL Query table

The NPE was raised because the code was retrieving the entity for a table not based on a db table. This kind of tables do not have BaseOBObjects.
The FIC was already controlling this when the tab has a related table of this type but it was not considering the case when the parent table is related to a table of this type.

fixes issue 29665: NP Error raised when using HQL Based Tables on a level 2 tab

A unnecessary request was being done when accessing to the HQL query based subtab. This was caused because the tab was not being identified properly as a subtab.
The view.parentProperty was used to identify if we are on a child tab. This property could be undefined depending on the configuration of the tab in the application dictionary.
Now view.parentView is used instead of view.parentProperty because it does not return undefined in this cases

related to issue 29719: fixed jsLint

- Do not use global functions exposed by QUnit but use the ones defined within
the QUnit global object
- Define QUnit as /*global*/ directive JSLint not to complain about it use

fixed issue 29270: prevent table recreation in update.database

When adding or removing columns to existing tables update.database does not
recreate the table but just appends the new columns. In order to do this,
column phyisical position wihtin the table is no longer important, being relative
only to the time they were created but not to the module they belong to.

[recreation] prevent table recreation

[recreation] added parameter to update.database to force recreation i


All/specific tables can be forced to be recreated when database is updated in
case they have model changes.

fixed issue 29719: Recover QUnit JavaSript test cases

QUnit test cases are now ready to be included in CI:
-QUnit library has been updated to latest version (1.1.18)
-JavaScript files are loaded in a proper order to guarantee dependencies
-Test cases create an Openbravo session not requiring to use a 2nd tab
-Results are kept in a js object so Selenium can easily access

[jsTest] reorganize OBTest object to be easier managed by Selenium

[jsTest] fixed test cases

[jsTest] fixed test case

[jsTest] proper js load and execution, create session before StaticResources

[jsTest] adapted tests to new QUnit version

[jsTest] Updated QUnit to 1.18.0