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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0023911
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 04. Warehouse managementmajoralways2013-05-24 09:552013-06-20 02:07
ReporterVictorVillarView Statuspublic 
Assigned Tojonalegriaesarte 
PriorityurgentResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisionc6159ccd2e9c
ProjectionnoneETAnoneTarget Version3.0MP25
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionpiSCM revision 
Review Assigned Tojecharri
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0023911: It is not possible to migrate to the new costing server if you are using second UOM, under some circunstancies

DescriptionIt is not possible to migrate to the new costing server if you are using second UOM. When you have not got stock for the first unit of measure, but you have stock for the second unit of measure, you can not migrate.
You get '/ by zero' error when the process calculates the cost:

Please check, CostingMigrationProcess.java, calculateCosts method, line: 406

  BigDecimal cost = trxCost.divide(trx.getMovementQuantity().abs(), costPrecision, BigDecimal.ROUND_HALF_UP);

the problem is that the MovementQuantity is zero
Steps To ReproduceIt is not possible to migrate to the new costing server if you are using second UOM. When you have not got stock for the first unit of measure, but you have stock for the second unit of measure, you can not migrate.
You get '/ by zero' error when the process calculates the cost:

Please check, CostingMigrationProcess.java, calculateCosts method, line: 406

  BigDecimal cost = trxCost.divide(trx.getMovementQuantity().abs(), costPrecision, BigDecimal.ROUND_HALF_UP);

the problem is that the MovementQuantity is zero
Proposed SolutionThe transaction cost for these transactions should be 0.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0059249)
hgbot (developer)
2013-06-06 17:35

Repository: erp/devel/pi
Changeset: c6159ccd2e9c9fe26a6c0195a14c549f6edaba06
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Thu Jun 06 17:34:05 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/c6159ccd2e9c9fe26a6c0195a14c549f6edaba06 [^]

Fixes Issue 23911. Fixes division by zero.

---
M src/org/openbravo/costing/CostingMigrationProcess.java
---
(0059547)
hudsonbot (developer)
2013-06-20 02:07

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/5caf871044ad [^]

Maturity status: Test

- Issue History
Date Modified Username Field Change
2013-05-24 09:55 VictorVillar New Issue
2013-05-24 09:55 VictorVillar Assigned To => dmiguelez
2013-05-24 09:55 VictorVillar Modules => Core
2013-05-24 09:55 VictorVillar Triggers an Emergency Pack => No
2013-05-24 09:55 VictorVillar Issue Monitored: networkb
2013-05-24 14:07 dmiguelez Assigned To dmiguelez => jonalegriaesarte
2013-06-03 16:38 jonalegriaesarte Target Version 3.0MP24 => 3.0MP25
2013-06-06 17:35 hgbot Checkin
2013-06-06 17:35 hgbot Note Added: 0059249
2013-06-06 17:35 hgbot Status new => resolved
2013-06-06 17:35 hgbot Resolution open => fixed
2013-06-06 17:35 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/c6159ccd2e9c9fe26a6c0195a14c549f6edaba06 [^]
2013-06-17 16:58 jecharri Review Assigned To => jecharri
2013-06-17 16:58 jecharri Status resolved => closed
2013-06-20 02:07 hudsonbot Checkin
2013-06-20 02:07 hudsonbot Note Added: 0059547


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker