Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0032098 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
backport | [Openbravo ERP] 04. Warehouse management | major | always | 2016-02-01 17:13 | 2016-02-11 11:02 | |||
Reporter | aferraz | View Status | public | |||||
Assigned To | aferraz | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | d5d1e92037f8 | ||||
Projection | none | ETA | none | Target Version | 3.0PR16Q1 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | dmiguelez | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | Packaging and release | |||||||
Regression date | 2015-12-04 | |||||||
Regression introduced in release | main | |||||||
Regression introduced by commit | https://code.openbravo.com/erp/devel/pi/rev/e32a6dfe59e6 [^] | |||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0032098: Reserved quantity is not correct under certain circumstances | |||||||
Description | Reserved quantity is not correct under certain circumstances. | |||||||
Steps To Reproduce | Reservations are disabled by default. To be able to use them please insert a new Preference using property Enable Stock Reservations with value Y. http://wiki.openbravo.com/wiki/Stock_Reservation [^] [^] As group Admin role, Go to Physical inventory and remove the stock for Cerveza Lager. Keep 700 units in Warehouse España Región Norte 0-0-0 Go to Sales Order window, Org: España Lines: Cerveza Lager: qty: 900 Stock Reservation: blank Book it. Once you select the line, you will see Manage Reservations button. Click on it, and indicate you would like to reserve 400 units. Click on Done. Go to Stock reservation window. Realize that the register created indicates 700 units, but the user said 400. That is wrong. If you manage Stock again from the Stock Reservation window, and you indicate 400, now the register has the correct qty. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||
|
Notes | |
(0083968) hgbot (developer) 2016-02-05 13:34 |
Repository: erp/backports/3.0PR16Q1 Changeset: d5d1e92037f8d8d90c34193bcda955a091bd742d Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com> Date: Fri Feb 05 11:37:39 2016 +0100 URL: http://code.openbravo.com/erp/backports/3.0PR16Q1/rev/d5d1e92037f8d8d90c34193bcda955a091bd742d [^] Fixes issue 32098: Reserved quantity is not correct In ManageReservationActionHandler, when clicking Done in Manage Reservation P&E, it was first reserving what the user was typing in the P&E, and if it was not processed the reservation, it was processing it. As the M_RESERVATION_POST tries always to reserve as much as possible, it was overriding the previously set amounts. In order to prevent this, now first the reservation is being processed and after that the values typed in the UI are being set to the reservation. In case no stock reservation is created by m_reservation_post when processing the reservation, res_status will be Draft. To be sure reservation is set as completed, force status after call ReservationUtils.processReserve method. Reservation will only be kept as draft in case it has no lines after execute manageReservedStockLines. --- M src/org/openbravo/common/actionhandler/ManageReservationActionHandler.java --- |
(0083969) dmiguelez (developer) 2016-02-05 13:34 |
Code Review + Testing Ok |
Issue History | |||
Date Modified | Username | Field | Change |
2016-02-02 17:14 | aferraz | Type | defect => backport |
2016-02-02 17:14 | aferraz | Target Version | 3.0PR16Q2 => 3.0PR16Q1 |
2016-02-05 13:34 | hgbot | Checkin | |
2016-02-05 13:34 | hgbot | Note Added: 0083968 | |
2016-02-05 13:34 | hgbot | Status | scheduled => resolved |
2016-02-05 13:34 | hgbot | Resolution | open => fixed |
2016-02-05 13:34 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/backports/3.0PR16Q1/rev/d5d1e92037f8d8d90c34193bcda955a091bd742d [^] |
2016-02-05 13:34 | dmiguelez | Review Assigned To | => dmiguelez |
2016-02-05 13:34 | dmiguelez | Note Added: 0083969 | |
2016-02-05 13:34 | dmiguelez | Status | resolved => closed |
2016-02-11 11:02 | aferraz | Assigned To | Triage Finance => aferraz |
Copyright © 2000 - 2009 MantisBT Group |