Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0035337Openbravo ERP04. Warehouse managementpublic2017-02-22 12:062017-03-15 20:19
ngarcia 
collazoandy4 
urgentmajoralways
closedfixed 
5
 
3.0PR17Q2 
markmm82
Core
No
0035337: [Negative Stock Correction]: Costing Background Process finishes with error if Warehouse dimension is selected in costing rule
[Negative Stock Correction]: Costing Background Process finishes with error if Warehouse dimension is selected in costing rule
As group admin role:
As group admin role:
   Set the Enable Negative Stock Corrections preference to Y
   Select the Allow negative stock option at Client level
   Validate a Costing Rule in F&B España S.A. with warehouse dimension selected
   Create a new product
   Create a purchase order
   Add 10 units of the previously created product with a price of 5
   Create the goods receipt in España Region Norte warehouse from it
   Launch the Costing Background process and check it finishes with the following error:
      could not resolve property: storageBin.warehouse.id of: MaterialMgmtMaterialTransaction
   The log shows:
      ERROR org.openbravo.costing.CostingBackground - could not resolve property: storageBin.warehouse.id of: MaterialMgmtMaterialTransaction
org.hibernate.QueryException: could not resolve property: storageBin.warehouse.id of: MaterialMgmtMaterialTransaction
    at org.hibernate.persister.entity.AbstractPropertyMapping.propertyException(AbstractPropertyMapping.java:81)
        ...
    at org.openbravo.dal.service.OBCriteria.uniqueResult(OBCriteria.java:129)
    at org.openbravo.costing.CostingUtils.existsProcessedTransactions(CostingUtils.java:810)
    at org.openbravo.costing.CostingServer.checkCostAdjustments(CostingServer.java:334)
    at org.openbravo.costing.CostingServer.process(CostingServer.java:132)
    at org.openbravo.costing.CostingBackground.doExecute(CostingBackground.java:129)
        ...
Use an alias in the criteria for the MaterialTransaction.PROPERTY_STORAGEBIN in existsProcessedTransactions function of CostingUtils class
No tags attached.
depends on defect 0035330 closed collazoandy4 [Negative Stock Correction]: Warehouse is not taken into account to check if stock was negative previous to an stock increase 
Issue History
2017-02-22 12:06ngarciaNew Issue
2017-02-22 12:06ngarciaAssigned To => Triage Finance
2017-02-22 12:06ngarciaModules => Core
2017-02-22 12:06ngarciaResolution time => 1488236400
2017-02-22 12:06ngarciaTriggers an Emergency Pack => No
2017-02-22 12:07ngarciaRelationship addeddepends on 0035330
2017-02-22 12:07ngarciaIssue Monitored: networkb
2017-02-22 14:56markmm82Assigned ToTriage Finance => collazoandy4
2017-02-22 15:55egoitzIssue Monitored: egoitz
2017-02-22 18:32PracticsIssue Monitored: Practics
2017-02-23 18:22hgbotCheckin
2017-02-23 18:22hgbotNote Added: 0094582
2017-02-23 18:22hgbotStatusnew => resolved
2017-02-23 18:22hgbotResolutionopen => fixed
2017-02-23 18:22hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/ef9c1ff7a9c2dae621c3012cb0203e2a2c9fc06c [^]
2017-02-23 18:25markmm82Review Assigned To => markmm82
2017-02-23 18:25markmm82Note Added: 0094583
2017-02-23 18:25markmm82Statusresolved => closed
2017-02-23 18:25markmm82Fixed in Version => 3.0PR17Q2
2017-03-15 20:19hudsonbotCheckin
2017-03-15 20:19hudsonbotNote Added: 0095227

Notes
(0094582)
hgbot   
2017-02-23 18:22   
Repository: erp/devel/pi
Changeset: ef9c1ff7a9c2dae621c3012cb0203e2a2c9fc06c
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Thu Feb 23 14:19:19 2017 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/ef9c1ff7a9c2dae621c3012cb0203e2a2c9fc06c [^]

Fixes issue 35337: Fix CostingUtils.existsProcessedTransactions method

Add an alias for Locator class in CostingUtils.existsProcessedTransactions method.

---
M src/org/openbravo/costing/CostingUtils.java
---
(0094583)
markmm82   
2017-02-23 18:25   
Code review + Testing OK
(0095227)
hudsonbot   
2017-03-15 20:19   
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/54e102bef53e [^]
Maturity status: Test