Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0036196
TypeCategorySeverityReproducibilityDate SubmittedLast Update
design defect[Retail Modules] StoreServerminorhave not tried2017-06-08 09:192017-06-16 15:47
ReportermtaalView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in VersionRR17Q3
StatusclosedFix in branchFixed in SCM revision37dc93155f97
ProjectionnoneETAnoneTarget VersionRR17Q3
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomarvintm
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0036196: Prevent same ticket from being send twice by the user from being processed

DescriptionThere are special cases that the user submits a ticket but the request is processed on the server but the response is lost. In this case the ticket remains in the browser and the user can send it again (after refreshing for example to enable the pay button).

Currently an 'ugly' error message is shown. The system should catch this case and show the proper error message.
Steps To ReproducePut breakpoint in orderloader
submit ticket
when it hits breakpoint refresh the page, ticket is shown again
Let the server run through the breakpoint, loading the ticket
Submit the ticket
See error message
Proposed SolutionRe-use the current logic [1] to check for duplicate sending of layaways for all tickets, show a nicer message.

[1]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/98e87f3b39b6/src/org/openbravo/retail/posterminal/OrderLoader.java#l247 [^]
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to design defect 0036033RR17Q3 closedmtaal Prevent duplicate/parallel processing of the same message from webpos if it gets retried due to long processing times 
related to defect 0035289 closedmtaal BUT: Duplicate transaction in POS (already done) 

-  Notes
(0097198)
hgbot (developer)
2017-06-08 09:20

Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 37dc93155f97c67749a5081f41fc00df38d53457
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Jun 08 09:19:56 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/37dc93155f97c67749a5081f41fc00df38d53457 [^]

Fixes issue 36196: Prevent same ticket from being send twice by the user from being processed
Use optimistic locking approach of layaways for all tickets, show nicer message

---
M src-db/database/sourcedata/AD_MESSAGE.xml
M src/org/openbravo/retail/posterminal/OrderLoader.java
---

- Issue History
Date Modified Username Field Change
2017-06-08 09:19 mtaal New Issue
2017-06-08 09:19 mtaal Assigned To => mtaal
2017-06-08 09:19 mtaal Triggers an Emergency Pack => No
2017-06-08 09:19 mtaal Review Assigned To => marvintm
2017-06-08 09:20 hgbot Checkin
2017-06-08 09:20 hgbot Note Added: 0097198
2017-06-08 09:20 hgbot Status new => resolved
2017-06-08 09:20 hgbot Resolution open => fixed
2017-06-08 09:20 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/37dc93155f97c67749a5081f41fc00df38d53457 [^]
2017-06-08 13:44 mtaal Relationship added related to 0036033
2017-06-16 15:47 marvintm Status resolved => closed
2017-06-16 15:47 marvintm Fixed in Version => RR17Q3
2017-06-19 11:09 mtaal Relationship added related to 0035289


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker