Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0039616 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] 09. Financial management | major | always | 2018-11-13 13:33 | 2019-03-28 14:55 | |||
| Reporter | dmiguelez | View Status | public | |||||
| Assigned To | dmiguelez | |||||||
| Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR19Q1 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | e028e67c832c | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Merge Request Status | ||||||||
| Review Assigned To | Sandrahuguet | |||||||
| OBNetwork customer | Gold | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Support ticket | 6116 | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0039616: Wrong Cost Adjustment created from a Landed Cost with multiple Currencies | |||||||
| Description | Wrong Cost Adjustment created from a Landed Cost with multiple Currencies | |||||||
| Steps To Reproduce | Log in as F&B Admin Go to Landed Cost Type Window and create two new records with: * Organization: F&B US * Name: LCT1 * Landed Cost Distribution Algorithm: Distribution By Amount * Account: Company Capital * Organization: F&B US * Name: LCT2 * Landed Cost Distribution Algorithm: Distribution By Amount * Account: Company Capital Go to Landed Cost Window and create a new record with: * Organization: F&B US West Coast Move to Cost Tab and create 4 records with: * Landed Cost Type: LCT1 * Amount: 10 * Currency: USD * Landed Cost Type: LCT1 * Amount: 20 * Currency: EUR * Landed Cost Type: LCT2 * Amount: 30 * Currency: USD * Landed Cost Type: LCT2 * Amount: 40 * Currency: EUR Move to Receipts Tab and create a record with: * Goods Receipt: 10000525 - 10-09-2018 - Be Soft Drinker, Inc. * Goods Receipt Line: 10000525 - 10-09-2018 - Be Soft Drinker, Inc. - 20 - Cola - 82000.00 Process the Landed Cost Document Go to Cost Adjustment Window and check the Cost Adjustment created when the Landed Cost was processed Check that it has one line with: * Adjustment Amount: 100 * Currency: EUR And it should have had two lines with: * Adjustment Amount: 40 * Currency: USD * Adjustment Amount: 60 * Currency: EUR | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0107881) dmiguelez (viewer) 2018-11-13 13:34 |
Test Plan I: Log in as F&B Admin Go to Landed Cost Type Window and create two new records with: * Organization: F&B US * Name: LCT1 * Landed Cost Distribution Algorithm: Distribution By Amount * Account: Company Capital * Organization: F&B US * Name: LCT2 * Landed Cost Distribution Algorithm: Distribution By Amount * Account: Company Capital Go to Landed Cost Window and create a new record with: * Organization: F&B US West Coast Move to Cost Tab and create 4 records with: * Landed Cost Type: LCT1 * Amount: 10 * Currency: USD * Landed Cost Type: LCT1 * Amount: 20 * Currency: EUR * Landed Cost Type: LCT2 * Amount: 30 * Currency: USD * Landed Cost Type: LCT2 * Amount: 40 * Currency: EUR Move to Receipts Tab and create a record with: * Goods Receipt: 10000525 - 10-09-2018 - Be Soft Drinker, Inc. * Goods Receipt Line: 10000525 - 10-09-2018 - Be Soft Drinker, Inc. - 20 - Cola - 82000.00 Process the Landed Cost Document Go to Cost Adjustment Window and check the Cost Adjustment created when the Landed Cost was processed has two lines with: * Adjustment Amount: 40 * Currency: USD * Adjustment Amount: 60 * Currency: EUR Test Plan II: Using the scenario created to reproduce the issue without the fix. Reactivate the Landed Cost and process it again. Check that a Cost Adjustment has been created with two lines with: * Adjustment Amount: 40 * Currency: USD * Adjustment Amount: 60 * Currency: EUR |
|
(0107905) hgbot (developer) 2018-11-14 12:41 |
Repository: erp/devel/pi Changeset: e028e67c832cd5a9de510e175778967be67a50a1 Author: David Miguelez <david.miguelez <at> openbravo.com> Date: Tue Nov 13 13:40:29 2018 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/e028e67c832cd5a9de510e175778967be67a50a1 [^] Fixes Issue 39616. When searching for previously created Cost Adjustment Lines take into account different currencies. This is important for Landed Costs in which the same adjustment can happen for the same transaction with different currencies. * Create new API to be able to send the currency of the Landed Cost line as a parameter * Use the currency in the Query to filter the Cost Adjustment Lines * Send the Landed Cost Currency when creating a new Cost Adjustment Line --- M src/org/openbravo/costing/CostAdjustmentUtils.java M src/org/openbravo/costing/LandedCostProcess.java --- |
|
(0107906) Sandrahuguet (viewer) 2018-11-14 12:43 |
Code review + testing OK |
|
(0108468) hudsonbot (viewer) 2018-12-11 20:22 |
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/470e3cd384c5 [^] Maturity status: Test |
|
(0109012) hgbot (developer) 2019-01-18 10:25 |
Repository: erp/devel/pi Changeset: 8d73ffc2b531b2a5d34039a1cff737480961325e Author: Atul Gaware <atul.gaware <at> openbravo.com> Date: Thu Jan 17 14:57:34 2019 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/8d73ffc2b531b2a5d34039a1cff737480961325e [^] Related To Issue 39616:Added junit tests for the Landed Cost with multiple currency records in Cost Tab --- M src-test/src/org/openbravo/test/costing/TestCosting.java A src-test/src/org/openbravo/test/costing/TestIssue39616.java --- |
|
(0110737) hudsonbot (viewer) 2019-03-28 14:55 |
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/b2fbc1588df4 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2018-11-13 13:33 | dmiguelez | New Issue | |
| 2018-11-13 13:33 | dmiguelez | Assigned To | => Triage Finance |
| 2018-11-13 13:33 | dmiguelez | OBNetwork customer | => Gold |
| 2018-11-13 13:33 | dmiguelez | Modules | => Core |
| 2018-11-13 13:33 | dmiguelez | Support ticket | => 6116 |
| 2018-11-13 13:33 | dmiguelez | Resolution time | => 1543878000 |
| 2018-11-13 13:33 | dmiguelez | Triggers an Emergency Pack | => No |
| 2018-11-13 13:34 | dmiguelez | Note Added: 0107881 | |
| 2018-11-13 16:17 | martinsdan | Issue Monitored: martinsdan | |
| 2018-11-13 17:12 | Sandrahuguet | Assigned To | Triage Finance => dmiguelez |
| 2018-11-14 09:54 | Sandrahuguet | Status | new => scheduled |
| 2018-11-14 12:41 | hgbot | Checkin | |
| 2018-11-14 12:41 | hgbot | Note Added: 0107905 | |
| 2018-11-14 12:41 | hgbot | Status | scheduled => resolved |
| 2018-11-14 12:41 | hgbot | Resolution | open => fixed |
| 2018-11-14 12:41 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/e028e67c832cd5a9de510e175778967be67a50a1 [^] |
| 2018-11-14 12:43 | Sandrahuguet | Review Assigned To | => Sandrahuguet |
| 2018-11-14 12:43 | Sandrahuguet | Note Added: 0107906 | |
| 2018-11-14 12:43 | Sandrahuguet | Status | resolved => closed |
| 2018-11-14 12:43 | Sandrahuguet | Fixed in Version | => 3.0PR19Q1 |
| 2018-12-11 20:22 | hudsonbot | Checkin | |
| 2018-12-11 20:22 | hudsonbot | Note Added: 0108468 | |
| 2019-01-18 10:25 | hgbot | Checkin | |
| 2019-01-18 10:25 | hgbot | Note Added: 0109012 | |
| 2019-03-28 14:55 | hudsonbot | Checkin | |
| 2019-03-28 14:55 | hudsonbot | Note Added: 0110737 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |