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 | ||||||||
Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
Notes |
|
|
(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 |
Issue History |
|||
| 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 |