Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0033169 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] 04. Warehouse management | major | always | 2016-06-07 13:52 | 2016-06-17 19:38 | |||
Reporter | ngarcia | View Status | public | |||||
Assigned To | markmm82 | |||||||
Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR16Q3 | |||
Status | closed | Fix in branch | Fixed in SCM revision | d7f652803074 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | aferraz | |||||||
OBNetwork customer | OBPS | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | 41935 | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0033169: Backdated Transaction adjustments are made for the same day when movementdate column of M_TRANSACTION is defined with hours | |||||||
Description | Backdated Transaction adjustments are made for the same day when movementdate column of M_TRANSACTION is defined with hours. Movementdate could be defined with hours in the following functions: - ReservedGoodMovementActionHandler.doExecute - OutboundPickingListProcess.createShipment ... | |||||||
Steps To Reproduce | Note: you should have the costing rule validated with Fix Backdated From before today's date As group admin role: Create a new product add a purchase price list to it Create a Goods Receipt with today's date and add the previously created product Create a Stock Reservation of that product Create Goods Movement from Stock Reservation window Check through database the m_transaction records associated with the movement have the movement without being truncated Create a new Goods Receipt for that product Launch the Costing Background Process and check a new Cost Adjustment has been created (Backdated Transaction) when it should not | |||||||
Proposed Solution | Truncate MaterialTransaction.PROPERTY_MOVEMENTDATE in CostAdjustmentUtils.isNeededBackdatedCostAdjustment method. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|
![]() |
|
(0087216) markmm82 (viewer) 2016-06-13 20:38 |
Test Plan: As group admin role: Create a new product Add a purchase price list to it Create a Goods Receipt with today's date Add a line with the previously created product Create a Stock Reservation of that product Manage Stock for it Create Goods Movement from Stock Reservation window Check through database the m_transaction records associated with the movement have the movement without being truncated Create a new Goods Receipt Add a line with the previously created product Launch the Costing Background Process Check that NO new Cost Adjustment has been created. It is correct! |
(0087368) hgbot (developer) 2016-06-16 19:45 |
Repository: erp/devel/pi Changeset: d7f6528030743c9207acf0b7c660bf43c9eebedc Author: Mark <markmm82 <at> gmail.com> Date: Mon Jun 13 14:16:47 2016 -0400 URL: http://code.openbravo.com/erp/devel/pi/rev/d7f6528030743c9207acf0b7c660bf43c9eebedc [^] Fixes issue 33169: Fix Backdated Transaction when movementdate has hours Was truncated the movementdate column of M_TRANSACTION, allowing to the CostAdjustmentUtils class calculates correctly if the given Material Transaction is a backdated transaction or not. --- M src/org/openbravo/costing/CostAdjustmentUtils.java --- |
(0087369) aferraz (viewer) 2016-06-16 19:45 |
Code review + Testing OK |
(0087634) hudsonbot (viewer) 2016-06-17 19:38 |
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/0dc7be081b1c [^] Maturity status: Test |
![]() |
|||
Date Modified | Username | Field | Change |
2016-06-07 13:52 | ngarcia | New Issue | |
2016-06-07 13:52 | ngarcia | Assigned To | => Triage Finance |
2016-06-07 13:52 | ngarcia | OBNetwork customer | => Yes |
2016-06-07 13:52 | ngarcia | Modules | => Core |
2016-06-07 13:52 | ngarcia | Resolution time | => 1467064800 |
2016-06-07 13:52 | ngarcia | Triggers an Emergency Pack | => No |
2016-06-07 14:02 | ngarcia | Issue Monitored: networkb | |
2016-06-07 14:02 | ngarcia | Support ticket | => 41935 |
2016-06-07 18:40 | aferraz | Assigned To | Triage Finance => markmm82 |
2016-06-13 18:28 | markmm82 | Status | new => scheduled |
2016-06-13 19:02 | aferraz | Proposed Solution updated | |
2016-06-13 20:38 | markmm82 | Note Added: 0087216 | |
2016-06-14 16:13 | tegik_forge | Issue Monitored: tegik_forge | |
2016-06-16 19:45 | hgbot | Checkin | |
2016-06-16 19:45 | hgbot | Note Added: 0087368 | |
2016-06-16 19:45 | hgbot | Status | scheduled => resolved |
2016-06-16 19:45 | hgbot | Resolution | open => fixed |
2016-06-16 19:45 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/d7f6528030743c9207acf0b7c660bf43c9eebedc [^] |
2016-06-16 19:45 | aferraz | Review Assigned To | => aferraz |
2016-06-16 19:45 | aferraz | Note Added: 0087369 | |
2016-06-16 19:45 | aferraz | Status | resolved => closed |
2016-06-16 19:45 | aferraz | Fixed in Version | => 3.0PR16Q3 |
2016-06-17 19:38 | hudsonbot | Checkin | |
2016-06-17 19:38 | hudsonbot | Note Added: 0087634 |
Copyright © 2000 - 2009 MantisBT Group |