Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033274Retail ModulesWeb POSpublic2016-06-15 13:032016-06-22 14:48
Reporterguillermogil 
Assigned Tomigueldejuana 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customerOBPS
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2016-06-06
Regression introduced in releaseRR16Q1.3
Regression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.retail.returns/rev/404e800a91154eea4e0519afb65f9e476a49ef34 [^]
Triggers an Emergency PackNo
Summary0033274: If you create a verified returns with an orderline that has 2 shipments it picks the wrong qty
DescriptionIf you create a verified returns with an orderline that has 2 shipments it picks the wrong qty.
It must pick the Qty from the shipment not the returnedQty
Steps To Reproduce-Create a ticket on WebPOS that creates 2 shipment lines. eg.- qty=2 and 2 warehouses with qty 1
-Then go to verified returns popup and find that Order
-Select the line
-Click Ok

Realize that 2 lines appeared with qty 2
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 0033283RR16Q2.1 closed Retail If you create a verified returns with an orderline that has 2 shipments it picks the wrong qty 
caused by defect 0033149 closed migueldejuana It is possible to add twice the same verified return partially to the order when the maximum quantity is reached 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-06-15 13:03guillermogilNew Issue
2016-06-15 13:03guillermogilAssigned To => Retail
2016-06-15 13:03guillermogilOBNetwork customer => Yes
2016-06-15 13:03guillermogilResolution time => 1466114400
2016-06-15 13:03guillermogilRegression level => Production - Confirmed Stable
2016-06-15 13:03guillermogilRegression date => 2016-06-06
2016-06-15 13:03guillermogilRegression introduced in release => RR16Q1.3
2016-06-15 13:03guillermogilRegression introduced by commit => https://code.openbravo.com/erp/pmods/org.openbravo.retail.returns/rev/404e800a91154eea4e0519afb65f9e476a49ef34 [^]
2016-06-15 13:03guillermogilTriggers an Emergency Pack => No
2016-06-15 13:08guillermogilRelationship addedcaused by 0033149
2016-06-15 17:30malsasuaIssue Monitored: malsasua
2016-06-16 09:25marvintmStatusnew => scheduled
2016-06-16 09:25marvintmStatusscheduled => acknowledged
2016-06-20 09:51migueldejuanaStatusacknowledged => scheduled
2016-06-20 09:51migueldejuanaNote Added: 0087650
2016-06-20 09:51migueldejuanaAssigned ToRetail => migueldejuana
2016-06-20 09:51migueldejuanaNote Added: 0087651
2016-06-20 09:51migueldejuanaStatusscheduled => resolved
2016-06-20 09:51migueldejuanaResolutionopen => fixed
2016-06-22 09:43marvintmReview Assigned To => marvintm
2016-06-22 09:43marvintmStatusresolved => closed
2016-06-22 14:48hgbotCheckin
2016-06-22 14:48hgbotNote Added: 0087794

Notes
(0087650)
migueldejuana   
2016-06-20 09:51   
Backout regressions and fix them avoiding creating more issues
(0087651)
migueldejuana   
2016-06-20 09:51   
Repository: erp/pmods/org.openbravo.retail.returns
Changeset: 2aef76e761868a6e58b9c89fdb5b51522e6d59a4
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu Jun 16 09:46:26 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.returns/rev/2aef76e761868a6e58b9c89fdb5b51522e6d59a4 [^] [^] [^]

Fixed issue 0030816: It is possible to return the same ticket more than once

- Check all open orders when adding a verified return line to the ticket
- If we exceed the qty available to return the line is not added
- Manage correctly remaining qty, initial qty and selected qty

---
M web/org.openbravo.retail.returns/js/modalReturnLines.js
---

Repository: erp/pmods/org.openbravo.retail.returns
Changeset: 5c1446007fc8dd4339071363c1782db4347cec7e
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu Jun 16 11:21:47 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.returns/rev/5c1446007fc8dd4339071363c1782db4347cec7e [^] [^] [^]

Fixed issue 0030816: It is possible to return the same ticket more than once

- Cover also lines with item in differents bins

---
M web/org.openbravo.retail.returns/js/modalReturnLines.js
---
(0087794)
hgbot   
2016-06-22 14:48   
Repository: tools/automation/pi-mobile
Changeset: 402654942b2e410c7ab3e0cabc86818953eecb6a
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Wed Jun 22 18:18:11 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/402654942b2e410c7ab3e0cabc86818953eecb6a [^]

Verifies issue 33274 : Added automated test 'I33274_VerifyReturnWithMultiShipment'

---
M src-test/org/openbravo/test/mobile/common/selenium/javascript/TestIdBase.java
M src-test/org/openbravo/test/mobile/quarantine/pack/RAN/returns/I33149_VerifyReturnQtyWithTotalQty.java
A src-test/org/openbravo/test/mobile/quarantine/pack/RAN/returns/I33274_VerifyReturnWithMultiShipment.java
A src-test/org/openbravo/test/mobile/quarantine/pack/RAN/returns/I33274_VerifyReturnWithMultiShipmentII.java
---