Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0026789Openbravo ERP04. Warehouse managementpublic2014-06-04 19:032014-10-01 16:49
VictorVillar 
caristu 
immediatemajoralways
closedfixed 
5
pi 
3.0PR14Q3 
VictorVillar
Core
Production - Confirmed Stable
2013-09-13
3.0MP28
https://code.openbravo.com/erp/devel/pi/rev/668effe90d34780551c9f1de1285022fc505ec66 [^]
Yes
0026789: It is not possible to migrate to the new costing server under some circumstancies
It is not possible to migrate to the new costing server under some circumstancies, if you are using second unit of measure (and you have stock) and the main uom is 0.
It is not possible to migrate to the new costing server under some circumstancies, if you are using second unit of measure (and you have stock) and the main uom is 0. You will receive a Null Pointer Exception after launching the Costing Background Process.

The reason is that the product will be taken into account in the closing inventories but after processing it, the transaction will no be created because the following statement in the inventorycountprocess.java:

insert.append(" and e." + InventoryCountLine.PROPERTY_QUANTITYCOUNT + " != e."
        + InventoryCountLine.PROPERTY_BOOKQUANTITY);

so when the process tries to calculate the cost for the opening inventory, it will not find any transaction.

The reason is that the product will be taken into account in the closing inventories but after processing it, the transaction will no be created because the following statement in the inventorycountprocess.java:

insert.append(" and e." + InventoryCountLine.PROPERTY_QUANTITYCOUNT + " != e."
        + InventoryCountLine.PROPERTY_BOOKQUANTITY);

Also, it should be added the following statement in the processInventory() method as a restriction:

  insert.append(" and (e." + InventoryCountLine.PROPERTY_QUANTITYCOUNT + " != e."
        + InventoryCountLine.PROPERTY_BOOKQUANTITY);
    insert.append(" or e." + InventoryCountLine.PROPERTY_ORDERQUANTITY + " != e."
            + InventoryCountLine.PROPERTY_QUANTITYORDERBOOK+")");
No tags attached.
Issue History
2014-06-04 19:03VictorVillarNew Issue
2014-06-04 19:03VictorVillarAssigned To => caristu
2014-06-04 19:03VictorVillarModules => Core
2014-06-04 19:03VictorVillarTriggers an Emergency Pack => No
2014-06-04 19:04VictorVillarIssue Monitored: networkb
2014-06-04 19:07VictorVillarRegression level => Production - Confirmed Stable
2014-06-04 19:07VictorVillarRegression date => 2013-09-13
2014-06-04 19:07VictorVillarRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/668effe90d34780551c9f1de1285022fc505ec66 [^]
2014-06-04 19:07VictorVillarTriggers an Emergency PackNo => Yes
2014-06-04 19:08VictorVillarProposed Solution updated
2014-06-06 08:46VictorVillarProposed Solution updated
2014-06-09 09:24VictorVillarProposed Solution updated
2014-06-09 10:33hgbotCheckin
2014-06-09 10:33hgbotNote Added: 0067818
2014-06-09 10:33hgbotStatusnew => resolved
2014-06-09 10:33hgbotResolutionopen => fixed
2014-06-09 10:33hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/e7a946efecef3532f4188fa3bf70a4cab386a670 [^]
2014-06-09 10:38VictorVillarReview Assigned To => VictorVillar
2014-06-09 10:38VictorVillarNote Added: 0067819
2014-06-09 10:38VictorVillarStatusresolved => closed
2014-06-10 22:24hudsonbotCheckin
2014-06-10 22:24hudsonbotNote Added: 0067890
2014-10-01 16:49plujanRegression introduced in release => 3.0MP28

Notes
(0067818)
hgbot   
2014-06-09 10:33   
Repository: erp/devel/pi
Changeset: e7a946efecef3532f4188fa3bf70a4cab386a670
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Mon Jun 09 10:33:08 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/e7a946efecef3532f4188fa3bf70a4cab386a670 [^]

fixes issue 26789: It is not possible to migrate to the new costing
 server under some circumstancies

---
M src/org/openbravo/materialmgmt/InventoryCountProcess.java
---
(0067819)
VictorVillar   
2014-06-09 10:38   
Code Review: Gorka Ion + Verified: Victor J. Villar
(0067890)
hudsonbot   
2014-06-10 22: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/763f16c2ad2e [^]
Maturity status: Test