Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0028115Openbravo ERP04. Warehouse managementpublic2014-11-06 18:112014-12-30 23:26
Reporterpsanjuan 
Assigned Toumartirena 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS20OS VersionUbuntu 8.04.1
Product Version 
Target VersionFixed in Versionpi 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0028115: [Costing] Add "Is Cost Permanent" transactions as input transactions in Costing tab, Inventory Transactions field
Description[Costing] Add "Is Cost Permanent" transactions as input transactions in Costing tab, Inventory Transactions field
Steps To ReproduceIn QA Testing client create a new product named "Costing GM1 no incremental". Select a purchase price list. Enter a unit/list price of 11.00

Go to purchase order window and create a new one dated on 29/10/2014. Enter 820 units. Book the order.

Go to goods receipt window and create a new one dated on 3/11/2014. Select the order and receipt 820 units. Complete the receipt

Go to process request window and launch costing background process.

Go back to goods receipt window and post it. Realize that value posted is 9020 ( 820 units * 11.00 €/unit)

Go to Goods Movement Window and create a new one dated on 13/11/2014

Move 400 units from Spain Warehouse to another warehouse named Spain West Warehouse, for instance. Complete the movement.

Go to process request window and launch costing background process.

Go to goods movement and post it. Realize that the value posted is 4400 ( 400 units + 11 €/unit)

Go to product window and search by the product. Go to transaction tab.
Select Movement From transaction and press the button "Manual Cost Adjustment".

In total cost field enter a total cost amount = 4000. Enter 18/11/2014 as accounting date and do not select the flag "Incremental". Press Done/OK.

Go to Cost Adjustment window and realize that a manual cost adjustment has been created, adjust amount = -400 (4400-4000).

Go to product window and search by the product. Go to transaction tab.
Select Movement From transaction and realize that it is marked as "Is Cost Permanent = Yes". Transaction original cost = 4400, Total Cost and Unit Cost = 4000

Go to costing tab and realize that there are 3 lines:

Line 1 - Cost 11 related to Vendor receipt (V+) dated on 3/11/2014 - total mov qty 820. Spain Warehouse.

Line 2 - Cost 10 (4000/400 units) related to Goods Movement To (M+) dated on 13/11/2014 - total mov qty 400 . Spain West Warehouse

Line 3 - Cost 11.95 [(420 units * 11.00) + 400 / 420 units] do not related to any inventory transaction - total mov qty 420. Spain Warehouse.

Line 3 is not correct. It should be:

Line 3 - Cost 11.95 [(420 units * 11.00) + 400 / 420 units] related to Goods Movement From (M-) dated on 13/11/2014 - total mov qty 420. Spain Warehouse.

After this a purchase invoice needs to be created dated on 8/11/2014. Create it by using Create Lines From, select the receipt. Realize that 820 units are shown. Change the price from 11.00 to 15.00. Complete and post the invoice. Realize that total value posted is 12300 (820 units * 15.00 €/unit).

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

Go to cost adjustment window and realize that a price correction adjustment has been created related to vendor receipt dated on 3/11/2014 Adj Amount 3280 ( 12300-9020).

Go to product window, search the product and navigate to costing tab. Realize that there are 4 lines:

Line 1 - Cost 15 related to Vendor receipt (V+) dated on 3/11/2014 - total mov qty 820. Spain Warehouse.

Line 2 - Cost 10 (4000/400 units) related to Goods Movement To (M+) dated on 13/11/2014 - total mov qty 400 . Spain West Warehouse

Line 3 - Cost 11.95 [(420 units * 11.00) + 400 / 420 units] do not related to any inventory transaction - total mov qty 420. Spain Warehouse.

Line 4 - Cost 19.76 [(420 units * 11.00) + 400 + 3280 / 420 units] do not related to any inventory transaction - total mov qty 420. Spain Warehouse.

This is not correct, correct lines should be:

Line 1 - Cost 15 related to Vendor receipt (V+) dated on 3/11/2014 - total mov qty 820. Spain Warehouse.

Line 2 - Cost 10 (4000/400 units) related to Goods Movement To (M+) dated on 13/11/2014 - total mov qty 400 . Spain West Warehouse

