Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0028640 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] 04. Warehouse management | major | always | 2015-01-16 10:35 | 2015-02-04 20:34 | |||
| Reporter | ngarcia | View Status | public | |||||
| Assigned To | umartirena | |||||||
| Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR15Q2 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | d4a038cf2708 | ||||
| Projection | none | ETA | none | Target Version | 3.0PR15Q2 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Merge Request Status | ||||||||
| Review Assigned To | Sandrahuguet | |||||||
| OBNetwork customer | OBPS | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Support ticket | 33659 | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0028640: Reservation line is duplicated if it is already released and it is selected when managing it from Sales Order window | |||||||
| Description | Reservation line is duplicated if it is already released and it is selected when managing it from Sales Order window | |||||||
| Steps To Reproduce | As group admin role: Create a Sales Order Add 10 units of Zumo de Naranja Bio 0.33L Book it Click on Manage Reservation and Done Click again and reduce the reserved quantity (4 units) Select a reservation line with a different attribute set and add a quantity less than the pending quantity (2 units) Create a Goods shipment with the same attribute set and quantity (4 units) than the first reservation line and complete it Return to the Sales Order, click on the Manage Reservation and increase the quantity of the second line without unmarking the already released one Check a new reservation line of 4 units is created Note: You should unselect the released line not to duplicate it. The problem is not reproduced from the Stock Reservation window | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0073726) hgbot (developer) 2015-01-22 17:05 |
Repository: erp/devel/pi Changeset: d4a038cf2708b1e64c68b9f9a0015e0d419126a1 Author: Unai Martirena <unai.martirena <at> openbravo.com> Date: Thu Jan 22 17:05:07 2015 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/d4a038cf2708b1e64c68b9f9a0015e0d419126a1 [^] Fixes bug 28640:Res. line with amt released not duplicated in Manage Reservation The problem is that when opening Manage Reservations P&E from sales order line, the property 'reservationStock' comes blank, even when a reservation stock line exists (When there is no reservation for the sales orderline a temporary one is created just to display lines in the selector selected correctly, and after that is deleted. This happens in the datasource of the P&E). In this cases the Manage Reservations action handler deletes these lines y they exists and creates new ones. But this is not possible for already released lines, so it was creating a duplicate of those. In order to fix this, the property 'id' is used (allways comes with a value) in the case that the reservation is released, because in this case for sure that the reservation stock line exists, so it will update the existing one instead of creating a new one. --- M src/org/openbravo/common/actionhandler/ManageReservationActionHandler.java --- |
|
(0073849) Sandrahuguet (viewer) 2015-01-28 10:21 |
Code review + testing OK |
|
(0074173) hudsonbot (viewer) 2015-02-04 20:34 |
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/f36c91d0ad63 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2015-01-16 10:35 | ngarcia | New Issue | |
| 2015-01-16 10:35 | ngarcia | Assigned To | => Sandrahuguet |
| 2015-01-16 10:35 | ngarcia | OBNetwork customer | => Yes |
| 2015-01-16 10:35 | ngarcia | Modules | => Core |
| 2015-01-16 10:35 | ngarcia | Support ticket | => 33659 |
| 2015-01-16 10:35 | ngarcia | Resolution time | => 1424300400 |
| 2015-01-16 10:35 | ngarcia | Triggers an Emergency Pack | => No |
| 2015-01-16 10:36 | ngarcia | Steps to Reproduce Updated | View Revisions |
| 2015-01-22 13:05 | umartirena | Assigned To | Sandrahuguet => umartirena |
| 2015-01-22 17:05 | hgbot | Checkin | |
| 2015-01-22 17:05 | hgbot | Note Added: 0073726 | |
| 2015-01-22 17:05 | hgbot | Status | new => resolved |
| 2015-01-22 17:05 | hgbot | Resolution | open => fixed |
| 2015-01-22 17:05 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/d4a038cf2708b1e64c68b9f9a0015e0d419126a1 [^] |
| 2015-01-22 17:08 | umartirena | Review Assigned To | => Sandrahuguet |
| 2015-01-28 10:21 | Sandrahuguet | Note Added: 0073849 | |
| 2015-01-28 10:21 | Sandrahuguet | Status | resolved => closed |
| 2015-01-28 10:21 | Sandrahuguet | Fixed in Version | => 3.0PR15Q2 |
| 2015-02-04 20:34 | hudsonbot | Checkin | |
| 2015-02-04 20:34 | hudsonbot | Note Added: 0074173 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |