Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0030481Openbravo ERP03. Procurement managementpublic2015-08-03 10:032015-08-25 15:48
ReporterVictorVillar 
Assigned ToAtulOpenbravo 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target Version3.0PR15Q4Fixed in Version 
Merge Request Status
Review Assigned Todmiguelez
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0030481: Wrong quantity conversion in goods receipts under some circumstancies
DescriptionWrong quantity conversion in goods receipts under some circumstancies.
Steps To ReproduceAs System Administrator:
  show Conversion tab of the UOM window
  UOM tab of Product window
  Order quanty and Order UOM fields of Purchase order window || lines tab
  Order quanty and Order UOM fields of Goods Receipt window || lines tab

As group admin role,
 Go to Unit of measure window
     select kilogram unit of measue
     Conversion tab:
         Add Unit: Multiply rate by 3.6
 Go to Product window,
    create a new product.
    Name: secondUOM
    UOM: Unit
   Price tab:
       Tarifas Bebidas Alegres
   UOM tab:
     Kilogram

Go to Purchase Order window,
   create a new one for Business Partner: Bebidas Alegres
  Lines tab:
    Product: secondUOM
    Order qty: 20
    Order UOM: Kilogram
So UOM: Unit
   Ordered qty: 72
Complete it.

Go to Pending Goods Receipts window,
   select the purchase order created before and process it.

Go to Goods Receipt window,
   realise that the movement qty is 72 and the order qty is 259.2
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 00279773.0PR15Q2 closed reinaldoguerra Error obtained in Pending Goods Receipt when working with secondary UOM and conversion is only defined from second UOM to first 
depends on backport 00306443.0PR15Q3.1 closed AtulOpenbravo Wrong quantity conversion in goods receipts under some circumstancies 
depends on backport 00306453.0PR15Q2.4 closed AtulOpenbravo Wrong quantity conversion in goods receipts under some circumstancies 
related to defect 0032943 closed markmm82 'Fix Stock QuantityOrder Process' should round the Order quantity of the opening physical inventory to the std precision 
related to defect 0032937 closed markmm82 Wrong parameter is passed to the c_uom_convert function 
related to defect 0034192 closed markmm82 Quantity in second UOM does not match in purchase order line and goods receipt line 
blocks defect 00311853.0PR16Q1 closed Triage Omni WMS Wrong quantity conversion in goods shipment under some circumstancies 
blocks defect 00311863.0PR16Q1 closed aferraz org.openbravo.fix.stock.qtyorderonhand module is not working with negative stock 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-08-03 10:03VictorVillarNew Issue
2015-08-03 10:03VictorVillarAssigned To => Triage Finance
2015-08-03 10:03VictorVillarModules => Core
2015-08-03 10:03VictorVillarResolution time => 1441231200
2015-08-03 10:03VictorVillarTriggers an Emergency Pack => No
2015-08-03 10:04VictorVillarIssue Monitored: networkb
2015-08-03 10:04VictorVillarNote Added: 0079163
2015-08-10 07:15AtulOpenbravoAssigned ToTriage Finance => AtulOpenbravo
2015-08-10 07:15AtulOpenbravoStatusnew => scheduled
2015-08-14 07:02AtulOpenbravoRegression date => 2014-11-12
2015-08-14 07:02AtulOpenbravoRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/e565273a954e [^]
2015-08-20 14:06aferrazRegression date2014-11-12 =>
2015-08-20 14:06aferrazRegression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/e565273a954e [^] =>
2015-08-21 08:56aferrazRelationship addedrelated to 0027977
2015-08-24 11:54vmromanosStatusscheduled => acknowledged
2015-08-24 11:55vmromanosStatusacknowledged => scheduled
2015-08-25 09:15hgbotCheckin
2015-08-25 09:15hgbotNote Added: 0079665
2015-08-25 09:15hgbotStatusscheduled => resolved
2015-08-25 09:15hgbotResolutionopen => fixed
2015-08-25 09:15hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/2e09807430444dc374dc9762a85662e6c19b3a49 [^]
2015-08-25 14:09hgbotCheckin
2015-08-25 14:09hgbotNote Added: 0079702
2015-08-25 14:09dmiguelezNote Added: 0079703
2015-08-25 14:09dmiguelezReview Assigned To => dmiguelez
2015-08-25 14:09dmiguelezNote Added: 0079704
2015-08-25 14:09dmiguelezStatusresolved => closed
2015-08-25 15:48hudsonbotCheckin
2015-08-25 15:48hudsonbotNote Added: 0079725
2015-10-20 12:39VictorVillarRelationship addedblocks 0031185
2015-10-20 12:44VictorVillarRelationship addedblocks 0031186
2016-05-16 11:06ngarciaRelationship addedrelated to 0032943
2016-05-16 11:40ngarciaRelationship addedrelated to 0032937
2016-10-11 19:15ngarciaRelationship addedrelated to 0034192

