Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034354Retail ModulesLayaway orderpublic2016-10-31 17:162016-11-21 13:55
Reporterlorenzofidalgo 
Assigned Toranjith_qualiantech_com 
PriorityimmediateSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customer
Support ticket
Regression levelPackaging and release
Regression date2016-09-13
Regression introduced in releasemain
Regression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/eab31cfd06da#l1.7 [^]
Triggers an Emergency PackNo
Summary0034354: A layaway receipt that has been voided must not be shown in "Layaways" menu option. Currently it is shown there
DescriptionWhen a Layaway receipt is voided it must not be found in Layaways menu option. Currently, it is being shown there. Amounts are shown as 0.
I have been able to reproduce this issue in PI and RR16Q4. It is not reproducible in RR16Q3.2.
Steps To Reproduce0-Login POS as vallblanca.
1-Create a new receipt and add 1x "Avalanche transceiver".
2-In dropdown menu, click "Layaway this Receipt".
3-Click Total Amount button and then click "Layaway" button.
4-Now, in dropdown menu, go to Layaways and recover the just created layaway.
5-In dropdown menu, click "Void this Layaway" and click Done.
6-Again, go to dropdown menu and click in Layaways. Click in the magnifier and the just voided layaway will be shown. DEFECT: It must not be shown.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 0034373RR16Q4 closed ranjith_qualiantech_com A layaway receipt that has been voided must not be shown in "Layaways" menu option. Currently it is shown there 
caused by defect 0033962 closed Sandrahuguet Performance: searching for a receipt should be improved 
Attached Filespng VoidedLayawayIsNotRemoved.png (193,462) 2016-10-31 17:16
https://issues.openbravo.com/file_download.php?file_id=10000&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2016-10-31 17:16lorenzofidalgoNew Issue
2016-10-31 17:16lorenzofidalgoAssigned To => Retail
2016-10-31 17:16lorenzofidalgoFile Added: VoidedLayawayIsNotRemoved.png
2016-10-31 17:16lorenzofidalgoRegression level => Packaging and release
2016-10-31 17:16lorenzofidalgoRegression introduced in release => main
2016-10-31 17:16lorenzofidalgoTriggers an Emergency Pack => No
2016-11-02 09:41lorenzofidalgoNote Added: 0091051
2016-11-02 16:06ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2016-11-02 16:06ranjith_qualiantech_comStatusnew => scheduled
2016-11-03 11:01hgbotCheckin
2016-11-03 11:01hgbotNote Added: 0091073
2016-11-03 11:01hgbotStatusscheduled => resolved
2016-11-03 11:01hgbotResolutionopen => fixed
2016-11-03 11:01hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/f0828665b4c30dad39bb9e6397db9f10d55addbe [^]
2016-11-03 11:02hgbotCheckin
2016-11-03 11:02hgbotNote Added: 0091074
2016-11-03 11:13ranjith_qualiantech_comStatusresolved => new
2016-11-03 11:13ranjith_qualiantech_comResolutionfixed => open
2016-11-03 11:13ranjith_qualiantech_comStatusnew => scheduled
2016-11-03 11:13ranjith_qualiantech_comStatusscheduled => resolved
2016-11-03 11:13ranjith_qualiantech_comResolutionopen => fixed
2016-11-04 10:47ranjith_qualiantech_comRegression introduced by commit => https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/eab31cfd06da#l1.7 [^]
2016-11-04 10:49ranjith_qualiantech_comRegression date => 2016-09-13
2016-11-04 17:05dmitry_mezentsevRelationship addedcaused by 0033962
2016-11-21 13:32migueldejuanaReview Assigned To => migueldejuana
2016-11-21 13:55migueldejuanaNote Added: 0091699
2016-11-21 13:55migueldejuanaStatusresolved => closed

Notes
(0091051)
lorenzofidalgo   
2016-11-02 09:41   
Besides, the voided layaway can still be "paid". Select that receipt in Layaways option and Total Amount button is enabled to continue the pay flow.
(0091073)
hgbot   
2016-11-03 11:01   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: f0828665b4c30dad39bb9e6397db9f10d55addbe
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Nov 03 15:31:09 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/f0828665b4c30dad39bb9e6397db9f10d55addbe [^]

Fixes issue 34354 : Added condition for layaway query to show completed layaways

---
M src/org/openbravo/retail/posterminal/PaidReceiptsHeader.java
---
(0091074)
hgbot   
2016-11-03 11:02   
Repository: tools/automation/pi-mobile
Changeset: ddc8619556e000e23e35840de345a39ddc7c2da6
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Nov 03 15:30:35 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/ddc8619556e000e23e35840de345a39ddc7c2da6 [^]

Verifies issue 34354 : Added automated test 'I34354_VerifyVoidLayawayInList'

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I34354_VerifyVoidLayawayInList.java
---
(0091699)
migueldejuana   
2016-11-21 13:55   
Tested and reviewed 87c1e0574f75