Openbravo Issue Tracking System - Retail Modules
View Issue Details
0033289Retail ModulesWeb POSpublic2016-06-16 12:292016-06-25 21:38
aaroncalero 
ranjith_qualiantech_com 
normalmajoralways
closedfixed 
5
 
RR16Q3 
mtaal
No
0033289: [SERQA 1216] Error loading a deferred service sale if the service is based on price rule ranges
Given a service with its price calculated based on a Price Rule configured by ranges, if we create a deferred ticket with that product, and then load that ticket again in web pos, an "Error calculating service price" is raised, which is not correct as the price has been calculated previously.
Using Retail sampledata:
Login in backend, go to the Product window, open the Customize Coat product and check the Allow Deferred Sale flag.
Login in webpos (VBS-1 terminal) and create a new ticket:
Add the product Coat, don't add any mandatory service (click on Continue), and pay the ticket.
Go to the Menu > Receipts and load the previous ticket.
Click on the Related Services button of the Coat line; the related services for Coat will be shown.
Try to add a Customize Coat product.
The "Not Editable Receipt" popup will be shown, allowing to add the Customize Coat service to a different ticket. Select *Create New Receipt and click on Apply.
A new ticket will be created with the Customize Coat product. Verify that an information line will show the relationship between this line and the original product line.
Pay the ticket.
Go to the Menu > Receipts and load the ticket with the Customize Coat.
After the ticket is loaded, the "Error calculating service price" error will raise and the service will be removed.
The issue comes from the fact that some values (gross, net, qty and promotions) are not being loaded with the Related Lines information when a deferred ticket is loaded again in web pos.
The attached diff adds these values on the PaidReceip.java
SER-QA
? I33289.export (2,750) 2016-06-16 12:32
https://issues.openbravo.com/file_download.php?file_id=9544&type=bug
Issue History
2016-06-16 12:29aaroncaleroNew Issue
2016-06-16 12:29aaroncaleroAssigned To => Retail
2016-06-16 12:29aaroncaleroResolution time => 1467237600
2016-06-16 12:29aaroncaleroTriggers an Emergency Pack => No
2016-06-16 12:31aaroncaleroSummaryError loading a deferred service sale if the service is based on price rule ranges => [SERQA 1216] Error loading a deferred service sale if the service is based on price rule ranges
2016-06-16 12:31aaroncaleroProposed Solution updated
2016-06-16 12:31aaroncaleroTag Attached: SER-QA
2016-06-16 12:32aaroncaleroFile Added: I33289.export
2016-06-16 15:52ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2016-06-16 15:52ranjith_qualiantech_comStatusnew => scheduled
2016-06-20 06:11hgbotCheckin
2016-06-20 06:11hgbotNote Added: 0087645
2016-06-20 06:11hgbotStatusscheduled => resolved
2016-06-20 06:11hgbotResolutionopen => fixed
2016-06-20 06:11hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6eec6a438276f5e178fff53ced0e73330ea477d0 [^]
2016-06-21 05:52hgbotCheckin
2016-06-21 05:52hgbotNote Added: 0087689
2016-06-23 19:20marvintmReview Assigned To => mtaal
2016-06-25 21:38mtaalNote Added: 0087919
2016-06-25 21:38mtaalStatusresolved => closed
2016-06-25 21:38mtaalFixed in Version => RR16Q3

Notes
(0087645)
hgbot   
2016-06-20 06:11   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 6eec6a438276f5e178fff53ced0e73330ea477d0
Author: Aaron Calero <aaron.calero <at> openbravo.com>
Date: Thu Jun 16 12:29:40 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6eec6a438276f5e178fff53ced0e73330ea477d0 [^]

Fixed issue 33289 : Error loading a deferred service sale if the service is based on price rule ranges

---
M src/org/openbravo/retail/posterminal/PaidReceiptRelatedLinesProperties.java
M src/org/openbravo/retail/posterminal/PaidReceipts.java
---
(0087689)
hgbot   
2016-06-21 05:52   
Repository: tools/automation/pi-mobile
Changeset: f14ec41ebf957d1d08402abc5bb93036424ff806
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Jun 21 09:22:39 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/f14ec41ebf957d1d08402abc5bb93036424ff806 [^]

Verifies issue 33289 : Modified automated test 'I32930_VerifyDeferredServiceAmount'

---
A src-test/org/openbravo/test/mobile/quarantine/pack/RAN/services/I32930_VerifyDeferredServiceAmount.java
R src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/services/deferredsell/I32930_VerifyDeferredServiceAmount.java
---
(0087919)
mtaal   
2016-06-25 21:38   
Reviewed and tested