Notes
(0079163)
VictorVillar   
2015-08-03 10:04   
It is necessary a script to fix the data.
Besides, it might be a regression.
(0079665)
hgbot   
2015-08-25 09:15   
Repository: erp/devel/pi
Changeset: 2e09807430444dc374dc9762a85662e6c19b3a49
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Thu Aug 13 10:45:12 2015 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/2e09807430444dc374dc9762a85662e6c19b3a49 [^]

Fixes Issue 30481:Wrong quantity conversion in goods receipts under some
circumstancies

When using conversion from primary UOM to secondary uom divideby rate
should be used and when vice versa multiply rate should be used.

---
M src/org/openbravo/erpCommon/ad_forms/MaterialReceiptPending.java
---
(0079702)
hgbot   
2015-08-25 14:09   
Repository: erp/pmods/org.openbravo.fix.stock.qtyorderonhand
Changeset: 2cd70f1f626dc414e119d73c3058018beb14002f
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Tue Aug 25 09:25:09 2015 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.fix.stock.qtyorderonhand/rev/2cd70f1f626dc414e119d73c3058018beb14002f [^]

Related to issue 30481: Fix wrong data

New module created with a Java process to fix wrong data.
Process will create a close physical inventory for wrong stock lines with quantity and order quantity setted as 0, and an open physical inventory for the same lines with the same quantity and the correct order quantity.

---
R modules/org.openbravo.fix.stock.qtyorderonhand/src-db/database/sourcedata/AD_MENU.xml
R modules/org.openbravo.fix.stock.qtyorderonhand/src-db/database/sourcedata/AD_MODEL_OBJECT.xml
R modules/org.openbravo.fix.stock.qtyorderonhand/src-db/database/sourcedata/AD_MODULE.xml
R modules/org.openbravo.fix.stock.qtyorderonhand/src-db/database/sourcedata/AD_MODULE_DBPREFIX.xml
R modules/org.openbravo.fix.stock.qtyorderonhand/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
R modules/org.openbravo.fix.stock.qtyorderonhand/src-db/database/sourcedata/AD_PROCESS.xml
R modules/org.openbravo.fix.stock.qtyorderonhand/src-db/database/sourcedata/AD_TREENODE.xml
R modules/org.openbravo.fix.stock.qtyorderonhand/src/org/openbravo/fix/stock/qtyorderonhand/FixStockQtyOrderProcess.java
---
(0079703)
dmiguelez   
2015-08-25 14:09   
Repo with the Process that fixes wrong data:
https://code.openbravo.com/erp/pmods/org.openbravo.fix.stock.qtyorderonhand [^]
(0079704)
dmiguelez   
2015-08-25 14:09   
Code Review + Testing Ok
(0079725)
hudsonbot   
2015-08-25 15:48   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/e847ea2dfcc5 [^]
Maturity status: Test