Openbravo Issue Tracking System - Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0048123ModulesAdvanced Warehouse Operationspublic2021-11-22 15:492021-12-22 13:56
Reporterguilleaer 
Assigned Toranjith_qualiantech_com 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Regression date
Regression introduced by commit
Regression level
Review Assigned To
Support ticket
OBNetwork customer
Regression introduced in release
Summary0048123: using not blind count -> qty counted is 0 if nothing is counted but UI shows qtyOnHand as counted
Descriptionusing not blind count -> qty counted is 0 if nothing is counted but UI shows qtyOnHand
Steps To ReproduceHaving preference OBAWO_BlindInventoryCount to false
Create a count for an specific bin
See that counted input is equal to qty on hand
Without using the UI confirm the count

Generated task count is 0 instead of qtyOnHand (Which is the value shown in the screeen)
Proposed Solution1. We must revert the changesets related to 0046635 (https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations/-/merge_requests/24 [^])

2. In the FE, the counted qty should always reflect what is stored into the backend. This simplifies a lot the logic in the FE.

3. Rename existing preference "Allow Blind Inventory Count" to "Force Blind Inventory Count" (and keep the same behavior). This better explains the purpose of the preference.

4. New preference "Create Blind Inventory Counts by default". Two values allowed:
- Y [default value if preference is not set]
- N

5. New flag "Blind Count" at Inventory Count header (both in BE and FE).
It is always visible and editable in BE (to give more power to the manager).
In the FE is only editable when the "Force Blind Inventory Count" preference is set to N.

The default value (BE and FE) will match the "Default behavior for WMS Inventory Count" preference.
Note: in case "Force Blind Inventory Count" is not available or set to Y, then the "Blind Count" flag will be set to Yes regardless of the "Default behavior for WMS Inventory Count" preference.

6. New flag "Initialize Quantity Count with Book Quantity" (false by default) at Inventory Count header (both in BE and FE). When set, the quantity count will be defaulted to the book quantity. If not set, the qty count will be set to 0 (no change). This behavior must be implemented in both BE and FE, and both when generating a count from a list, or when manually adding new lines to count.

Frontend flag summary:
2 flags:
flag 1: Inventory count
Preference Blind Inventory Count = true -> "Blind count" field will be shown in read only mode with value YES
Preference Blind Inventory Count = false -> "Blind count" field will be shown as editable and the value will come from preference "Default behavior for WMS Inventory Count"

Flag 2: Initialize Quantity Count with Book Quantity
Only will be shown if previous flag is editable and it is marked as FALSE

7. The new flag "Initialize Quantity Count with Book Quantity" will be visible only for Non-blind counts. For blind counts, the qty count is defaulted to 0 (no change) so there is no need to show this flag.

8. When creating new lines in frontend -> the property "Initialize Quantity Count with Book Quantity" should be checked to decide if lines are created with 0 or with qtyOnHand

9. When creating new lines in frontend -> the property "Initialize Quantity Count with Book Quantity" should be checked to decide if lines are created with 0 or with qtyOnHand
Additional Information
TagsNo tags attached.
Relationships
has duplicate defect 0048135 closed vmromanos Inventory Count: unable to make difference between "not counted yet" and "counted at 0" 
related to defect 0048241 closed vmromanos "Initialize Quantity Count with Book Quantity" and manual backoffice lines 
related to defect 0048379 closed vmromanos Blind Count and Initialize Qty Count with Book Qty when launching a Count/Recount from Warehouse Operations 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2021-11-22 15:49guilleaerNew Issue
2021-11-22 15:49guilleaerAssigned To => davidverrier
2021-11-23 13:20guilleaerRelationship addedblocks 0048135
2021-11-24 13:55PracticsIssue Monitored: Practics
2021-12-09 11:26guilleaerProposed Solution updated
2021-12-09 11:26guilleaerStatusnew => scheduled
2021-12-09 11:26guilleaerAssigned Todavidverrier => ranjith_qualiantech_com
2021-12-09 11:40guilleaerProposed Solution updated
2021-12-09 12:20guilleaerProposed Solution updated
2021-12-09 12:28guilleaerProposed Solution updated
2021-12-13 07:01hgbotNote Added: 0133630
2021-12-13 13:44hgbotNote Added: 0133647
2021-12-13 18:05guilleaerRelationship addedrelated to 0048241
2021-12-14 12:29hgbotNote Added: 0133679
2021-12-14 15:03hgbotNote Added: 0133689
2021-12-14 15:03hgbotNote Added: 0133690
2021-12-14 15:04hgbotNote Added: 0133691
2021-12-14 15:04hgbotNote Added: 0133692
2021-12-14 15:04hgbotResolutionopen => fixed
2021-12-14 15:04hgbotStatusscheduled => closed
2021-12-14 15:04hgbotNote Added: 0133693
2021-12-14 15:04hgbotNote Added: 0133694
2021-12-14 16:10vmromanosNote Added: 0133697
2021-12-14 16:13vmromanosRelationship replacedhas duplicate 0048135
2021-12-21 20:03hgbotNote Added: 0133888
2021-12-22 13:54hgbotNote Added: 0133902
2021-12-22 13:54hgbotNote Added: 0133903
2021-12-22 13:56guilleaerNote Added: 0133904
2022-01-11 17:32vmromanosRelationship addedrelated to 0048379
2022-09-06 17:18caristuCategoryAdvance Warehouse Operations => Advanced Warehouse Operations

