Anonymous | Login
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Modules] Advance Warehouse Operationsmajoralways2018-05-02 17:072018-09-18 13:27
ReporterdmiguelezView Statuspublic 
Assigned Tovmromanos 
PriorityurgentResolutionopenFixed in Version
StatusnewFix in branchFixed in SCM revisiondda305110cef
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Regression date
Regression introduced by commit
Regression level
Review Assigned Todmiguelez
Regression introduced in release

0038482: Pending to Generate Picking column in Sales Order Lines not properly updated when Delta tasks are generated

DescriptionPending to Generate Picking column in Sales Order Lines not properly updated when Delta tasks are generated
Steps To ReproduceWith 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 SolutionDelete 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
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
hgbot (developer)
2018-09-14 14:58

Repository: retail/backports/AWO/3.0RR18Q3/org.openbravo.warehouse.advancedwarehouseoperations
Changeset: dda305110cefdf799f1274f4d371c933f5e7e77f
Author: Víctor Martínez Romanos <victor.martinez <at>>
Date: Thu Sep 06 12:00:20 2018 +0200
URL: [^]


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.

M src-db/database/model/functions/OBAWO_CLOSE_ORDER_EP.xml
M src-db/database/model/modifiedTables/C_ORDER.xml
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/AD_ELEMENT.xml
M src-db/database/sourcedata/AD_FIELD.xml
M src-test/org/openbravo/warehouse/advancedwarehouseoperations/test/
M src-test/org/openbravo/warehouse/advancedwarehouseoperations/utils/
M src/org/openbravo/warehouse/advancedwarehouseoperations/eventobserver/
M src/org/openbravo/warehouse/advancedwarehouseoperations/ittalgorithm/implementation/
M src/org/openbravo/warehouse/advancedwarehouseoperations/ittalgorithm/implementation/
M src/org/openbravo/warehouse/advancedwarehouseoperations/ittalgorithm/implementation/
A src-db/database/model/functions/OBAWO_ORDERLINE_PENDINGPICKING.xml
A src-db/database/model/functions/OBAWO_ORDER_PENDINGPICKING.xml
R src-db/database/model/modifiedTables/C_ORDERLINE.xml
R src/org/openbravo/warehouse/advancedwarehouseoperations/eventobserver/
R src/org/openbravo/warehouse/advancedwarehouseoperations/eventobserver/

- Issue History
Date Modified Username Field Change
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 Checkin
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 => [^]
2018-09-18 13:27 vmromanos Status resolved => new
2018-09-18 13:27 vmromanos Resolution fixed => open

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker