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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0028157
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 04. Warehouse managementmajoralways2014-11-12 14:252014-12-30 23:27
ReporterpsanjuanView Statuspublic 
Assigned Toumartirena 
PriorityurgentResolutionfixedFixed in Version3.0PR15Q1
StatusclosedFix in branchFixed in SCM revision28ff19fe345e
ProjectionnoneETAnoneTarget Version
OSLinux 32 bitDatabaseOracleJava version1.6
OS VersionUbuntu 8.04.1Database version11.1.0.6.0Ant version1.7.0
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

0028157: [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
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0028389 closedumartirena [Costing] Landed Cost Cost posting is not correct in case there is a conversion matching caused by an exchange rate 

-  Notes
(0071638)
hgbot (developer)
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 (developer)
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 (manager)
2014-12-11 14:22

Verified.
(0072425)
psanjuan (manager)
2014-12-11 14:31

Removed information related to the posting as a new issue has been created.
See issue related.
(0072508)
Sandrahuguet (developer)
2014-12-15 12:09

Code review done
(0073055)
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
(0073204)
hudsonbot (developer)
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

- Issue History
Date Modified Username Field Change
2014-11-12 14:25 psanjuan New Issue
2014-11-12 14:25 psanjuan Assigned To => dmiguelez
2014-11-12 14:25 psanjuan Modules => Core
2014-11-12 14:25 psanjuan Triggers an Emergency Pack => No
2014-11-12 14:41 psanjuan Steps to Reproduce Updated View Revisions
2014-11-12 14:41 psanjuan Proposed Solution updated
2014-11-13 14:43 hgbot Checkin
2014-11-13 14:43 hgbot Note Added: 0071638
2014-11-18 15:55 Sandrahuguet Status new => scheduled
2014-11-18 15:55 Sandrahuguet Assigned To dmiguelez => Sandrahuguet
2014-11-18 15:55 Sandrahuguet fix_in_branch => pi
2014-12-10 14:11 hgbot Checkin
2014-12-10 14:11 hgbot Note Added: 0072400
2014-12-10 14:11 hgbot Status scheduled => resolved
2014-12-10 14:11 hgbot Resolution open => fixed
2014-12-10 14:11 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/28ff19fe345e102bad99077c473d430b9e745714 [^]
2014-12-11 10:16 psanjuan fix_in_branch pi =>
2014-12-11 14:22 psanjuan Note Added: 0072424
2014-12-11 14:29 psanjuan Steps to Reproduce Updated View Revisions
2014-12-11 14:29 psanjuan Proposed Solution updated
2014-12-11 14:29 psanjuan Relationship added related to 0028389
2014-12-11 14:31 psanjuan Note Added: 0072425
2014-12-11 18:21 Sandrahuguet Review Assigned To => Sandrahuguet
2014-12-11 18:21 Sandrahuguet Assigned To Sandrahuguet => umartirena
2014-12-15 12:09 Sandrahuguet Note Added: 0072508
2014-12-15 12:09 Sandrahuguet Status resolved => closed
2014-12-15 12:09 Sandrahuguet Fixed in Version => 3.0PR15Q1
2014-12-30 23:24 hudsonbot Checkin
2014-12-30 23:24 hudsonbot Note Added: 0073055
2014-12-30 23:27 hudsonbot Checkin
2014-12-30 23:27 hudsonbot Note Added: 0073204


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker