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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0028019
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 04. Warehouse managementmajoralways2014-10-29 20:312014-12-30 23:24
ReporteraferrazView Statuspublic 
Assigned Toumartirena 
PriorityhighResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisionf679bcf20802
ProjectionnoneETAnoneTarget Version
OSLinux 64 bitDatabasePostgreSQLJava version1.6.0_18
OS VersionProfessional ApplianceDatabase version8.3.9Ant version1.7.1
Product VersionSCM revision 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0028019: New lines created in costing tab which should not be there

DescriptionNew lines created in costing tab which should not be there
Steps To ReproduceIn "QA Testing" client:

Navigate to Warehouse and Storage Bin and create a new warehouse for Spain organization, with name "Spain West warehouse" and Search Key "spainWWH". Navigate to Storage Bin tab and create a new storage bin with name "M01", row M, stack 0, level 1 and priority 50.

Navigate to Organization window, select Spain organization and go to Warehouse tab. Create a new record with warehouse Spain West warehouse and priority 20.

Create a new Product named "Costing GM1" for Spain organization.
Go to Price tab, select "Purchase" price list and enter a unit/list price of 11 €.

Go to Purchase Order window and create a new order for Spain organization, product "Costing GM1", order date 29-10-2014, quantity 820 and price 11.
Book the order. Let's say this is order 801370.

Go to Goods Receipt window and create a new receipt dated on 03-11-2014. Click on Create Lines From, select order 801370, select storage bin L01 and enter 820 units. Complete. Let's say this is receipt 10001795.

Navigate to Process Request window and run "Costing Background process".

Go to receipt window and post the receipt 10001795.
Realize that the value posted to the ledger is 9,020.00 (820 units * 11 €/unit)

Go to Goods Movements window and create a new movement dated on 13-11-2014, name "Costing GM1 - 13-11-2014", product Costing GM1, quantity 400, storage bin L01 (Spain warehouse) and new storage bin M01 (Spain West warehouse). Click on Process Movements. Let's say this is movement 10000059.

Navigate to Process Request window and run "Costing Background process".

Go to movement window and post the movement 10000059.
Realize that the value posted to the ledger is 4,400.00 (400 units * 11 €/unit)

Navigate to product window, select product Costing GM1 and go to Transactions tab. Select the second record (Movement From). Click on Manual Cost Adjustment, select date 18-11-2014, total cost amount 4,000.00 and incremental false.

Go to Purchase Invoice window and create a new invoice for Spain organization, invoice date 08-11-2014. Click on Create Lines From, select receipt 10001795 and change the unit price to 15. Complete the invoice and post it. Let's say this is invoice 10000925.
Realize that the value posted to the ledger is 12,300.00 (820 units * 15 €/unit)

Navigate to Process Request window and run "Price Correction Background".

Navigate to Product window, select Costing GM1 and go to Costing tab, realize that there are 4 records:

- Costing 1: Cost 15 - Transaction Receipt 10001795 - Quantity 820 - Warehouse Spain warehouse

- Costing 2: Cost 10 - Transaction Movement 10000059 - Quantity 400 - Warehouse Spain West warehouse

- Costing 3: Cost 11.95 - Transaction null - Quantity null - Warehouse Spain warehouse

- Costing 4: Cost 15 - Transaction null - Quantity null - Warehouse Spain warehouse

This is not correct, correct should be:


- Costing 1: Cost 15 - Transaction Receipt 10001795 - Quantity 820 - Warehouse Spain warehouse

- Costing 2: Cost 10 - Transaction Movement 10000059 - Quantity 400 - Warehouse Spain West warehouse

- Costing 3: Cost 11.95 - Transaction null - Quantity null - Warehouse Spain warehouse

- Costing 4: Cost 19.76 - Transaction null - Quantity null - Warehouse Spain warehouse

Proposed SolutionSee above

See also: https://issues.openbravo.com/view.php?id=27884 [^]
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]

-  Notes
(0071393)
hgbot (developer)
2014-11-04 10:54

 Repository: erp/devel/pi
Changeset: f679bcf20802033600bfeffadcbc5ca5e6d00b22
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Tue Nov 04 10:20:30 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/f679bcf20802033600bfeffadcbc5ca5e6d00b22 [^]

Fixes issue 28019: Recalculate the cost at the end of averageCostAdjustment

The extra lines in Costing tab in this case are correct, because there is an out transaction (movement from), and these transactions they dont modify the average cost.
Also, at the end of the Average Cost Adjustment the last cost has to be recalculated in order to prevent more out transactions with bad cost until another in transaction happens that recalculates the cost.

---
M src/org/openbravo/costing/AverageCostAdjustment.java
---
(0071454)
psanjuan (viewer)
2014-11-06 11:35

 Verified for:
Incremental = NO
Increment = Yes & Unit Cost = NO
Incremental = Yes & Unit Cost =Yes

See excel sheet below, item "Costing GM1"
https://docs.google.com/a/openbravo.com/spreadsheets/d/1Ilm1oct0vH4iKFlI1I0VbmYM-r8fdo1moK02jd2Zigg/edit#gid=1046089920 [^]
(0071500)
hgbot (developer)
2014-11-07 13:36

 Repository: erp/devel/pi
Changeset: 90b35b04c6afabfba28c6a03d5a95c1f2341c3ba
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Fri Nov 07 13:36:16 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/90b35b04c6afabfba28c6a03d5a95c1f2341c3ba [^]

Fixes Issue 28114: Remove cost re-calculation related to output transactions

The cost recalculation related to adjustments on output transactions has been removed because it was causing amounts rounding issues and because of this extra records in Costing tab were being created. This recalculation was being doing because of reported issue 28019.

---
M src/org/openbravo/costing/AverageCostAdjustment.java
---
(0073014)
hudsonbot (viewer)
2014-12-30 23:24

 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/6525fe229e06 [^]
Maturity status: Test
(0073030)
hudsonbot (viewer)
2014-12-30 23:24

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

- Issue History
Date Modified Username Field Change
2014-10-29 20:31 aferraz New Issue
2014-10-29 20:31 aferraz Assigned To => umartirena
2014-10-29 20:31 aferraz OBNetwork customer => No
2014-10-29 20:31 aferraz Modules => Core
2014-10-29 20:31 aferraz Triggers an Emergency Pack => No
2014-10-30 09:14 aferraz Steps to Reproduce Updated View Revisions
2014-10-30 17:50 aferraz Steps to Reproduce Updated View Revisions
2014-11-04 10:54 hgbot Checkin
2014-11-04 10:54 hgbot Note Added: 0071393
2014-11-04 10:54 hgbot Status new => resolved
2014-11-04 10:54 hgbot Resolution open => fixed
2014-11-04 10:54 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/f679bcf20802033600bfeffadcbc5ca5e6d00b22 [^]
2014-11-06 11:32 psanjuan Status resolved => closed
2014-11-06 11:32 psanjuan Steps to Reproduce Updated View Revisions
2014-11-06 11:35 psanjuan Note Added: 0071454
2014-11-06 13:54 psanjuan Steps to Reproduce Updated View Revisions
2014-11-07 13:36 hgbot Checkin
2014-11-07 13:36 hgbot Note Added: 0071500
2014-12-30 23:24 hudsonbot Checkin
2014-12-30 23:24 hudsonbot Note Added: 0073014
2014-12-30 23:24 hudsonbot Checkin
2014-12-30 23:24 hudsonbot Note Added: 0073030

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker