Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034634Retail ModulesWeb POSpublic2016-11-25 11:372018-07-13 22:10
Reporterasiermartirena 
Assigned Toasiermartirena 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q1 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0034634: When doing a Cancel Layaway, the new order must have the terminal ID of the current terminal
DescriptionWhen doing a Cancel Layaway, the new order must have the terminal ID of the current terminal
Steps To ReproduceLogin the Web POS.
Create a layaway.
Login in the Web POS with other terminal.
Load the previously created layaway and cancel it.
The new ticket has the firs terminal and should have the second one.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
has duplicate defect 0034606 closed mario_castello [SER-QA 2049] Cannot cancel a partially paid layaway created in another terminal 
related to defect 0034084 new Retail Pending test to be developed 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-11-25 11:37asiermartirenaNew Issue
2016-11-25 11:37asiermartirenaAssigned To => asiermartirena
2016-11-25 11:37asiermartirenaTriggers an Emergency Pack => No
2016-11-28 09:58hgbotCheckin
2016-11-28 09:58hgbotNote Added: 0091878
2016-11-28 09:58hgbotStatusnew => resolved
2016-11-28 09:58hgbotResolutionopen => fixed
2016-11-28 09:58hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c11fa631858d319812a5f8a0e3beba2320b9c88b [^]
2016-11-28 12:13marvintmRelationship addedhas duplicate 0034606
2016-12-05 09:27hgbotCheckin
2016-12-05 09:27hgbotNote Added: 0092086
2016-12-05 13:26hgbotCheckin
2016-12-05 13:26hgbotNote Added: 0092090
2016-12-21 17:44marvintmRelationship addedrelated to 0034084
2016-12-21 18:01marvintmReview Assigned To => marvintm
2016-12-21 18:01marvintmStatusresolved => closed
2016-12-21 18:01marvintmFixed in Version => RR17Q1
2018-07-13 22:10hgbotCheckin
2018-07-13 22:10hgbotNote Added: 0105743

Notes
(0091878)
hgbot   
2016-11-28 09:58   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: c11fa631858d319812a5f8a0e3beba2320b9c88b
Author: Asier Martirena <asier.martirena <at> openbravo.com>
Date: Mon Nov 28 09:58:30 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c11fa631858d319812a5f8a0e3beba2320b9c88b [^]

Fixed issue 34634: When doing a Cancel Layaway, the new order must have the terminal ID of the current terminal

---
M web/org.openbravo.retail.posterminal/js/pointofsale/model/pointofsale-model.js
---
(0092086)
hgbot   
2016-12-05 09:27   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: ae2043a00df634e9355757ec409360d0169206ee
Author: Asier Martirena <asier.martirena <at> openbravo.com>
Date: Mon Dec 05 09:27:20 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/ae2043a00df634e9355757ec409360d0169206ee [^]

Related to issue 34634: The terminalId was not correctly set after doing a clone of the original order

---
M src/org/openbravo/retail/posterminal/CancelAndReplaceHook.java
M src/org/openbravo/retail/posterminal/CancelLayawayPaymentsHookExtension.java
M src/org/openbravo/retail/posterminal/POSUtils.java
---
(0092090)
hgbot   
2016-12-05 13:26   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: bcf7143e066e415d4753b46422c5c623de3e25d1
Author: Asier Martirena <asier.martirena <at> openbravo.com>
Date: Mon Dec 05 13:09:26 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/bcf7143e066e415d4753b46422c5c623de3e25d1 [^]

Related to issue 34634: Code in hook must be executed if jsonorder exists

---
M src/org/openbravo/retail/posterminal/CancelAndReplaceHook.java
---
(0105743)
hgbot   
2018-07-13 22:10   
Repository: tools/automation/pi-mobile
Changeset: 5f2391dce5c1159169c34a98364ad9af36829375
Author: Alejandro <alekosmp86 <at> gmail.com>
Date: Fri Jul 13 09:48:21 2018 -0400
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/5f2391dce5c1159169c34a98364ad9af36829375 [^]

Related to issue 34634: added automated test

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I34634_VerifyNewTicketAfterCancelLayaway.java
---