Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0034461
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajoralways2016-11-10 12:462021-01-21 15:33
ReporterxabiermerinoView Statuspublic 
Assigned Tomario_castello 
PriorityurgentResolutionfixedFixed in VersionRR17Q1
StatusclosedFix in branchFixed in SCM revision42c6e292b16f
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomarvintm
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0034461: [SERQA 2019] When creating a sales order in the pos, the shipment is created with an incorrect warehouse in the header

DescriptionThere are different things to fix:
1) The Query to select the warehouse needs to be improved and check if the WH is active or not. Nowaday this validation is not done and it can select a WH that is not active.
2) The lines of the shipments have a storage bin that does not belong to the warehouse defined in the header. Proposed solution is the following: when inserting the first line of the shipment check if the storage bin belongs to the WH defined in the header. If it belongs to the WH do nothing. If it does not belong, update the WH in header with the correct WH.

Furthermore, The button "Active" in the header of the screen "Warehouse and Storage Bin" is not effective. For instance, if we unflag this field, the warehouse still appears in the Webpos (cf. Video enclosed)
Steps To ReproduceSteps to reproduce: when the store has more than one WH and all of them has the same priority.
1) Create an order in the WebPOS
2) check in the backend the shipment created for that order. The lines has storge bins that don't belong to the WH defined in the header.

Other issue: (see video)
1) In the BO, unflag the field "Active" for the screen "Warehouse and Storage Bin"
2) In the Webpos, creating a ticket, the warehouse is still available
TagsSER-QA
Attached Filesdiff file icon issue34461.diff [^] (1,840 bytes) 2016-11-10 13:08 [Show Content]
? file icon 34461.mp4 [^] (2,649,068 bytes) 2016-11-11 15:43

- Relationships [ Relation Graph ] [ Dependency Graph ]
has duplicate defect 0035358 closedmario_castello Storage Bin does not correspond with the Warehouse in those cases where a store has several warehouses 
related to defect 0043233 closedranjith_qualiantech_com Stock is not taken from the warehouse that has stock but for the warehouse without stock if it has only one storage bin 

-  Notes
(0091405)
hgbot (developer)
2016-11-11 15:41

 Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 0c7ecbfdafd982eb0cebf69901caa9f219ff97d1
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Thu Nov 10 09:33:45 2016 -0600
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/0c7ecbfdafd982eb0cebf69901caa9f219ff97d1 [^]

Fixed issue 34461: Set the same warehouse in the shipmentLines and in the header of the shipment

---
M src/org/openbravo/retail/posterminal/OrderLoader.java
M src/org/openbravo/retail/posterminal/POSUtils.java
---
(0091406)
hgbot (developer)
2016-11-11 15:42

 Repository: tools/automation/pi-mobile
Changeset: ce8d5c699dd0fe1ffcad563e3ed8377a0cdd2d96
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Thu Nov 10 09:57:06 2016 -0600
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/ce8d5c699dd0fe1ffcad563e3ed8377a0cdd2d96 [^]

Verifies issue 34461: Added automated test I34461_VerifyWareHouseShipmetLines

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/receipts/I34461_VerifyWareHouseShipmetLines.java
---
(0091566)
hgbot (developer)
2016-11-15 15:00

 Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 9bed60ef230869b21181ac5fd23060be171daea9
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Mon Nov 14 10:03:22 2016 -0600
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/9bed60ef230869b21181ac5fd23060be171daea9 [^]

Fixed issue 34461: Added validation in warehouse class to load the active warehouse

---
M src/org/openbravo/retail/posterminal/term/Warehouses.java
---
(0091567)
hgbot (developer)
2016-11-15 15:13

 Repository: tools/automation/pi-mobile
Changeset: f21a78cdcf6baa268e237ba34aebd2f03d6ec0af
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Mon Nov 14 10:20:49 2016 -0600
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/f21a78cdcf6baa268e237ba34aebd2f03d6ec0af [^]

Verifies issue 34461: Added automated test I34461_VerifyWareHouseOrderAndShipmetLines

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/receipts/I34461_VerifyWareHouseOrderAndShipmetLines.java
---
(0091687)
marvintm (manager)
2016-11-21 10:07

 Now only the active warehouses are loaded in the Web POS, which is correct. However, the "Store stock: " and "Other stores stock: " buttons seemingly still don't filter by the active property for the warehouses, so they still appear there if they are not active, and this is not correct.
(0091850)
hgbot (developer)
2016-11-25 14:51

 Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 42c6e292b16f77732ad8890dc3e5edb482c97a7b
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Mon Nov 21 10:39:07 2016 -0600
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/42c6e292b16f77732ad8890dc3e5edb482c97a7b [^]

Fixed issue 0034461: [SERQA 2019] When creating a sales order in the pos, the shipment is created with an incorrect warehouse in the header

- Added validation in the hql query for StoreDetailedStock class to load the information for all active warehouse
- Added validation in OtherStoresDetailedStock class

---
M src/org/openbravo/retail/posterminal/stock/OtherStoresDetailedStock.java
M src/org/openbravo/retail/posterminal/stock/StoreDetailedStock.java
---
(0091851)
hgbot (developer)
2016-11-25 14:53

 Repository: tools/automation/pi-mobile
Changeset: 70c2ceeedd2cdd1644f514a6c9e7d0c6b55f9dde
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Tue Nov 22 08:46:14 2016 -0600
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/70c2ceeedd2cdd1644f514a6c9e7d0c6b55f9dde [^]

Verifies issue 34461: Added automated test I34461_VerifyWareHouseAndStock

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/receipts/I34461_VerifyWareHouseAndStock.java
---
(0125714)
hgbot (developer)
2021-01-21 15:33

 Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: eea3efdf43d8a3f00d547420379dd59b5d5a520c
Author: Prakash M <prakash@qualiantech.com>
Date: 2021-01-21T20:02:22+05:30
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/eea3efdf43d8a3f00d547420379dd59b5d5a520c [^]

Related to BUG-34461: Fixes test I34461_VerifyWareHouseShipmetLines in try high volume
* Handled missed scenarios to set the same warehouse in the shipmentLines and in the header of the shipment
* Refer Commit 0c7ecbfdafd982eb0cebf69901caa9f219ff97d1

---
M src/org/openbravo/retail/posterminal/utility/ShipmentUtils.java
---

- Issue History
Date Modified Username Field Change
2016-11-10 12:46 xabiermerino New Issue
2016-11-10 12:46 xabiermerino Assigned To => Retail
2016-11-10 12:46 xabiermerino Resolution time => 1478818800
2016-11-10 12:46 xabiermerino Triggers an Emergency Pack => No
2016-11-10 12:46 xabiermerino Tag Attached: SER-QA
2016-11-10 13:08 marvintm File Added: issue34461.diff
2016-11-10 13:11 xabiermerino Resolution time 1478818800 => 1479423600
2016-11-10 14:58 mario_castello Assigned To Retail => mario_castello
2016-11-10 15:08 mario_castello Status new => acknowledged
2016-11-10 15:22 mario_castello Status acknowledged => scheduled
2016-11-11 15:41 hgbot Checkin
2016-11-11 15:41 hgbot Note Added: 0091405
2016-11-11 15:41 hgbot Status scheduled => resolved
2016-11-11 15:41 hgbot Resolution open => fixed
2016-11-11 15:41 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/0c7ecbfdafd982eb0cebf69901caa9f219ff97d1 [^]
2016-11-11 15:42 hgbot Checkin
2016-11-11 15:42 hgbot Note Added: 0091406
2016-11-11 15:43 paul_guiot Status resolved => scheduled
2016-11-11 15:43 paul_guiot Resolution fixed => open
2016-11-11 15:43 paul_guiot Fixed in SCM revision http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/0c7ecbfdafd982eb0cebf69901caa9f219ff97d1 [^] =>
2016-11-11 15:43 paul_guiot Description Updated View Revisions
2016-11-11 15:43 paul_guiot Steps to Reproduce Updated View Revisions
2016-11-11 15:43 paul_guiot File Added: 34461.mp4
2016-11-15 15:00 hgbot Checkin
2016-11-15 15:00 hgbot Note Added: 0091566
2016-11-15 15:00 hgbot Status scheduled => resolved
2016-11-15 15:00 hgbot Resolution open => fixed
2016-11-15 15:00 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/9bed60ef230869b21181ac5fd23060be171daea9 [^]
2016-11-15 15:13 hgbot Checkin
2016-11-15 15:13 hgbot Note Added: 0091567
2016-11-21 10:07 marvintm Note Added: 0091687
2016-11-21 10:07 marvintm Status resolved => new
2016-11-21 10:07 marvintm Resolution fixed => open
2016-11-21 15:39 mario_castello Status new => acknowledged
2016-11-21 15:39 mario_castello Status acknowledged => scheduled
2016-11-25 14:51 hgbot Checkin
2016-11-25 14:51 hgbot Note Added: 0091850
2016-11-25 14:51 hgbot Status scheduled => resolved
2016-11-25 14:51 hgbot Resolution open => fixed
2016-11-25 14:51 hgbot Fixed in SCM revision http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/9bed60ef230869b21181ac5fd23060be171daea9 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/42c6e292b16f77732ad8890dc3e5edb482c97a7b [^]
2016-11-25 14:53 hgbot Checkin
2016-11-25 14:53 hgbot Note Added: 0091851
2016-11-25 15:13 marvintm Review Assigned To => marvintm
2016-11-25 15:13 marvintm Status resolved => closed
2016-11-25 15:13 marvintm Fixed in Version => RR17Q1
2017-02-27 18:04 mario_castello Relationship added blocks 0035358
2017-02-28 15:08 mario_castello Relationship deleted blocks 0035358
2017-02-28 15:09 mario_castello Relationship added has duplicate 0035358
2020-02-24 12:06 ranjith_qualiantech_com Relationship added related to 0043233
2021-01-21 15:33 hgbot Note Added: 0125714

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker