Wed, 07 Oct 2015 09:31:15 +0200[HGVOL] Merged from PI
Sandra Huguet <sandra.huguet@openbravo.com> [Wed, 07 Oct 2015 09:31:15 +0200] rev 27917
[HGVOL] Merged from PI

Wed, 07 Oct 2015 09:25:04 +0200Related to issue 30832: code review improvements
Víctor Martínez Romanos <victor.martinez@openbravo.com> [Wed, 07 Oct 2015 09:25:04 +0200] rev 27916
Related to issue 30832: code review improvements

M_INOUTLINE_TRG:
+ Updated copyright
+ Added more conditions to the if clause when updating
+ Usage of MOVEMENTQTY and QUANTITYORDER instead of v_qtyold, v_qty, v_qtyorderold and v_qtyorder as these variables always make true the if condition (so the fix would useless)
+ In the update part, update first the inventory for the old product/locator as it was done before the refactor

M_inoutlinetrgTest.java:
+ Totally rewritten of the JUnit
+ Support for more scenarios: update locator, update attributesetinstance, work with several lines, purchase and sales flows, etc.
+ Fixed scenarios not properly working before
+ Avoid HQL injection

Mon, 05 Oct 2015 14:56:31 +0530Fixes Issue 30832:potential unneeded storage detail updates
Atul Gaware <atul.gaware@openbravo.com> [Mon, 05 Oct 2015 14:56:31 +0530] rev 27915
Fixes Issue 30832:potential unneeded storage detail updates
on changes for m_inoutlines

Check provided in M_InOutLine_Trg before update operation whether
any of the following is changed in M_InOutLine.

Organization, Product, UOM, AttributeSetInstance, Movement Quantity,
ProductUom, Order Quantity.

Junit test cases to test M_InoutLine_Trg covers following scenarios.
- Insert M_InOutLine.
- Set Product Null in the M_InOutLine.
- Set Product in M_InOutLine where Product was null.
- Set Description in M_InOutLine.
- Delete a M_InOutLine.

Fri, 02 Oct 2015 12:39:49 +0200Fixed bug 31015: hidden fields in several windows related to acct. dimensions
Víctor Martínez Romanos <victor.martinez@openbravo.com> [Fri, 02 Oct 2015 12:39:49 +0200] rev 27914
Fixed bug 31015: hidden fields in several windows related to acct. dimensions

The display logic of some fields driven by the accounting dimension display logic utility was not working fine when the auxiliary input "DOCBASETYPE" (used by to calculate the acct. dimension display logic) was null.
This auxiliary input has a hardcoded value in some windows, however in other windows it is gotten from the DB using the selected document type. In the latter scenario, when the document type field is blank, the "DOCBASETYPE" auxiliary input was null, so the system was unable to detect the right configuration, thus hiding the fields.

The regression is caused by issue #29767. The business partner and organization fields were configured to use the accounting dimension display logic utility in the following windows:
* Sales Order
* Purchase Order
* Sales Invoice
* Purchase Invoice
* Payment In (not affected as it has a hardcoded "DOCBASETYPE")
* Payment Out (not affected as it has a hardcoded "DOCBASETYPE")

The fix improves the SQL for getting the "DOCBASETYPE" auxiliary input in the affected windows, so in case the document type is null, a generic value (the most commonly used) is returned depending on the window:
* Sales Order: SOO
* Purchase Order: POO
* Sales Invoice: ARI
* Purchase Invoice: API

Fri, 02 Oct 2015 12:24:48 +0200Fixes issue 31006: Credit consumed although payment not completed
Alvaro Ferraz <alvaro.ferraz@openbravo.com> [Fri, 02 Oct 2015 12:24:48 +0200] rev 27913
Fixes issue 31006: Credit consumed although payment not completed

When consuming a credit with a currency different than Business Partner's currency, an error is shown but credit was also consumed, which was wrong.
Now, this check will be also done before consuming credit in AddPaymentActionHandler addCredit method, as it was done in FIN_PaymentProcess doProcessPayment method.

Mon, 28 Sep 2015 13:17:45 +0200fixed bug 30948: auxiliary input to default a FK doesn't display identifier
Asier Lostalé <asier.lostale@openbravo.com> [Mon, 28 Sep 2015 13:17:45 +0200] rev 27912
fixed bug 30948: auxiliary input to default a FK doesn't display identifier

When an auxiliary input is used to default a FK value, its value was correctly
calculated but it's identifier was not, this made the UI not to display the
identifier on NEW, but to correctly set it being visible after save.

This has been fixed by detecting this special case, when in this situation:

- Value is calculated in computeAuxiliaryInputs method, including also its
identifier (obtained from field's ui defintion)
- Value is not calculated in computeColumnValues method because it has already
been computed

Mon, 05 Oct 2015 09:57:41 +0200fixed bug 30993: can't define default with preference if column has default
Asier Lostalé <asier.lostale@openbravo.com> [Mon, 05 Oct 2015 09:57:41 +0200] rev 27911
fixed bug 30993: can't define default with preference if column has default

When a column had a default value, it was not possible to overwrite it using a
preference. This was possible if the column had not default value.

This behavior has been changed: now default value defined at preference prevails
over the one defined at column level.

Thu, 01 Oct 2015 14:31:43 +0530Fixes Issue 30509:The MRP only takes into account the process plan version
Atul Gaware <atul.gaware@openbravo.com> [Thu, 01 Oct 2015 14:31:43 +0530] rev 27910
Fixes Issue 30509:The MRP only takes into account the process plan version
valid when it is being processed

- MA_ProcessPlan_Totalized view is changed to not only to consider processplan
versions in the current date now(). Also Column added showing Date From and
Date To for Process Plan versions.
- Latest version is selected for MRP_Run_Lines's Produced product based on
Planned Order Date from MA_ProcessPlan_Totalized available versions.

Mon, 28 Sep 2015 19:19:54 +0530Fixes Issue 30754:Stock of inactive products can be increased.
Atul Gaware <atul.gaware@openbravo.com> [Mon, 28 Sep 2015 19:19:54 +0530] rev 27909
Fixes Issue 30754:Stock of inactive products can be increased.

Before processing Goods Receipt, Goods Shipment with -ve lines, BOM Production
and Work effort transaction line product active flag is checked.

Thu, 01 Oct 2015 14:57:25 +0200related to issue 31007: fixed uneeded if nesting
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 01 Oct 2015 14:57:25 +0200] rev 27908
related to issue 31007: fixed uneeded if nesting