Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0028157Openbravo ERP04. Warehouse managementpublic2014-11-12 14:252014-12-30 23:27
Reporterpsanjuan 
Assigned Toumartirena 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS20OS VersionUbuntu 8.04.1
Product Version 
Target VersionFixed in Version3.0PR15Q1 
Merge Request Status
Review Assigned ToSandrahuguet
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0028157: [Costing] Landed Cost calculation does not take into account exchange rate at purchase invoice (landed cost invoice) level
Description[Costing] Landed Cost calculation does not take into account exchange rate at purchase invoice (landed cost invoice) level
Steps To ReproduceIn QA testing client create a new product named "LC 555". Select a Purchase price list in EUR and enter a unit/list price = 555 eur.

Create another product named "Cost USD". Select a Purchase Price List in USD. Enter a unit/list price = 100 USD.

Go to Landed Cost type window and create a new one related to this product for Spain Org.

Make sure you have a system conversion rate between USD and EUR, let's say = 0.4, in "Conversion Rate" window.

Go to Goods receipt and create a new one for Spain org dated on 12/11/2014. Enter the product, enter an amount = 1 and select an storage bin. Complete the receipt.

Go to process request window and run costing background process.

Go back to the receipt and post it. Realize that the value post is 555.00

Go to Product window and realize that the cost of the product and the cost of the transaction is 555

Go to landed cost window and create a new one for Spain org, dated on 15/11/2014

Navigate to Cost tab and enter landed cost type "Cost USD". Enter an amount = 100. Currency = USD.

Go to Receipt window and select the receipt.

Process and post landed cost. Realize that posting is:
D: Product asset 40.00 (100*0.4)
C: Landed Cost expense account 40.00 (100*0.4)

Go to purchase invoice window and create a new one for Spain org, dated on 30/11/2014. Select a USA Vendor.

Go to lines and enter the product "Cost USD", enter qty = 1, price should be equal to 100 USD.
Go to exchange rate tab an enter a new conversion rate = 0.8
Realize that foreign amount is 80.00 EUR.

Complete and Post the invoice.

In the purchase invoice line click on the process button "Match LC Cost". Select the landed cost and enter a matched amount = 100.
Select "Is matching adjusted" and "Process matching" check-boxes.
Press OK/Done.

Realize that no cost adjustment is created.

Go to product window and search the product. Navigate to Costing tab.
Realize that the cost of the product is still 595 (555 + 40)

Go to product window and search the product. Navigate to Transactions tab.
Realize that the cost of the receipt transaction is still 595 (555 + 40)

Proposed SolutionA new cost adjustment should be created by the difference between 100*0.4 = 40
and 100*0.8 = 80. that is an adjustment of +40 to be allocated to the cost of the product.

The cost of the product should be 555 + 40 + 40 = 635
The total cost of the receipt transaction should be 635
Trx original cost = Unit Cost = 555
Additional Information
TagsNo tags attached.
Relationships
related to defect 0028389 closed umartirena [Costing] Landed Cost Cost posting is not correct in case there is a conversion matching caused by an exchange rate 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2014-11-12 14:25psanjuanNew Issue
2014-11-12 14:25psanjuanAssigned To => dmiguelez
2014-11-12 14:25psanjuanOBNetwork customer => No
2014-11-12 14:25psanjuanModules => Core
2014-11-12 14:25psanjuanTriggers an Emergency Pack => No
2014-11-12 14:41psanjuanSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=7037#r7037
2014-11-12 14:41psanjuanProposed Solution updated
2014-11-13 14:43hgbotCheckin
2014-11-13 14:43hgbotNote Added: 0071638
2014-11-18 15:55SandrahuguetStatusnew => scheduled
2014-11-18 15:55SandrahuguetAssigned Todmiguelez => Sandrahuguet
2014-11-18 15:55Sandrahuguetfix_in_branch => pi
2014-12-10 14:11hgbotCheckin
2014-12-10 14:11hgbotNote Added: 0072400
2014-12-10 14:11hgbotStatusscheduled => resolved
2014-12-10 14:11hgbotResolutionopen => fixed
2014-12-10 14:11hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/28ff19fe345e102bad99077c473d430b9e745714 [^]
2014-12-11 10:16psanjuanfix_in_branchpi =>
2014-12-11 14:22psanjuanNote Added: 0072424
2014-12-11 14:29psanjuanSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=7248#r7248
2014-12-11 14:29psanjuanProposed Solution updated
2014-12-11 14:29psanjuanRelationship addedrelated to 0028389
2014-12-11 14:31psanjuanNote Added: 0072425
2014-12-11 18:21SandrahuguetReview Assigned To => Sandrahuguet
2014-12-11 18:21SandrahuguetAssigned ToSandrahuguet => umartirena
2014-12-15 12:09SandrahuguetNote Added: 0072508
2014-12-15 12:09SandrahuguetStatusresolved => closed
2014-12-15 12:09SandrahuguetFixed in Version => 3.0PR15Q1
2014-12-30 23:24hudsonbotCheckin
2014-12-30 23:24hudsonbotNote Added: 0073055
2014-12-30 23:27hudsonbotCheckin
2014-12-30 23:27hudsonbotNote Added: 0073204

Notes
(0071638)
hgbot   
2014-11-13 14:43   
Repository: erp/devel/pi
Changeset: 50eb50e316272204ab73249de855eb14f150df9f
Author: Gorka Ion Damián <gorkaion.damian <at> openbravo.com>
Date: Thu Nov 13 10:52:31 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/50eb50e316272204ab73249de855eb14f150df9f [^]

Related to issue 28155.Related to issue 28157.Use conversionRateDocs

Some documents might include custom conversion rates. The FinancialUtils
utility class is extended with a new method that receives a List of rates
defined at document level. The existing method now calls the new one setting
an empty list on the new parameter.

---
M src/org/openbravo/financial/FinancialUtils.java
---
(0072400)
hgbot   
2014-12-10 14:11   
Repository: erp/devel/pi
Changeset: 28ff19fe345e102bad99077c473d430b9e745714
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Wed Dec 10 14:09:54 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/28ff19fe345e102bad99077c473d430b9e745714 [^]

Fixes Issue 28157: Landed Cost takes into account exchange rate at invoice level

If an exchange rate different from the System is added in Purchase Invoice and this Invoice is matched in a Landed Cost, a new record will be created in 'Matched Amount' tab under 'Landed Cost Cost' tab of 'Landed Cost' window. This new record will have the new flag of 'Is Conversion Matching' as true, and the amount will be the difference obtained from using the Invoice exchange rate compared to the one of the System.
This will create a new Cost Adjustment of source 'Landed Cost' with the same amount.

---
M src-db/database/model/tables/M_LC_MATCHED.xml
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/AD_ELEMENT.xml
M src-db/database/sourcedata/AD_FIELD.xml
M src/org/openbravo/common/inserters/LCMatchFromInvoiceInserter.java
M src/org/openbravo/costing/LCCostMatchFromInvoiceHandler.java
M src/org/openbravo/costing/LandedCostProcess.java
---
(0072424)
psanjuan   
2014-12-11 14:22   
Verified.
(0072425)
psanjuan   
2014-12-11 14:31   
Removed information related to the posting as a new issue has been created.
See issue related.
(0072508)
Sandrahuguet   
2014-12-15 12:09   
Code review done
(0073055)
hudsonbot   
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
(0073204)
hudsonbot   
2014-12-30 23:27   
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