Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0041327Openbravo ERP07. Sales managementpublic2019-07-08 12:022019-07-11 20:19
inaki_garcia 
collazoandy4 
immediatemajoralways
closedfixed 
5
pi 
3.0PR19Q1.23.0PR19Q1.2 
Sandrahuguet
Core
Automated tests
2019-07-03
https://code.openbravo.com/erp/devel/pi/rev/f831b0b71a5ec700751e645c9e6980001acd8c7b [^]
No
0041327: [AUoM] Operative Quantity isn't correctly calculated when Creating Lines in Purchase Invoice from a Return To Vendor order
When creating lines from a Return To Vendor order in the Purchase Invoice window, those that make use of Alternate Unit of Measure don't have their Operative Quantity
correctly calculated, in accordance to the Conversion Rate of the alternate unit as the assertion of the AUMRTVc010InvoiceAReturnToVendorShipment test expects it.

This issue has arised from thanks to qa-pi-aum jobs executing under the QA tab of CI, and with the mentioned test. More details of one of the latest execution showing the
issue can be found in the following link:
https://ci.openbravo.com/view/qa/job/qa-pi-aum-pgsql/485/ [^]
As group admin role:
   Set 'Enable UOM Management' preference to Y
   Log out and log in
   Add Liter unit to 'Cerveza Ale 0,5L' in Alternate UOM tab as Primary UOM
   Add a conversion of 2
   Create a Goods Receipt
   Add 10 Liters of 'Cerveza Ale 0,5L':
      Operative Quantity: 10
      Alternative UOM: Liter
      Ordered Quantity: 20
      Storage Bin: O01
      UOM: Unit
   Book it
   Go to Return To Vendor
   Open the Pick/Edit Lines popup to select the lines of the previously created Goods Receipt
   Check the quantities and other fields are shown as in the order line in the pop-up
   Set the "returned" field to 3, "returned UOM" to "Liter" and "return reason" to "Defective"
   Check the Alternative UOM is updated with Unit in the Return To Vendor line:
      Movement Quantity: 6
   Book it
   Go to Return to Vendor Shipment, and use the Pick/Edit Lines popup to create the lines from the Goods Receipt, and book it
   Go to Purchase Invoice window, and create a new record by importing the lines with the Return To Vendor.
   NOTICE that the Operative Quantity shows as "-6" when the value expected by the test is "-3"
   
No tags attached.
blocks defect 0041315 closed collazoandy4 [AUoM] Operative Quantity isn't correctly calculated when Creating Lines in Purchase Invoice from a Return To Vendor order 
Issue History
2019-07-10 08:54SandrahuguetTypedefect => backport
2019-07-10 08:54SandrahuguetTarget Version => 3.0PR19Q3.1
2019-07-10 08:54SandrahuguetTarget Version3.0PR19Q3.1 => 3.0PR19Q1.3
2019-07-10 09:54SandrahuguetTarget Version3.0PR19Q1.3 => 3.0PR19Q1.2
2019-07-11 06:28priyamIssue Monitored: priyam
2019-07-11 09:41inaki_garciaSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=19084#r19084
2019-07-11 20:17hgbotCheckin
2019-07-11 20:17hgbotNote Added: 0113307
2019-07-11 20:17hgbotStatusscheduled => resolved
2019-07-11 20:17hgbotResolutionopen => fixed
2019-07-11 20:17hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR19Q1.2/rev/4640b9a054db47670ccd7976dab7a90febfad7c8 [^]
2019-07-11 20:17hgbotCheckin
2019-07-11 20:17hgbotNote Added: 0113308
2019-07-11 20:19SandrahuguetReview Assigned To => Sandrahuguet
2019-07-11 20:19SandrahuguetNote Added: 0113314
2019-07-11 20:19SandrahuguetStatusresolved => closed
2019-07-11 20:19SandrahuguetFixed in Version => 3.0PR19Q1.2

Notes
(0113307)
hgbot   
2019-07-11 20:17   
Repository: erp/backports/3.0PR19Q1.2
Changeset: 4640b9a054db47670ccd7976dab7a90febfad7c8
Author: Armaignac <collazoandy4 <at> gmail.com>
Date: Thu Jul 11 10:53:26 2019 -0400
URL: http://code.openbravo.com/erp/backports/3.0PR19Q1.2/rev/4640b9a054db47670ccd7976dab7a90febfad7c8 [^]

Fixes issue 41327: [AUoM] Operative Quantity isn't correctly calculated when
Creating Lines in Purchase Invoice from a Return To Vendor order

A wrong operative UOM was used to create the new invoiceline if the orderline is
related to a shipment, the quantity is calculated in the P&E and the UOM from the copied
line.

Now when a line is created from an orderline and is related to a not invoiced shipmentline
this shipmentline is used to create the new invoiceline, in case the orderline is a return
the return to vendor shipment line is used to create a new invoiceline

---
M src/org/openbravo/common/actionhandler/createlinesfromprocess/CreateLinesFromUtil.java
M src/org/openbravo/common/actionhandler/createlinesfromprocess/UpdateQuantitiesAndUOMs.java
---
(0113308)
hgbot   
2019-07-11 20:17   
Repository: erp/backports/3.0PR19Q1.2
Changeset: ac17e6d2ff4f57ce6bdc0b9537bef6ecf34e446d
Author: Sandra Huguet <sandra.huguet <at> openbravo.com>
Date: Thu Jul 11 17:35:58 2019 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR19Q1.2/rev/ac17e6d2ff4f57ce6bdc0b9537bef6ecf34e446d [^]

related to issue 41327 code review changes

---
M src/org/openbravo/common/actionhandler/createlinesfromprocess/CreateLinesFromUtil.java
---
(0113314)
Sandrahuguet   
2019-07-11 20:19   
code review + testing ok