Notes
(0133630)
hgbot   
2021-12-13 07:01   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations/-/merge_requests/39 [^]
(0133647)
hgbot   
2021-12-13 13:44   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/313 [^]
(0133679)
hgbot   
2021-12-14 12:29   
Merge Request created: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/151 [^]
(0133689)
hgbot   
2021-12-14 15:03   
Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: f9e5303370402568b101e6ba29a8f0cef6f62173
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 14-12-2021 14:03:55
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/f9e5303370402568b101e6ba29a8f0cef6f62173 [^]

Related to ISSUE-48123: Added CSS for Checkbox Checked and Disabled Status

---
M web/org.openbravo.mobile.core/assets/css/obmobc-main.css
---
(0133690)
hgbot   
2021-12-14 15:03   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/313 [^]
(0133691)
hgbot   
2021-12-14 15:04   
Repository: https://gitlab.com/openbravo/ci/mobile-test [^]
Changeset: 08b6f00215365b8a5134f4be165283fdd0cff733
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 14-12-2021 14:03:59
URL: https://gitlab.com/openbravo/ci/mobile-test/-/commit/08b6f00215365b8a5134f4be165283fdd0cff733 [^]

Verifies ISSUE-48123: Updated AWO test ICI030_BlindCount

---
M src-test/org/openbravo/test/mobile/awo/tests/Count/ICI030_BlindCount.java
---
(0133692)
hgbot   
2021-12-14 15:04   
Merge request merged: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/151 [^]
(0133693)
hgbot   
2021-12-14 15:04   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations [^]
Changeset: 91c205c24cb94e6141ff7f1fdb711a75327dc860
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 14-12-2021 14:04:05
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations/-/commit/91c205c24cb94e6141ff7f1fdb711a75327dc860 [^]

Fixed ISSUE-48123: Updated Count Quantity by default based on Preference

---
M src-db/database/model/tables/OBAWO_COUNT.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-db/database/sourcedata/AD_MESSAGE.xml
M src-db/database/sourcedata/AD_PREFERENCE.xml
M src-db/database/sourcedata/AD_REF_LIST.xml
M src/org/openbravo/warehouse/advancedwarehouseoperations/eventobserver/InventoryCountEventObserver.java
M src/org/openbravo/warehouse/advancedwarehouseoperations/mobile/datasources/InventoryCountInfoProvider.java
M src/org/openbravo/warehouse/advancedwarehouseoperations/mobile/processors/InventoryCountProcessor.java
M src/org/openbravo/warehouse/advancedwarehouseoperations/utils/inventorycount/InventoryCountGenerateListManager.java
M src/org/openbravo/warehouse/advancedwarehouseoperations/utils/inventorycount/InventoryCountGenerateList_NonShared.java
M src/org/openbravo/warehouse/advancedwarehouseoperations/utils/inventorycount/InventoryCountGenerateList_Shared.java
M web/org.openbravo.warehouse.advancedwarehouseoperations/source/components/obawo-invcountfilter-selector.js
M web/org.openbravo.warehouse.advancedwarehouseoperations/source/components/obawo-users-selector.js
M web/org.openbravo.warehouse.advancedwarehouseoperations/source/inventorycount/components/obawo-productselector.js
M web/org.openbravo.warehouse.advancedwarehouseoperations/source/inventorycount/components/obawo-qtykeypad.js
M web/org.openbravo.warehouse.advancedwarehouseoperations/source/inventorycount/obawo-inventorycount-model.js
M web/org.openbravo.warehouse.advancedwarehouseoperations/source/inventorycount/obawo-inventorycount-view.js
M web/org.openbravo.warehouse.advancedwarehouseoperations/source/models/centralbroker.js
---
(0133694)
hgbot   
2021-12-14 15:04   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations/-/merge_requests/39 [^]
(0133697)
vmromanos   
2021-12-14 16:10   
Documentation: http://wiki.openbravo.com/wiki/Modules:Advanced_Warehouse_Operations/Inventory_Count#Blind_counts [^]
(0133888)
hgbot   
2021-12-21 20:03   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations/-/merge_requests/42 [^]
(0133902)
hgbot   
2021-12-22 13:54   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations [^]
Changeset: 0821d84848a6ea60d87c3d0a7c39002dd52ede2e
Author: Guillermo Alvarez de Eulate <guillermo.alvarez@openbravo.com>
Date: 22-12-2021 12:54:06
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations/-/commit/0821d84848a6ea60d87c3d0a7c39002dd52ede2e [^]

fixes ISSUE-48123: Add preference to default how not blind counts are initialized

---
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/AD_PREFERENCE.xml
M src-db/database/sourcedata/AD_REF_LIST.xml
M web/org.openbravo.warehouse.advancedwarehouseoperations/source/components/obawo-users-selector.js
---
(0133903)
hgbot   
2021-12-22 13:54   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations/-/merge_requests/42 [^]
(0133904)
guilleaer   
2021-12-22 13:56   
Documentation:

http://wiki.openbravo.com/wiki/Modules:Advanced_Warehouse_Operations/Inventory_Count#Blind_counts [^]