0028121: Transaction total cost is not property calculated

Description

Transaction total cost is not property calculated

Steps To Reproduce

In "QA Testing" client:

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

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

Go to Goods Receipt window and create a new receipt dated on 08-11-2014. Click on Create Lines From, select order 801370, select storage bin L01 and enter 320 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 38,400.00 (320 units * 120 €/unit)

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

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

Go to receipt window and post the receipt 10001796.
Realize that the value posted to the ledger is 21,600.00 (180 units * 120 €/unit)

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

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

Go to receipt window and post the receipt 10001797.
Realize that the value posted to the ledger is 36,000.00 (300 units * 120 €/unit)

Go to Purchase Invoice window and create a new invoice for Spain organization, invoice date 07-11-2014, financial invoice line true, account Fees, quantity 3 and unit price 95. Complete the invoice and post it. Let's say this is invoice 10000923.
Realize that the value posted to the ledger is 285.00 (3 units * 95 €/unit)

Go to Purchase Invoice window and create a new invoice for Spain organization, invoice date 07-11-2014, financial invoice line false, product Transportation Cost, quantity 3 and unit price 105. Complete the invoice and post it. Let's say this is invoice 10000924.
Realize that the value posted to the ledger is 346.50 (3 units * 105 €/unit + (3 units * 105 €/unit * 0.10 tax))

Go to Landed Cost window and create a new landed cost for Spain organization, reference date 07-11-2014. Move to Cost tab and add a new line with landed cost type Fees and invoice line the line of invoice 10000923. Add another line with landed cost type Transportation Cost and invoice line the line of invoice 10000924. Move to Receipt tab and add a new line with goods receipt 10001795. Add another line with goods receipt 10001796. Add another line with goods receipt 10001797. Click on Process button.

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

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

Navigate to Product window, select Costing LC2 and go to Costing tab, realize that there are 3 records with cost 130.75 ((104,000 + 285 + 315) / 800):

- Costing 1: Cost 130.75 - Transaction Receipt 10001795 - Quantity 320

- Costing 2: Cost 130.75 - Transaction Receipt 10001796 - Quantity 180

- Costing 3: Cost 130.75 - Transaction Receipt 10001797 - Quantity 300

Go to Transactions tab, realize that there are 3 records:

- Transaction 1: Receipt 10001795 - Transaction Original Cost 38,400.00 - Total Cost 41,839.98 - Unit Cost 41,600.00

- Transaction 2: Receipt 10001796 - Transaction Original Cost 21,600.00 - Total Cost 23,535.01 - Unit Cost 23,400.00

- Transaction 3: Receipt 10001797 - Transaction Original Cost 36,000.00 - Total Cost 39,225.01 - Unit Cost 39,000.00

This is not correct, correct should be:

- Transaction 1: Receipt 10001795 - Transaction Original Cost 38,400.00 - Total Cost 41,840.00 (320 * 130.75) - Unit Cost 41,600.00

- Transaction 2: Receipt 10001796 - Transaction Original Cost 21,600.00 - Total Cost 23,535.00 (180 * 130.75) - Unit Cost 23,400.00

- Transaction 3: Receipt 10001797 - Transaction Original Cost 36,000.00 - Total Cost 39,225.00 (300 * 130.75) - Unit Cost 39,000.00

Proposed Solution

See above

Tags

No tags attached.

Attached Files

Relationships [ Relation Graph ] [ Dependency Graph ]

Notes
(0071547) hgbot (developer) 2014-11-11 10:01	Repository: erp/devel/pi Changeset: 603663ae3bb06cfc85c691aa0c63bdfa938be9eb Author: Unai Martirena <unai.martirena <at> openbravo.com> Date: Tue Nov 11 09:54:03 2014 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/603663ae3bb06cfc85c691aa0c63bdfa938be9eb [^] Fixes Issue 28121: Used Costing Precision instead of Standard Precision in LC In Distribution by Amount Algorithm of Landed Cost, Costing Precision should be used instead of Standard Precision. The recommended Costing Precision is 4. Also the Line No. of Cost Adjustment Lines has been fixed when creating a Landed Cost Adjustment. --- M src/org/openbravo/costing/LandedCostDistributionByAmount.java M src/org/openbravo/costing/LandedCostProcess.java ---

(0071553) psanjuan (manager) 2014-11-11 12:52	Verified. See test scenario related to products: "LC 400" and "LC 400 USD" in this excel sheet. https://docs.google.com/a/openbravo.com/spreadsheets/d/1Ilm1oct0vH4iKFlI1I0VbmYM-r8fdo1moK02jd2Zigg/edit#gid=726064952 [^]

(0073039) hudsonbot (developer) 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-11-07 12:33	aferraz	New Issue
2014-11-07 12:33	aferraz	Assigned To	=> umartirena
2014-11-07 12:33	aferraz	Modules	=> Core
2014-11-07 12:33	aferraz	Triggers an Emergency Pack	=> No
2014-11-10 12:37	aferraz	Steps to Reproduce Updated	View Revisions
2014-11-11 10:01	hgbot	Checkin
2014-11-11 10:01	hgbot	Note Added: 0071547
2014-11-11 10:01	hgbot	Status	new => resolved
2014-11-11 10:01	hgbot	Resolution	open => fixed
2014-11-11 10:01	hgbot	Fixed in SCM revision	=> http://code.openbravo.com/erp/devel/pi/rev/603663ae3bb06cfc85c691aa0c63bdfa938be9eb [^]
2014-11-11 12:52	psanjuan	Note Added: 0071553
2014-11-11 12:52	psanjuan	Status	resolved => closed
2014-12-30 23:24	hudsonbot	Checkin
2014-12-30 23:24	hudsonbot	Note Added: 0073039