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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0033268
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajoralways2016-06-15 09:552016-07-12 17:19
ReporterguillermogilView Statuspublic 
Assigned Toranjith_qualiantech_com 
PriorityimmediateResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisioneb17a591481b
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomarvintm
Regression levelProduction - Confirmed Stable
Regression date2015-03-08
Regression introduced in releaseRR15Q2
Regression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/45cc17f59918 [^]
Triggers an Emergency PackNo
Summary

0033268: M_GetStock in OrderLoader is not taking into account the Priority Warehouse

DescriptionM_GetStock in OrderLoader is not taking into account the Priority Warehouse.
Once the new way to call M_getStock was implemented it change the behaviour and instead of sending the priority_warehouse the warehouse parameter gets sent.
Steps To ReproduceHave 2 warehouses defined for the store, one with some qty for avalanche transceiver and other with none (The priority warehouse).
Then, on webPOS, add the avalanche transceiver and complete the sale.
You will see that on the priority warehouse you have negative stock.
Proposed SolutionJust change the call to:
StockUtils.getStock(processId, recordID, quantity, productId, null, null, warehouseId, orgId, attributesetinstanceId, OBContext.getOBContext().getUser().getId(), clientId, warehouseRuleId, uomId, null, null, null, null, null, reservationId, "N");
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
depends on backport 0033281RR16Q2.1 closedranjith_qualiantech_com M_GetStock in OrderLoader is not taking into account the Priority Warehouse 
depends on backport 0033282RR16Q1.4 closedranjith_qualiantech_com M_GetStock in OrderLoader is not taking into account the Priority Warehouse 

-  Notes
(0087292)
dmitry_mezentsev (developer)
2016-06-15 10:26

 RR15Q2 is QAA.
Anyway we need to check if the regression was introduced earlier or not.
(0087294)
guillermogil (manager)
2016-06-15 11:54

 15Q2 -> 2015, it has more than one year
(0087297)
dmitry_mezentsev (developer)
2016-06-15 12:49

 Yes, my bad, confused 15 with 16.
(0087698)
hgbot (developer)
2016-06-21 08:03

 Repository: tools/automation/pi-mobile
Changeset: b915c6b1fc0a82ab14ea18037bb2f20bd7fd5832
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Jun 21 11:18:00 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/b915c6b1fc0a82ab14ea18037bb2f20bd7fd5832 [^]

Verifies issue 33268 : Added automated test 'I30734_CheckWarehouseByLine'

---
A src-test/org/openbravo/test/mobile/quarantine/pack/RAN/sales/I30734_CheckWarehouseByLine.java
R src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/sales/I30734_CheckWarehouseByLine.java
---
(0087700)
hgbot (developer)
2016-06-21 08:03

 Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: eb17a591481b2a1dd4349177155662489f6eb547
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Jun 21 11:17:53 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/eb17a591481b2a1dd4349177155662489f6eb547 [^]

Fixes issue 33268 : Order Loader getStock, warehouseId is replaced with priorityWarehouseId

---
M src/org/openbravo/retail/posterminal/OrderLoader.java
---

- Issue History
Date Modified Username Field Change
2016-06-15 09:55 guillermogil New Issue
2016-06-15 09:55 guillermogil Assigned To => Retail
2016-06-15 09:55 guillermogil Resolution time => 1466114400
2016-06-15 09:55 guillermogil Regression level => Production - Confirmed Stable
2016-06-15 09:55 guillermogil Regression date => 2015-03-08
2016-06-15 09:55 guillermogil Regression introduced in release => RR15Q2
2016-06-15 09:55 guillermogil Regression introduced by commit => https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c0a35478e6ff [^]
2016-06-15 09:55 guillermogil Triggers an Emergency Pack => No
2016-06-15 10:26 dmitry_mezentsev Regression level Production - Confirmed Stable => Production - QA Approved
2016-06-15 10:26 dmitry_mezentsev Note Added: 0087292
2016-06-15 11:54 guillermogil Note Added: 0087294
2016-06-15 12:48 dmitry_mezentsev Regression level Production - QA Approved => Production - Confirmed Stable
2016-06-15 12:49 dmitry_mezentsev Note Added: 0087297
2016-06-16 09:23 marvintm Status new => scheduled
2016-06-16 09:24 marvintm Status scheduled => acknowledged
2016-06-16 11:30 ranjith_qualiantech_com Assigned To Retail => ranjith_qualiantech_com
2016-06-16 11:30 ranjith_qualiantech_com Status acknowledged => scheduled
2016-06-21 08:03 hgbot Checkin
2016-06-21 08:03 hgbot Note Added: 0087698
2016-06-21 08:03 hgbot Checkin
2016-06-21 08:03 hgbot Note Added: 0087700
2016-06-21 08:03 hgbot Status scheduled => resolved
2016-06-21 08:03 hgbot Resolution open => fixed
2016-06-21 08:03 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/eb17a591481b2a1dd4349177155662489f6eb547 [^]
2016-06-21 09:02 marvintm Review Assigned To => marvintm
2016-06-21 09:02 marvintm Status resolved => closed
2016-07-12 17:19 marvintm Regression introduced by commit https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c0a35478e6ff [^] => https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/45cc17f59918 [^]

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker