Openbravo Issue Tracking System - Openbravo ERP | |||||
| View Issue Details | |||||
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0024022 | Openbravo ERP | 04. Warehouse management | public | 2013-06-07 11:32 | 2013-07-23 18:25 |
| Reporter | VictorVillar | ||||
| Assigned To | jecharri | ||||
| Priority | urgent | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Platform | OS | 5 | OS Version | ||
| Product Version | pi | ||||
| Target Version | 3.0MP26 | Fixed in Version | |||
| Merge Request Status | |||||
| Review Assigned To | VictorVillar | ||||
| OBNetwork customer | |||||
| Web browser | |||||
| Modules | Core | ||||
| Support ticket | |||||
| Regression level | |||||
| Regression date | |||||
| Regression introduced in release | |||||
| Regression introduced by commit | |||||
| Triggers an Emergency Pack | No | ||||
| Summary | 0024022: Wrong Transaction cost calculated for legacy transaction cost under some circunstancies | ||||
| Description | Wrong Transaction cost calculated for legacy transaction cost under some circunstancies. Old cost engine introduced the Starting date and Ending date in the m_costing table, as DATE, without hours, minutes and seconds. But the CostingMigrationProcess.java -- method updateLegacyCosts() --statement legacyCosts = getLegacyCostScroll(clientId, naturalTree); is searching data, between the data range: where.append(" and trx." + MaterialTransaction.PROPERTY_MOVEMENTDATE + " >= c." + Costing.PROPERTY_STARTINGDATE); where.append(" and trx." + MaterialTransaction.PROPERTY_PRODUCT + " = c." + Costing.PROPERTY_PRODUCT); where.append(" and trx." + MaterialTransaction.PROPERTY_MOVEMENTDATE + " < c." + Costing.PROPERTY_ENDINGDATE); but the problem is the movementdate parameter because is with hours, minutes and seconds. As a result of that if you have registers in the m_costing, introduced with the same date, the transaction cost will be wrong calculated, because this query does not find the correct m_costing register for the transaction movement date. | ||||
| Steps To Reproduce | Wrong Transaction cost calculated for legacy transaction cost under some circunstancies. Old cost engine introduced the Starting date and Ending date in the m_costing table, as DATE, without hours, minutes and seconds. But the CostingMigrationProcess.java -- method updateLegacyCosts() --statement legacyCosts = getLegacyCostScroll(clientId, naturalTree); is searching data, between the data range: where.append(" and trx." + MaterialTransaction.PROPERTY_MOVEMENTDATE + " >= c." + Costing.PROPERTY_STARTINGDATE); where.append(" and trx." + MaterialTransaction.PROPERTY_PRODUCT + " = c." + Costing.PROPERTY_PRODUCT); where.append(" and trx." + MaterialTransaction.PROPERTY_MOVEMENTDATE + " < c." + Costing.PROPERTY_ENDINGDATE); but the problem is the movementdate parameter because is with hours, minutes and seconds. As a result of that if you have registers in the m_costing, introduced with the same date, the transaction cost will be wrong calculated, because this query does not find the correct m_costing register for the transaction movement date. | ||||
| Proposed Solution | The solution could be to truncate the movement date | ||||
| Additional Information | |||||
| Tags | No tags attached. | ||||
| Relationships | |||||
| Attached Files | |||||
| Issue History | |||||
| Date Modified | Username | Field | Change | ||
| 2013-06-07 11:32 | VictorVillar | New Issue | |||
| 2013-06-07 11:32 | VictorVillar | Assigned To | => dmiguelez | ||
| 2013-06-07 11:32 | VictorVillar | Modules | => Core | ||
| 2013-06-07 11:32 | VictorVillar | Resolution time | => 1372629600 | ||
| 2013-06-07 11:32 | VictorVillar | Triggers an Emergency Pack | => No | ||
| 2013-06-07 11:33 | VictorVillar | Issue Monitored: networkb | |||
| 2013-06-07 11:34 | VictorVillar | Description Updated | bug_revision_view_page.php?rev_id=4781#r4781 | ||
| 2013-06-07 11:34 | VictorVillar | Steps to Reproduce Updated | bug_revision_view_page.php?rev_id=4783#r4783 | ||
| 2013-06-07 14:45 | dmiguelez | Assigned To | dmiguelez => jonalegriaesarte | ||
| 2013-06-07 16:10 | VictorVillar | Proposed Solution updated | |||
| 2013-06-21 12:29 | jonalegriaesarte | Assigned To | jonalegriaesarte => jecharri | ||
| 2013-06-28 14:12 | jonalegriaesarte | Target Version | 3.0MP25 => 3.0MP26 | ||
| 2013-07-02 01:32 | hgbot | Checkin | |||
| 2013-07-02 01:32 | hgbot | Note Added: 0059799 | |||
| 2013-07-02 01:32 | hgbot | Status | new => resolved | ||
| 2013-07-02 01:32 | hgbot | Resolution | open => fixed | ||
| 2013-07-02 01:32 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/5444242cd48cb455535e4f7b9f63fc1b72f951b1 [^] | ||
| 2013-07-16 14:26 | hudsonbot | Checkin | |||
| 2013-07-16 14:26 | hudsonbot | Note Added: 0060007 | |||
| 2013-07-23 18:25 | VictorVillar | Review Assigned To | => VictorVillar | ||
| 2013-07-23 18:25 | VictorVillar | Note Added: 0060200 | |||
| 2013-07-23 18:25 | VictorVillar | Status | resolved => closed | ||
| Notes | |||||
|
|
|||||
|
|
||||
|
|
|||||
|
|
||||
|
|
|||||
|
|
||||