Line 3 - Cost 19.76 related to Goods Movement From (M-) dated on 13/11/2014 - total mov qty 420. Spain Warehouse.
Proposed Solutionto be defined
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2014-11-06 18:11psanjuanNew Issue
2014-11-06 18:11psanjuanAssigned To => dmiguelez
2014-11-06 18:11psanjuanOBNetwork customer => No
2014-11-06 18:11psanjuanModules => Core
2014-11-06 18:11psanjuanTriggers an Emergency Pack => No
2014-11-07 12:00psanjuanAssigned Todmiguelez => umartirena
2014-11-07 12:00psanjuanSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=6990#r6990
2014-11-07 12:09psanjuanSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=6991#r6991
2014-11-07 12:22psanjuanSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=6994#r6994
2014-11-07 12:24psanjuanSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=6995#r6995
2014-11-12 09:46umartirenaStatusnew => scheduled
2014-11-12 09:46umartirenafix_in_branch => pi
2014-11-12 12:22hgbotCheckin
2014-11-12 12:22hgbotNote Added: 0071602
2014-11-12 12:22hgbotStatusscheduled => resolved
2014-11-12 12:22hgbotResolutionopen => fixed
2014-11-12 12:22hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/3b8d1e5bf5347507aa289bc205ed71bf7c072d9d [^]
2014-11-13 10:59psanjuanNote Added: 0071627
2014-11-13 10:59psanjuanStatusresolved => closed
2014-11-13 10:59psanjuanfix_in_branchpi =>
2014-11-13 10:59psanjuanSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=7047#r7047
2014-11-13 12:31psanjuanNote Added: 0071631
2014-11-13 12:31psanjuanStatusclosed => new
2014-11-13 12:53hgbotCheckin
2014-11-13 12:53hgbotNote Added: 0071633
2014-11-13 12:53hgbotCheckin
2014-11-13 12:53hgbotNote Added: 0071634
2014-11-13 12:54umartirenaStatusnew => scheduled
2014-11-13 12:54umartirenafix_in_branch => pi
2014-11-13 12:54umartirenaStatusscheduled => resolved
2014-11-13 12:54umartirenaFixed in Version => pi
2014-11-13 12:54umartirenaFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/3b8d1e5bf5347507aa289bc205ed71bf7c072d9d [^] => https://code.openbravo.com/erp/devel/pi/rev/f007a86ec722 [^]
2014-11-13 13:27psanjuanNote Added: 0071636
2014-11-13 13:27psanjuanStatusresolved => closed
2014-11-13 13:27psanjuanfix_in_branchpi =>
2014-12-04 12:21hgbotCheckin
2014-12-04 12:21hgbotNote Added: 0072279
2014-12-30 23:24hudsonbotCheckin
2014-12-30 23:24hudsonbotNote Added: 0073050
2014-12-30 23:24hudsonbotCheckin
2014-12-30 23:24hudsonbotNote Added: 0073052
2014-12-30 23:24hudsonbotCheckin
2014-12-30 23:24hudsonbotNote Added: 0073053
2014-12-30 23:26hudsonbotCheckin
2014-12-30 23:26hudsonbotNote Added: 0073171

Notes
(0071602)
hgbot   
2014-11-12 12:22   
Repository: erp/devel/pi
Changeset: 3b8d1e5bf5347507aa289bc205ed71bf7c072d9d
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Wed Nov 12 12:19:40 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/3b8d1e5bf5347507aa289bc205ed71bf7c072d9d [^]

Fixes Bug 28115: Is Cost Permanent transactions added as input in Costing tab

---
M src/org/openbravo/costing/AverageCostAdjustment.java
---
(0071627)
psanjuan   
2014-11-13 10:59   
Verified for Manual cost adjustments set as:
- "incremental" = No
- "incremental" = Yes and "Unit Cost" = NO
- and "Incremental" = Yes and "Unit Cost" = Yes.

See excel sheet below:
https://docs.google.com/a/openbravo.com/spreadsheets/d/1Ilm1oct0vH4iKFlI1I0VbmYM-r8fdo1moK02jd2Zigg/edit#gid=1046089920 [^]
(0071631)
psanjuan   
2014-11-13 12:31   
Issue re-opened because there is a minor thing to change.

Third cost line related to Goods Movement From (M-) is set to "Permanent" = No in the Costing tab.

Above is not correct as it should be set as "Yes".
(0071633)
hgbot   
2014-11-13 12:53   
Repository: erp/devel/pi
Changeset: 1d388346f06044397927b89aadf44e3d3ab21e0b
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Thu Nov 13 12:46:25 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/1d388346f06044397927b89aadf44e3d3ab21e0b [^]

Related to Issue 28115: Fix recal of trxPrice in M_Costing

---
M src/org/openbravo/costing/AverageCostAdjustment.java
---
(0071634)
hgbot   
2014-11-13 12:53   
Repository: erp/devel/pi
Changeset: f007a86ec722993eaf5fe4de358d76dc99de48a8
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Thu Nov 13 12:48:51 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/f007a86ec722993eaf5fe4de358d76dc99de48a8 [^]

Related to Issue 28115: Set isPermanent true when Costing is permanent

---
M src/org/openbravo/costing/AverageCostAdjustment.java
---
(0071636)
psanjuan   
2014-11-13 13:27   
Verified that last cost line (third one) is now set as "Permanent" = Yes
(0072279)
hgbot   
2014-12-04 12:21   
Repository: erp/devel/pi
Changeset: 7084255c151e87dba55b4f7a7c475b4db9f277a0
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Thu Dec 04 12:12:34 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/7084255c151e87dba55b4f7a7c475b4db9f277a0 [^]

Related to Issue 28113: Revert a change on issue 28115.

In this cases the net unit price is the cost of the transaction

---
M src/org/openbravo/costing/AverageCostAdjustment.java
---
(0073050)
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
(0073052)
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
(0073053)
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
(0073171)
hudsonbot   
2014-12-30 23:26   
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