Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0031972Openbravo ERP04. Warehouse managementpublic2016-01-21 10:392016-03-17 10:55
Reporterngarcia 
Assigned Toaferraz 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Todmiguelez
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2014-12-18
Regression introduced in release3.0PR15Q1
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/8dd8af8a26e5 [^]
Triggers an Emergency PackNo
Summary0031972: Costing Migration Process:Cost of products in closing inventory is calculated by taking into account only shipments and receipts
DescriptionCosting Migration Process:Cost of products in closing inventory is calculated by taking into account only shipments and receipts.

The problem is that insertTrxCost function is making an inner join between the M_TRANSACTION AND M_INOUTLINE tables
Steps To ReproduceCheck that after launching the first phase of the Costing Migration process the average cost of the products in the closing inventories is calculated only with the values of the transactions related with goods shipments and goods receipts
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 00319973.0PR16Q1 closed Triage Omni WMS Costing Migration Process:Cost of products in closing inventory is calculated by taking into account only shipments and receipts 
depends on backport 00319983.0PR15Q4.3 closed Triage Omni WMS Costing Migration Process:Cost of products in closing inventory is calculated by taking into account only shipments and receipts 
depends on backport 00319993.0PR15Q3.6 closed Triage Omni WMS Costing Migration Process:Cost of products in closing inventory is calculated by taking into account only shipments and receipts 
caused by defect 0028450 closed umartirena [Costing] Costing Migration Process fails 
related to defect 0031979 closed aferraz Costing Migration Process: processed column of m_transaction table is not set as Y for records with its cost calculated 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-01-21 10:39ngarciaNew Issue
2016-01-21 10:39ngarciaAssigned To => Triage Finance
2016-01-21 10:39ngarciaModules => Core
2016-01-21 10:39ngarciaTriggers an Emergency Pack => No
2016-01-21 10:40ngarciaIssue Monitored: networkb
2016-01-21 10:40ngarciaRelationship addedcaused by 0028450
2016-01-21 11:50ngarciaRelationship addedrelated to 0031979
2016-01-21 13:02ngarciaResolution time => 1456009200
2016-01-21 16:11ahernandezgilIssue Monitored: ahernandezgil
2016-01-22 14:51aferrazStatusnew => scheduled
2016-01-22 14:51aferrazAssigned ToTriage Finance => aferraz
2016-01-22 15:08aferrazRegression level => Production - Confirmed Stable
2016-01-22 15:08aferrazRegression date => 2014-12-18
2016-01-22 15:08aferrazRegression introduced in release => 3.0PR15Q1
2016-01-22 15:08aferrazRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/8dd8af8a26e5 [^]
2016-01-25 12:14hgbotCheckin
2016-01-25 12:14hgbotNote Added: 0083587
2016-01-25 12:14hgbotStatusscheduled => resolved
2016-01-25 12:14hgbotResolutionopen => fixed
2016-01-25 12:14hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/0159f66bd6e0d64d86b01b9a8a501fd048c20019 [^]
2016-01-25 12:14dmiguelezReview Assigned To => dmiguelez
2016-01-25 12:14dmiguelezNote Added: 0083588
2016-01-25 12:14dmiguelezStatusresolved => closed
2016-03-17 10:55hudsonbotCheckin
2016-03-17 10:55hudsonbotNote Added: 0085079

Notes
(0083587)
hgbot   
2016-01-25 12:14   
Repository: erp/devel/pi
Changeset: 0159f66bd6e0d64d86b01b9a8a501fd048c20019
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Thu Jan 21 14:27:39 2016 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/0159f66bd6e0d64d86b01b9a8a501fd048c20019 [^]

Fixes issue 31972: Wrong cost calculation in Costing Migration Process

HQL in insertTrxCosts method in CostingMigrationProcess class was doing an inner join between m_transaction and m_inoutline instead of a left join.

---
M src/org/openbravo/costing/CostingMigrationProcess.java
---
(0083588)
dmiguelez   
2016-01-25 12:14   
Code Review + Testing Ok
(0085079)
hudsonbot   
2016-03-17 10:55   
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/b22fb0500156 [^]
Maturity status: Test