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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0028192
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 04. Warehouse managementmajorhave not tried2014-11-17 15:032014-12-30 23:26
ReporteraferrazView Statuspublic 
Assigned Toumartirena 
PriorityurgentResolutionfixedFixed in Versionpi
StatusclosedFix in branchFixed in SCM revisionbc6fcc83e7dc
ProjectionnoneETAnoneTarget Version
OSLinux 64 bitDatabasePostgreSQLJava version1.6.0_18
OS VersionProfessional ApplianceDatabase version8.3.9Ant version1.7.1
Product VersionSCM revision 
Review Assigned ToSandrahuguet
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0028192: [Costing] Net Unit Price field in Costing tab is not properly calculated

Description[Costing] Net Unit Price field in Costing tab is not properly calculated
Steps To ReproduceIn QA Testing client create a new product named "Costing GM12". Select a purchase price list. Enter a unit/list price of 11.00

Go to purchase order window and create a new one dated on 29/10/2014. Enter 820 units. Book the order.

Go to goods receipt window and create a new one dated on 3/11/2014. Select the order and receipt 820 units. Complete the receipt

Go to process request window and launch costing background process.

Go back to goods receipt window and post it. Realize that value posted is 9020 ( 820 units * 11.00 €/unit)

Go to Goods Movement Window and create a new one dated on 13/11/2014

Move 400 units from Spain Warehouse to another warehouse named Spain West Warehouse, for instance. Complete the movement.

Go to process request window and launch costing background process.

Go to goods movement and post it. Realize that the value posted is 4400 ( 400 units + 11 €/unit)

Go to product window and search by the product. Go to transaction tab.
Select Movement From transaction and press the button "Manual Cost Adjustment".

In total cost field enter a total cost amount = -400. Enter 18/11/2014 as accounting date, select the flag "Incremental" but do not select the flag "Unit Cost". Press Done/OK.

After this a purchase invoice needs to be created dated on 8/11/2014. Create it by using Create Lines From, select the receipt. Realize that 820 units are shown. Change the price from 11.00 to 15.00. Complete and post the invoice. Realize that total value posted is 12300 (820 units * 15.00 €/unit).

Go to process request window and run price correction background process.

Go to product window and search by the product. Go to Transaction tab and realize that there are 3 lines:

- Transaction 1: Movement Quantity 820 - Transaction Original Cost 9,020.00 - Total Cost 12,300.00 - Unit Cost 12,300.00

- Transaction 2: Movement Quantity -400 - Transaction Original Cost 4,400.00 - Total Cost 5,600.00 - Unit Cost 6,000.00

- Transaction 3: Movement Quantity 400 - Transaction Original Cost 4,400.00 - Total Cost 5,600.00 - Unit Cost 5,600.00

Go to Costing tab and realize that there are 3 lines:

- Line 1: Cost 15 - Net Unit Price 15.00 - Original Cost 11

- Line 2: Cost 14 - Net Unit Price 14.00 - Original Cost 11

- Line 3: Cost 15.9524 - Net Unit Price 6.00 - Original Cost 11.9524

Line 3 is not correct. It should be:

- Line 3: Cost 15.9524 - Net Unit Price 15.95 (because that is the cost of the Movement From transacton) - Original Cost 11.9524
Proposed SolutionSee above
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0028301 closedumartirena [Costing] Cost not properly calculated after booking two backdated transactions 

-  Notes
(0072280)
hgbot (developer)
2014-12-04 12:21

Repository: erp/devel/pi
Changeset: bc6fcc83e7dc2b93c3b615fc0f540457062f512a
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Thu Dec 04 12:20:59 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/bc6fcc83e7dc2b93c3b615fc0f540457062f512a [^]

Fixes issue 28192: Net Unit Price now is properly calculated.

In order to calculate the Net Unit Price, the adjustments of the transaction are used, but it was not being used the signMultiplier of the transaction, and needs to be used.

---
M src/org/openbravo/costing/AverageCostAdjustment.java
---
(0072303)
psanjuan (manager)
2014-12-05 10:43

Verified.
(0072304)
psanjuan (manager)
2014-12-05 10:48

Issue resolved that was closed by mistake.
Code review required to close the issue.
(0072723)
Sandrahuguet (developer)
2014-12-19 10:44

Code review done
(0073172)
hudsonbot (developer)
2014-12-30 23:26

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-11-17 15:03 aferraz New Issue
2014-11-17 15:03 aferraz Assigned To => Sandrahuguet
2014-11-17 15:03 aferraz Modules => Core
2014-11-17 15:03 aferraz Triggers an Emergency Pack => No
2014-11-18 11:18 psanjuan Summary Net Unit Price field in Costing tab is not properly calculated => [Costing] Net Unit Price field in Costing tab is not properly calculated
2014-11-18 11:18 psanjuan Description Updated View Revisions
2014-12-01 10:40 psanjuan Relationship added related to 0028301
2014-12-01 12:16 umartirena Steps to Reproduce Updated View Revisions
2014-12-04 12:21 hgbot Checkin
2014-12-04 12:21 hgbot Note Added: 0072280
2014-12-04 12:21 hgbot Status new => resolved
2014-12-04 12:21 hgbot Resolution open => fixed
2014-12-04 12:21 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/bc6fcc83e7dc2b93c3b615fc0f540457062f512a [^]
2014-12-04 15:03 psanjuan Steps to Reproduce Updated View Revisions
2014-12-05 10:43 psanjuan Note Added: 0072303
2014-12-05 10:43 psanjuan Status resolved => closed
2014-12-05 10:48 psanjuan Note Added: 0072304
2014-12-05 10:48 psanjuan Status closed => new
2014-12-05 10:48 psanjuan Status new => scheduled
2014-12-05 10:54 umartirena Status scheduled => resolved
2014-12-05 10:54 umartirena Fixed in Version => pi
2014-12-05 10:54 umartirena Fixed in SCM revision http://code.openbravo.com/erp/devel/pi/rev/bc6fcc83e7dc2b93c3b615fc0f540457062f512a [^] => https://code.openbravo.com/erp/devel/pi/rev/bc6fcc83e7dc2b93c3b615fc0f540457062f512a [^]
2014-12-12 08:08 Sandrahuguet Assigned To Sandrahuguet => umartirena
2014-12-12 12:35 Sandrahuguet Review Assigned To => gorkaion
2014-12-19 10:44 Sandrahuguet Review Assigned To gorkaion => Sandrahuguet
2014-12-19 10:44 Sandrahuguet Note Added: 0072723
2014-12-19 10:44 Sandrahuguet Status resolved => closed
2014-12-30 23:26 hudsonbot Checkin
2014-12-30 23:26 hudsonbot Note Added: 0073172


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker