Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0028157 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] 04. Warehouse management | major | always | 2014-11-12 14:25 | 2014-12-30 23:27 | |||
Reporter | psanjuan | View Status | public | |||||
Assigned To | umartirena | |||||||
Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR15Q1 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 28ff19fe345e | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Linux 32 bit | Database | Oracle | Java version | 1.6 | |||
OS Version | Ubuntu 8.04.1 | Database version | 11.1.0.6.0 | Ant version | 1.7.0 | |||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | Sandrahuguet | |||||||
OBNetwork customer | No | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
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 Reproduce | In 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 Solution | A 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 | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
||||||||
|
![]() |
|
(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 (viewer) 2014-12-11 14:22 |
Verified. |
(0072425) psanjuan (viewer) 2014-12-11 14:31 |
Removed information related to the posting as a new issue has been created. See issue related. |
(0072508) Sandrahuguet (viewer) 2014-12-15 12:09 |
Code review done |
(0073055) hudsonbot (viewer) 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 (viewer) 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 |
![]() |
|||
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 | OBNetwork customer | => No |
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 |