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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0026789
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 04. Warehouse managementmajoralways2014-06-04 19:032014-10-01 16:49
ReporterVictorVillarView Statuspublic 
Assigned Tocaristu 
PriorityimmediateResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisione7a946efecef
ProjectionnoneETAnoneTarget Version3.0PR14Q3
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionpiSCM revision 
Review Assigned ToVictorVillar
Web browser
ModulesCore
Regression levelProduction - Confirmed Stable
Regression date2013-09-13
Regression introduced in release3.0MP28
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/668effe90d34780551c9f1de1285022fc505ec66 [^]
Triggers an Emergency PackYes
Summary

0026789: It is not possible to migrate to the new costing server under some circumstancies

DescriptionIt 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.
Steps To ReproduceIt 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.

Proposed SolutionThe 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+")");
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0067818)
hgbot (developer)
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 (developer)
2014-06-09 10:38

Code Review: Gorka Ion + Verified: Victor J. Villar
(0067890)
hudsonbot (developer)
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

- Issue History
Date Modified Username Field Change
2014-06-04 19:03 VictorVillar New Issue
2014-06-04 19:03 VictorVillar Assigned To => caristu
2014-06-04 19:03 VictorVillar Modules => Core
2014-06-04 19:03 VictorVillar Triggers an Emergency Pack => No
2014-06-04 19:04 VictorVillar Issue Monitored: networkb
2014-06-04 19:07 VictorVillar Regression level => Production - Confirmed Stable
2014-06-04 19:07 VictorVillar Regression date => 2013-09-13
2014-06-04 19:07 VictorVillar Regression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/668effe90d34780551c9f1de1285022fc505ec66 [^]
2014-06-04 19:07 VictorVillar Triggers an Emergency Pack No => Yes
2014-06-04 19:08 VictorVillar Proposed Solution updated
2014-06-06 08:46 VictorVillar Proposed Solution updated
2014-06-09 09:24 VictorVillar Proposed Solution updated
2014-06-09 10:33 hgbot Checkin
2014-06-09 10:33 hgbot Note Added: 0067818
2014-06-09 10:33 hgbot Status new => resolved
2014-06-09 10:33 hgbot Resolution open => fixed
2014-06-09 10:33 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/e7a946efecef3532f4188fa3bf70a4cab386a670 [^]
2014-06-09 10:38 VictorVillar Review Assigned To => VictorVillar
2014-06-09 10:38 VictorVillar Note Added: 0067819
2014-06-09 10:38 VictorVillar Status resolved => closed
2014-06-10 22:24 hudsonbot Checkin
2014-06-10 22:24 hudsonbot Note Added: 0067890
2014-10-01 16:49 plujan Regression introduced in release => 3.0MP28


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker