Wed, 09 Aug 2017 05:59:33 +0000Update AD_MODULE version to 3.0PR17Q2.2 3.0PR17Q2.2
RM packaging bot <staff.rm@openbravo.com> [Wed, 09 Aug 2017 05:59:33 +0000] rev 32565
Update AD_MODULE version to 3.0PR17Q2.2

Tue, 18 Jul 2017 11:06:02 -0400Fixes issue 36483: Cannot add some sales order lines to goods shipments after
Armaignac <collazoandy4@gmail.com> [Tue, 18 Jul 2017 11:06:02 -0400] rev 32564
Fixes issue 36483: Cannot add some sales order lines to goods shipments after
upgrading to 3.0PR17Q1 as a modulescript set the order as isdelivered

Cannot add some sales order lines to goods shipments because the modulescript
set the order as isdelivered when the order contains lines with negative qtyordered
and the sum(qtyordered) = sum(qtydelivered)

Now the update only happend when the order don't have a line where
QTYORDERED <> QTYDELIVERED.

Because the NOT EXISTS change the check to issotrx field now is made in the main
query because a wrong evaluation in the subquery happends when issotrx = 'N' and
NOT EXISTS

Tue, 11 Jul 2017 09:13:34 +0200fixed bug 36461: in grid can trigger save before FIC response is received
Asier Lostalé <asier.lostale@openbravo.com> [Tue, 11 Jul 2017 09:13:34 +0200] rev 32563
fixed bug 36461: in grid can trigger save before FIC response is received

While editing in grid mode, it is possible to trigger save action by clicking
grid's save button while FIC response is not received yet. If this occurs, the
data sent to backend misses information that FIC should update.

If grid's save button is clicked while in FIC call, postpone save action till
FIC return.

Mon, 10 Jul 2017 15:08:05 -0400Related to issue 36398: Hide secondUOM fields if product doesn't have secondUOM
Armaignac <collazoandy4@gmail.com> [Mon, 10 Jul 2017 15:08:05 -0400] rev 32562
Related to issue 36398: Hide secondUOM fields if product doesn't have secondUOM

When a product with a second UOM was selected the associated fields are shown,
but not hidden when the product is change and doesn't have a second UOM because
the has_second_uom var is not updated with the corresponding value according
to the product selection.

The value of has_second_uom var is updated according to the product selection
in the result for a correct use in display logic function.

Mon, 10 Jul 2017 12:49:45 +0200Fixes issue 36398: Avoid casting when adding numeric results in SimpleCallout
Armaignac <collazoandy4@gmail.com> [Mon, 10 Jul 2017 12:49:45 +0200] rev 32561
Fixes issue 36398: Avoid casting when adding numeric results in SimpleCallout

The casting to Object was replace by a integer conversion, sending the
corresponding numeric value when it applies.

Fri, 23 Jun 2017 10:40:44 -0400Fixes issue 36298: Wrong schedule details when voiding a prepaid invoice
Armaignac <collazoandy4@gmail.com> [Fri, 23 Jun 2017 10:40:44 -0400] rev 32560
Fixes issue 36298: Wrong schedule details when voiding a prepaid invoice

The void process of prepaid invoice was taking into account all payments schedule
detail to update their amount with the invoice payment schedule amount.

Now the order payment schedule detail update it it's the one not asociated to any invoice
or a new one is created if the order have no pending payment schedule detail.

The invoice prepayment amount was also incorrect when the invoice is created from
CreateInvoicesFromOrder process and the order have a discount line.

Now the compute of prepayment amount is made without join the order lines to prevent
duplicate the payment schedule paidamt.

Mon, 14 Aug 2017 12:42:12 +0200Fixes Issue 36648. Changes definition of 'In Transit' Inventory Status
David Miguelez <david.miguelez@openbravo.com> [Mon, 14 Aug 2017 12:42:12 +0200] rev 32559
Fixes Issue 36648. Changes definition of 'In Transit' Inventory Status
to Available = 'Y'

Mon, 17 Jul 2017 18:28:09 +0200Fixed issue 36324: Error doing cancel and replace of an already paid and returned receipt.
Asier Martirena <asier.martirena@openbravo.com> [Mon, 17 Jul 2017 18:28:09 +0200] rev 32558
Fixed issue 36324: Error doing cancel and replace of an already paid and returned receipt.

Also changed the C&R process to work correctly with invoices. Now, after doing a C&R, the lines that were delivered in the initial ticket creates also an invoice line, even if the line doesn't have a goods shipment related.
Managed also the C&R properties. If a netting shipment is created, all invoice lines now has a shipment related, and if the shipment lines are moved from the original ticket to the new ticket, the original invoice lines (if there are) don't have related shipment lines no more and the new invoice lines will be related to the moved shipment lines.

Fri, 11 Aug 2017 12:17:29 +0200fixed bug 36609: unneeded flushes + commit on every FIC execution
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 11 Aug 2017 12:17:29 +0200] rev 32557
fixed bug 36609: unneeded flushes + commit on every FIC execution

When executing FIC, before invoking callouts, a flush + commit (including an
extra flush) was always performed to release any lock that could have been
acquired before.

That code was obsolete, as currently FIC does not perform any DB change before
that point.

Fri, 11 Aug 2017 11:59:39 +0200fixed bug 36636: double flush using DalConnectionProvider(true) if con==null
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 11 Aug 2017 11:59:39 +0200] rev 32556
fixed bug 36636: double flush using DalConnectionProvider(true) if con==null

When trying to get connection from DalConnectionProvider having set flush=true,
extra flush is performed in case there's no connection or it's already closed.

As flush is performed later, this case should alway retrieve connection preventing
flush.