Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033268Retail ModulesWeb POSpublic2016-06-15 09:552016-07-12 17:19
Reporterguillermogil 
Assigned Toranjith_qualiantech_com 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customer
Support ticket
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
Summary0033268: 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");
Additional Information
TagsNo tags attached.
Relationships
depends on backport 0033281RR16Q2.1 closed ranjith_qualiantech_com M_GetStock in OrderLoader is not taking into account the Priority Warehouse 
depends on backport 0033282RR16Q1.4 closed ranjith_qualiantech_com M_GetStock in OrderLoader is not taking into account the Priority Warehouse 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-06-15 09:55guillermogilNew Issue
2016-06-15 09:55guillermogilAssigned To => Retail
2016-06-15 09:55guillermogilResolution time => 1466114400
2016-06-15 09:55guillermogilRegression level => Production - Confirmed Stable
2016-06-15 09:55guillermogilRegression date => 2015-03-08
2016-06-15 09:55guillermogilRegression introduced in release => RR15Q2
2016-06-15 09:55guillermogilRegression introduced by commit => https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c0a35478e6ff [^]
2016-06-15 09:55guillermogilTriggers an Emergency Pack => No
2016-06-15 10:26dmitry_mezentsevRegression levelProduction - Confirmed Stable => Production - QA Approved
2016-06-15 10:26dmitry_mezentsevNote Added: 0087292
2016-06-15 11:54guillermogilNote Added: 0087294
2016-06-15 12:48dmitry_mezentsevRegression levelProduction - QA Approved => Production - Confirmed Stable
2016-06-15 12:49dmitry_mezentsevNote Added: 0087297
2016-06-16 09:23marvintmStatusnew => scheduled
2016-06-16 09:24marvintmStatusscheduled => acknowledged
2016-06-16 11:30ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2016-06-16 11:30ranjith_qualiantech_comStatusacknowledged => scheduled
2016-06-21 08:03hgbotCheckin
2016-06-21 08:03hgbotNote Added: 0087698
2016-06-21 08:03hgbotCheckin
2016-06-21 08:03hgbotNote Added: 0087700
2016-06-21 08:03hgbotStatusscheduled => resolved
2016-06-21 08:03hgbotResolutionopen => fixed
2016-06-21 08:03hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/eb17a591481b2a1dd4349177155662489f6eb547 [^]
2016-06-21 09:02marvintmReview Assigned To => marvintm
2016-06-21 09:02marvintmStatusresolved => closed
2016-07-12 17:19marvintmRegression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c0a35478e6ff [^] => https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/45cc17f59918 [^]

Notes
(0087292)
dmitry_mezentsev   
2016-06-15 10:26   
RR15Q2 is QAA.
Anyway we need to check if the regression was introduced earlier or not.
(0087294)
guillermogil   
2016-06-15 11:54   
15Q2 -> 2015, it has more than one year
(0087297)
dmitry_mezentsev   
2016-06-15 12:49   
Yes, my bad, confused 15 with 16.
(0087698)
hgbot   
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   
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
---