Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0028155Openbravo ERP04. Warehouse managementpublic2014-11-12 12:432014-12-30 23:24
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
Summary0028155: [Costing] product cost transaction do not calculated properly in case of entering an Exchange Rate at purchase invoice level.
Description[Costing] product cost transaction do not calculated properly in case of entering an Exchange Rate at purchase invoice level.
Steps To ReproduceIn QA testing client create a new product named: LC 444
Select a purchase price list in USD. Enter a price of 444 (USD).

Make sure you do have a conversion rate USD to EUR, for instance 0.4

Go to purchase order and create a new one for Spain org, dated on 12/11/2014.
Select a USD Vendor, enter the product and enter qty = 1.
Book the order

Go to Goods Receipt and create a new one for Spain Org and USA vendor, dated on 15/11/2014. Click on "Create Lines From" and select the order just created.
Enter a storage bin. Complete the receipt.

Go to process request window and launch costing background process.

Go back to Goods Receipt and post it. Realize that the value posted in EUR general ledger is 177.60 (444 USD * 0.4 USD/EUR).

Go to purchase invoice and create a new one for Spain org and Vendor USA, dated on 30/11/2014. Click on "Create Lines From" and select the receipt.
Go to Exchange rate tab and create a new line. Enter an exchange rate of 0.8
Realize that Foreign Amount is 355.20
Complete and Post the invoice.

Realize that value posting to product asset account is 355.20

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

Go to product window, search LC 444 product and realize that the cost is still 177.60 (Qty 1), same as transaction cost Trx Original Cost, Total Cost and Unit Cost.

This is not correct.
Proposed SolutionThe correct product cost should be: 355.20 same as Transaction Cost.
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2014-11-12 12:43psanjuanNew Issue
2014-11-12 12:43psanjuanAssigned To => dmiguelez
2014-11-12 12:43psanjuanOBNetwork customer => No
2014-11-12 12:43psanjuanModules => Core
2014-11-12 12:43psanjuanTriggers an Emergency Pack => No
2014-11-12 12:53psanjuanAssigned Todmiguelez => umartirena
2014-11-12 12:53psanjuanSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=7035#r7035
2014-11-12 12:53psanjuanProposed Solution updated
2014-11-13 14:43hgbotCheckin
2014-11-13 14:43hgbotNote Added: 0071637
2014-11-13 14:43hgbotCheckin
2014-11-13 14:43hgbotNote Added: 0071639
2014-11-13 14:43hgbotStatusnew => resolved
2014-11-13 14:43hgbotResolutionopen => fixed
2014-11-13 14:43hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/b43300702aff961a009af62eabfbab7bdb122cc8 [^]
2014-11-20 09:21psanjuanNote Added: 0071841
2014-11-20 09:21psanjuanStatusresolved => new
2014-12-04 13:37psanjuanSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=7213#r7213
2014-12-05 14:37psanjuanNote Added: 0072314
2014-12-05 14:41umartirenaStatusnew => scheduled
2014-12-05 14:41umartirenafix_in_branch => pi
2014-12-05 14:41umartirenaStatusscheduled => resolved
2014-12-05 14:41umartirenaFixed in Version => pi
2014-12-11 18:21SandrahuguetReview Assigned To => Sandrahuguet
2014-12-11 18:21Sandrahuguetfix_in_branchpi =>
2014-12-12 14:57SandrahuguetNote Added: 0072456
2014-12-12 14:57SandrahuguetStatusresolved => closed
2014-12-12 14:57SandrahuguetFixed in Versionpi => 3.0PR15Q1
2014-12-30 23:24hudsonbotCheckin
2014-12-30 23:24hudsonbotNote Added: 0073054
2014-12-30 23:24hudsonbotCheckin
2014-12-30 23:24hudsonbotNote Added: 0073056

Notes
(0071637)
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
---
(0071639)
hgbot   
2014-11-13 14:43   
Repository: erp/devel/pi
Changeset: b43300702aff961a009af62eabfbab7bdb122cc8
Author: Gorka Ion Damián <gorkaion.damian <at> openbravo.com>
Date: Thu Nov 13 11:01:31 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/b43300702aff961a009af62eabfbab7bdb122cc8 [^]

Fixed issue 28155. Use custom conversion rate of documents.

Use new getConvertedAmount with the list of conversion rated defined for the
invoice.
Create a Invoice object to ease method readability.

---
M src/org/openbravo/costing/PriceDifferenceProcess.java
---
(0071841)
psanjuan   
2014-11-20 09:21   
This is not yet working. Issue re-opened.
(0072314)
psanjuan   
2014-12-05 14:37   
Verified.
(0072456)
Sandrahuguet   
2014-12-12 14:57   
Code review done
(0073054)
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
(0073056)
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