|View Issue Details|
|Type||Category||Severity||Reproducibility||Date Submitted||Last Update|
|defect||[Modules] Advance Warehouse Operations||major||always||2018-05-02 17:07||2018-09-18 13:27|
|Priority||urgent||Resolution||open||Fixed in Version|
|Status||new||Fix in branch||Fixed in SCM revision||dda305110cef|
|OS Version||Database version||Ant version|
|Product Version||SCM revision|
|Regression introduced by commit|
|Review Assigned To||dmiguelez|
|Regression introduced in release|
0038482: Pending to Generate Picking column in Sales Order Lines not properly updated when Delta tasks are generated
|Description||Pending to Generate Picking column in Sales Order Lines not properly updated when Delta tasks are generated|
|Steps To Reproduce||With AWO and AWO SampleData modules installed|
Log in as AWO QA Admin
From the Front End, create Stock for Ale Beer in Bin Beverage101, Beverage102, Beverage103
Click on Count button from menu.
Search for Beverage101 Bin
Search for Ale Beer Product
Set 100 as confirmed quantity and confirm
Set again 100 as confirmed quantity and confirm
Go to Sales Order window and create a new record with:
- Organization: US West Coast
- Business Partner: Healthy Food Supermarkets
- Warehouse: US West Coast
Move to Lines and create a new record with:
- Product: Ale Beer
- Operative Quantity: 5
Book the Order.
Check Pending To Generate Picking column of Sales Order Line is 5
Click on Pick. A Task is generated and Pending to Generate Picking is 0
Move to the Task and set 1 as confirmed.
Click on confirm and set different as delta
A new Task is created for 4 units and the Pending to Generate Picking is 4 (this is already wrong)
Delete the new Task, the Pending to Generate Picking is 8
|Proposed Solution||Delete db columns C_Order.EM_OBAWO_INCOMPLETE_PICKING and C_OrderLine.EM_OBAWO_PENDING_QTY_PICKING|
Create computed columns instead.
Ensure no filtering/sorting is allowed at Sales Order header's column to avoid performance issues.
The logic in C_Order column is currently wrong and must be fixed.
This column represents the number of lines pending to be completely picked. It shows how many lines are pending to confirm the picking action through tasks. For example, if a line has a picking task for the full ordered quantity but not yet confirmed, it will be considered as a Pending To Pick Line.
The logic in C_OrderLine column is OK.
This column represents the pending Quantity to generate Picking for products declared as stocked. If the order line has picking task(s) (independent of the status) for the full ordered quantity, this field will show 0, as no more picking is necessary to be generated.
Update elements' help to properly explain the purpose of both columns
|Tags||No tags attached.|
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Thu Sep 06 12:00:20 2018 +0200
URL: http://code.openbravo.com/retail/backports/AWO/3.0RR18Q3/org.openbravo.warehouse.advancedwarehouseoperations/rev/dda305110cefdf799f1274f4d371c933f5e7e77f [^]
Fixed bug 38482: C_Order.EM_OBAWO_INCOMPLETE_PICKING and C_OrderLine.EM_OBAWO_PENDING_QTY_PICKING
These db columns have been deleted and transformed to computed columns.
This drastically reduces the complexity to maintain these two columns.
From a performance POV I don't see a problem in C_OrderLine.EM_OBAWO_PENDING_QTY_PICKING as the number of lines in an order are usually low.
In the case C_Order.EM_OBAWO_INCOMPLETE_PICKING I do expect performance problems, so it's not possible to filter/order by this column. We should maybe think on replacing this column by a more meaningful physical column.
The old C_Order.EM_OBAWO_INCOMPLETE_PICKING logic was wrong, because it was just like a count of order lines with something pending to generate picking.
The purpose of the C_Order.EM_OBAWO_INCOMPLETE_PICKING is to show the lines pending to CONFIRM picking (not to generate), so the logic has been modified accordingly.
These changes have required to adapt JUnit tests to:
1. Work properly with the computed columns
2. Assert the right values in the C_Order.EM_OBAWO_INCOMPLETE_PICKING column.
Any old setter from these columns have been deleted.
|2018-05-02 17:07||dmiguelez||New Issue|
|2018-05-02 17:07||dmiguelez||Assigned To||=> dmiguelez|
|2018-05-02 17:07||dmiguelez||Assigned To||dmiguelez => Triage Finance|
|2018-09-05 17:32||vmromanos||Status||new => scheduled|
|2018-09-05 17:32||vmromanos||Assigned To||Triage Finance => vmromanos|
|2018-09-05 17:38||vmromanos||Review Assigned To||=> dmiguelez|
|2018-09-05 17:38||vmromanos||Proposed Solution updated|
|2018-09-14 14:58||hgbot||Note Added: 0106783|
|2018-09-14 14:58||hgbot||Status||scheduled => resolved|
|2018-09-14 14:58||hgbot||Resolution||open => fixed|
|2018-09-14 14:58||hgbot||Fixed in SCM revision||=> http://code.openbravo.com/retail/backports/AWO/3.0RR18Q3/org.openbravo.warehouse.advancedwarehouseoperations/rev/dda305110cefdf799f1274f4d371c933f5e7e77f [^]|
|2018-09-18 13:27||vmromanos||Status||resolved => new|
|2018-09-18 13:27||vmromanos||Resolution||fixed => open|
|Copyright © 2000 - 2009 MantisBT Group|