Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0032937
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 04. Warehouse managementmajoralways2016-05-11 19:052016-06-17 19:37
ReporterngarciaView Statuspublic 
Assigned Tomarkmm82 
PriorityurgentResolutionfixedFixed in Version3.0PR16Q3
StatusclosedFix in branchFixed in SCM revision121bcf7ae05b
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Toaferraz
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0032937: Wrong parameter is passed to the c_uom_convert function

DescriptionWrong parameter is passed to the c_uom_convert function: it should be the c_uom_id associated to the m_product_uom_id but not the m_product_uom_id:

C_ORDERLINE2_TRG.xml:
         v_QtyOrderReserved:=C_Uom_Convert(v_QtyReserved, :new.C_UOM_ID, :new.m_product_uom_id, 'Y') ;

C_ORDER_POST1.xml:
 
v_QtyOrderPO := -C_Uom_Convert(v_QtyPO, Cur_ResLine.C_UOM_ID, Cur_ResLine.M_Product_UOM_ID, 'Y');

v_QtyOrderSO := -C_Uom_Convert(v_QtySO, Cur_ResLine.C_UOM_ID, Cur_ResLine.M_Product_UOM_ID, 'Y');

v_QtyOrderSO := C_Uom_Convert(v_QtySO, Cur_ResLine.C_UOM_ID, Cur_ResLine.M_Product_UOM_ID, 'Y');

v_QtyOrderPO := C_Uom_Convert(v_QtyPO, Cur_ResLine.C_UOM_ID, Cur_ResLine.M_Product_UOM_ID, 'Y');
Steps To Reproduce*
Proposed SolutionAttached a patch to be analyzed
TagsNo tags attached.
Attached Filesdiff file icon FixUOMConvert.diff [^] (3,496 bytes) 2016-05-16 13:10 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
related to defect 00229133.0MP21 closedjecharri Second UOM not shown when creating a receipt from an order that has been partially received 
related to defect 00304813.0PR15Q4 closedAtulOpenbravo Wrong quantity conversion in goods receipts under some circumstancies 

-  Notes
(0086724)
hgbot (developer)
2016-05-25 10:27

Repository: erp/devel/pi
Changeset: 121bcf7ae05b5e9cd1331b0c180570ea5f5d141b
Author: Mark <markmm82 <at> gmail.com>
Date: Tue May 17 11:39:36 2016 -0400
URL: http://code.openbravo.com/erp/devel/pi/rev/121bcf7ae05b5e9cd1331b0c180570ea5f5d141b [^]

Fixes issue 32937: Correct parameter is passed to the c_uom_convert function

The correct parameter is passed to the c_uom_convert function: it is the c_uom_id associated to the m_product_uom_id instead the m_product_uom_id.

---
M src-db/database/model/functions/C_ORDER_POST1.xml
M src-db/database/model/triggers/C_ORDERLINE2_TRG.xml
---
(0086725)
hgbot (developer)
2016-05-25 10:27

Repository: erp/devel/pi
Changeset: 8d4bd74e71c7260c42df2e1a25ee93e52cfb11a0
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Tue May 24 18:11:55 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/8d4bd74e71c7260c42df2e1a25ee93e52cfb11a0 [^]

Related to issue 32937: Code review improvements

---
M src-db/database/model/functions/C_ORDER_POST1.xml
M src-db/database/model/functions/M_INOUT_CREATE.xml
M src-db/database/model/triggers/C_ORDERLINE2_TRG.xml
---
(0086730)
aferraz (manager)
2016-05-25 10:35

Code review + Testing OK
(0087564)
hudsonbot (developer)
2016-06-17 19:37

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/0dc7be081b1c [^]
Maturity status: Test
(0087565)
hudsonbot (developer)
2016-06-17 19:37

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/0dc7be081b1c [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2016-05-11 19:05 ngarcia New Issue
2016-05-11 19:05 ngarcia Assigned To => Triage Finance
2016-05-11 19:05 ngarcia Modules => Core
2016-05-11 19:05 ngarcia Resolution time => 1466200800
2016-05-11 19:05 ngarcia Triggers an Emergency Pack => No
2016-05-11 19:06 ngarcia Issue Monitored: networkb
2016-05-11 19:06 ngarcia Relationship added related to 0022913
2016-05-12 09:26 aferraz Assigned To Triage Finance => markmm82
2016-05-16 11:40 ngarcia Relationship added related to 0030481
2016-05-16 13:10 ngarcia File Added: FixUOMConvert.diff
2016-05-16 13:10 ngarcia Proposed Solution updated
2016-05-17 17:17 markmm82 Status new => scheduled
2016-05-25 10:27 hgbot Checkin
2016-05-25 10:27 hgbot Note Added: 0086724
2016-05-25 10:27 hgbot Status scheduled => resolved
2016-05-25 10:27 hgbot Resolution open => fixed
2016-05-25 10:27 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/121bcf7ae05b5e9cd1331b0c180570ea5f5d141b [^]
2016-05-25 10:27 hgbot Checkin
2016-05-25 10:27 hgbot Note Added: 0086725
2016-05-25 10:35 aferraz Review Assigned To => aferraz
2016-05-25 10:35 aferraz Note Added: 0086730
2016-05-25 10:35 aferraz Status resolved => closed
2016-05-25 10:35 aferraz Fixed in Version => 3.0PR16Q3
2016-06-17 19:37 hudsonbot Checkin
2016-06-17 19:37 hudsonbot Note Added: 0087564
2016-06-17 19:37 hudsonbot Checkin
2016-06-17 19:37 hudsonbot Note Added: 0087565


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker