Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033083Retail ModulesWeb POSpublic2016-05-31 12:142016-06-27 11:14
Reporterpaul_guiot 
Assigned Toranjith_qualiantech_com 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR16Q3 
Merge Request Status
Review Assigned Tomtaal
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033083: [SERQA 1197] Wrong label "To be return" appears in the Layaway and it should not
DescriptionIf we attempt to return a layaway, we got a wrong label on the layaway: "To be return"
Steps To Reproduceo to the WEBPOS with the user Vallblanca (https://livebuilds.openbravo.com/retail_pi_pgsql/web/org.openbravo.retail.posterminal/?terminal=VBS-1#retail.pointofsale [^])

- Create a ticket:
o Search
o Select the article: Adhesive body warmers
o The product is added to the receipt
- In the menu, click on “Layaway this receipt”
- In the menu, click on “Return this receipt”
- The following message appears: “You cannot layaway a sale with returns”.

Nevertheless, the “to be Returned” Label appears on the layaway after those steps. This label should not appear on the ticket.
Proposed SolutionRemove the label "To be Return"
Additional Information
TagsNo tags attached.
Relationships
related to defect 0033068 closed ranjith_qualiantech_com [SERQA 1182] Layaway: The system changes the customer name without reasons 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-05-31 12:14paul_guiotNew Issue
2016-05-31 12:14paul_guiotAssigned To => Retail
2016-05-31 12:14paul_guiotResolution time => 1466632800
2016-05-31 12:14paul_guiotTriggers an Emergency Pack => No
2016-06-01 13:09OrekariaStatusnew => acknowledged
2016-06-01 14:07ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2016-06-01 14:47ranjith_qualiantech_comStatusacknowledged => scheduled
2016-06-02 12:32hgbotCheckin
2016-06-02 12:32hgbotNote Added: 0086910
2016-06-02 12:32hgbotCheckin
2016-06-02 12:32hgbotNote Added: 0086911
2016-06-02 12:32hgbotStatusscheduled => resolved
2016-06-02 12:32hgbotResolutionopen => fixed
2016-06-02 12:32hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c2cec31c97a493524a35aa253ddd081670cb0cf6 [^]
2016-06-02 17:30migueldejuanaNote Added: 0086952
2016-06-02 17:30migueldejuanaStatusresolved => new
2016-06-02 17:30migueldejuanaResolutionfixed => open
2016-06-03 08:10ranjith_qualiantech_comStatusnew => scheduled
2016-06-03 11:27ranjith_qualiantech_comRelationship addedrelated to 0033068
2016-06-06 07:47hgbotCheckin
2016-06-06 07:47hgbotNote Added: 0087008
2016-06-06 07:47hgbotStatusscheduled => resolved
2016-06-06 07:47hgbotResolutionopen => fixed
2016-06-06 07:47hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c2cec31c97a493524a35aa253ddd081670cb0cf6 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/2fd48f07be2c4a3714c3d318a75ab9b297e705f0 [^]
2016-06-06 07:47hgbotCheckin
2016-06-06 07:47hgbotNote Added: 0087009
2016-06-22 09:08marvintmReview Assigned To => mtaal
2016-06-25 20:29mtaalNote Added: 0087918
2016-06-25 20:29mtaalStatusresolved => new
2016-06-25 20:29mtaalResolutionfixed => open
2016-06-27 08:27ranjith_qualiantech_comStatusnew => scheduled
2016-06-27 08:38hgbotCheckin
2016-06-27 08:38hgbotNote Added: 0087938
2016-06-27 08:39ranjith_qualiantech_comStatusscheduled => resolved
2016-06-27 08:39ranjith_qualiantech_comResolutionopen => fixed
2016-06-27 11:13mtaalReview Assigned Tomtaal => Orekaria
2016-06-27 11:14mtaalReview Assigned ToOrekaria => mtaal
2016-06-27 11:14mtaalNote Added: 0087954
2016-06-27 11:14mtaalStatusresolved => closed
2016-06-27 11:14mtaalFixed in Version => RR16Q3

Notes
(0086910)
hgbot   
2016-06-02 12:32   
Repository: tools/automation/pi-mobile
Changeset: 815c40ca9407093285a66a08967aaf9aaf7d1891
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Jun 02 15:59:26 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/815c40ca9407093285a66a08967aaf9aaf7d1891 [^]

Verifies issue 33083 : Added automated test 'I33083_VerifyReturnWithLayaway'

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/returns/I33083_VerifyReturnWithLayaway.java
---
(0086911)
hgbot   
2016-06-02 12:32   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: c2cec31c97a493524a35aa253ddd081670cb0cf6
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Jun 02 16:00:56 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c2cec31c97a493524a35aa253ddd081670cb0cf6 [^]

Fixes issue 33083 : Restrict to make layaway receipt as "Return" return

- if there are any changes in layaway receipt, it must be restrict to make as "Return" return

---
M web/org.openbravo.retail.posterminal/js/components/menu.js
---
(0086952)
migueldejuana   
2016-06-02 17:30   
If we press "Return this receipt" option, "Layaway this receipt" option disappears.
We should do the same in the other way round.
If we press "Layaway this receipt" option, "Return this receipt" option disappears.
We don't have to show a message because we have to hide this option to the user.
(0087008)
hgbot   
2016-06-06 07:47   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 2fd48f07be2c4a3714c3d318a75ab9b297e705f0
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Mon Jun 06 11:16:21 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/2fd48f07be2c4a3714c3d318a75ab9b297e705f0 [^]

Fixes issue 33083 : Hide "Return this receipt" from menu if current receipt is layaway

- Backout changeset c2cec31c97a4

---
M web/org.openbravo.retail.posterminal/js/components/menu.js
---
(0087009)
hgbot   
2016-06-06 07:47   
Repository: tools/automation/pi-mobile
Changeset: 3d4ac6078df101cc1dedeb349119b7f04d4d8419
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Mon Jun 06 11:16:05 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/3d4ac6078df101cc1dedeb349119b7f04d4d8419 [^]

Verifies issue 33083 : Modified automated test 'I33083_VerifyReturnWithLayaway'

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/returns/I33083_VerifyReturnWithLayaway.java
---
(0087918)
mtaal   
2016-06-25 20:29   
Reviewed, works, have only tiny comment:

Change the description on 2 lines ([1], [2]) in the automated test:
// Verify "Return Receipt" is Visible
to
// Verify "Return Receipt" is not Visible
and
// Verify "Layaway Receipt" is Visible
to
// Verify "Layaway Receipt" is not Visible

[1]
https://code.openbravo.com/tools/automation/pi-mobile/file/3d4ac6078df1/src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/returns/I33083_VerifyReturnWithLayaway.java#l44 [^]

[2]
https://code.openbravo.com/tools/automation/pi-mobile/file/3d4ac6078df1/src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/returns/I33083_VerifyReturnWithLayaway.java#l64 [^]
(0087938)
hgbot   
2016-06-27 08:38   
Repository: tools/automation/pi-mobile
Changeset: 703fad49c015ec816b547ad3661402891ebedd89
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Mon Jun 27 12:08:28 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/703fad49c015ec816b547ad3661402891ebedd89 [^]

Verifies issue 33083 : Modified automated test 'I33083_VerifyReturnWithLayaway'

- Moved test to quarantine
- Add changes from https://issues.openbravo.com/view.php?id=33083#c87918 [^]

---
A src-test/org/openbravo/test/mobile/quarantine/pack/RAN/returns/I33083_VerifyReturnWithLayaway.java
R src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/returns/I33083_VerifyReturnWithLayaway.java
---
(0087954)
mtaal   
2016-06-27 11:14   
Reviewed and tested