Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0036196 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| design defect | [Retail Modules] StoreServer | minor | have not tried | 2017-06-08 09:19 | 2017-06-16 15:47 | |||
| Reporter | mtaal | View Status | public | |||||
| Assigned To | mtaal | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | RR17Q3 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | 37dc93155f97 | ||||
| Projection | none | ETA | none | Target Version | RR17Q3 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Merge Request Status | ||||||||
| Review Assigned To | marvintm | |||||||
| OBNetwork customer | No | |||||||
| Support ticket | ||||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0036196: Prevent same ticket from being send twice by the user from being processed | |||||||
| Description | There 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 Reproduce | Put 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 Solution | Re-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 [^] | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|||||||||||||||
|
|||||||||||||||
 Relationships |
|
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 --- |
|
Notes |
|
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 | OBNetwork customer | => No |
| 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 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |