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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0028502
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 04. Warehouse managementmajoralways2014-12-30 10:522015-02-04 20:33
ReporterpsanjuanView Statuspublic 
Assigned Toumartirena 
PriorityurgentResolutionfixedFixed in Version3.0PR15Q2
StatusclosedFix in branchFixed in SCM revisione087d572eed7
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

0028502: [Costing] Inventory Amount Update process is always taking a warehouse regardless warehouse is or not a costing dimension.

Description[Costing] Inventory Amount Update process is always taking a warehouse regardless warehouse is or not a costing dimension.
Steps To ReproduceAs QA Testing Admin (Set Spain as Organization and Spain warehouse as Warehouse as default session values):
* Create a Costing Rule for Spain org with Warehouse Dimension not selected with Starting Date 30/12/2014. Validate it.
* Create a warehouse named "Spain East warehouse" for Spain org.
* Create a new Product for Spain org and put a price for Purchase of 10.
* Create a Purchase Order dated on 31/12/2014 of 10 unit of the new product in Spain Warehouse. Receive that Purchase Order.
* Create another Purchase Order dated on 31/12/2014 of 20 unit of the new product in Spain East Warehouse. Receive that Purchase Order.
* Run Costing Background Process.
* Create an Inventory Amount update with Document Date 31/12/2014.
* Create a Line with Reference date 31/12/2014 and put as Unit Cost 15. Process the Inventory Amount Update.
* Check in Inventories that only an Inventory for Spain warehouse has been created. This is wrong. It should have been created two. One for Spain warehouse and another one for Spain East Warehouse.
Proposed SolutionIf the costing rule does NOT have Warehouse dimension selected, Inventory Amount Update should not shown warehouse value as it is useless.

If the costing rules does have Warehouse dimension selected, Inventory Amount Update should show as many lines as warehouse where product selected has stock.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0073295)
hgbot (developer)
2015-01-02 15:46

Repository: erp/devel/pi
Changeset: e087d572eed72f40c1fa9f995fa29a7b511de754
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Wed Dec 31 11:03:15 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/e087d572eed72f40c1fa9f995fa29a7b511de754 [^]

Fixes bug 28502: Inv Amt Update takes into account Costing Rule warehouse dim

The problem was that the process was checking if the warehouse in Inventory Amount Update is null or not, and if not, it uses it. This is correct when the Costing Rule is set as Warehouse Dimension, but if is not, it should not use it, and even if the Costing Rule is not set as Warehouse Dimension and the warehouse field is hidden in Inventory Amount Lines, it is filled with the Warehouse set in session value.

---
M src/org/openbravo/costing/InventoryAmountUpdateProcess.java
---
(0073358)
psanjuan (manager)
2015-01-08 12:17

Verified it is working now in case:
(1) Warehouse is do not set as costing dimension
(2) Warehouse is set as costing dimension.
(0073365)
Sandrahuguet (developer)
2015-01-08 13:45

Code review done
(0074097)
hudsonbot (developer)
2015-02-04 20:33

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/f36c91d0ad63 [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2014-12-30 10:52 psanjuan New Issue
2014-12-30 10:52 psanjuan Assigned To => dmiguelez
2014-12-30 10:52 psanjuan Modules => Core
2014-12-30 10:52 psanjuan Triggers an Emergency Pack => No
2014-12-31 10:43 umartirena Steps to Reproduce Updated View Revisions
2014-12-31 10:44 umartirena Steps to Reproduce Updated View Revisions
2014-12-31 10:44 umartirena Steps to Reproduce Updated View Revisions
2014-12-31 11:04 umartirena Assigned To dmiguelez => umartirena
2015-01-02 15:46 hgbot Checkin
2015-01-02 15:46 hgbot Note Added: 0073295
2015-01-02 15:46 hgbot Status new => resolved
2015-01-02 15:46 hgbot Resolution open => fixed
2015-01-02 15:46 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/e087d572eed72f40c1fa9f995fa29a7b511de754 [^]
2015-01-08 12:17 psanjuan Note Added: 0073358
2015-01-08 13:45 Sandrahuguet Review Assigned To => Sandrahuguet
2015-01-08 13:45 Sandrahuguet Note Added: 0073365
2015-01-08 13:45 Sandrahuguet Status resolved => closed
2015-01-08 13:45 Sandrahuguet Fixed in Version => 3.0PR15Q2
2015-02-04 20:33 hudsonbot Checkin
2015-02-04 20:33 hudsonbot Note Added: 0074097


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker