Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0038627 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| backport | [Retail Modules] Web POS | major | always | 2018-05-24 17:44 | 2018-08-02 10:29 | |||
| Reporter | migueldejuana | View Status | public | |||||
| Assigned To | migueldejuana | |||||||
| Priority | normal | Resolution | out of date | Fixed in Version | ||||
| Status | closed | Fix in branch | Fixed in SCM revision | |||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | ||||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0038627: Avoid loosing orders if backend throws exception while inserting import entry | |||||||
| Description | When an order is send to backend, it is removed from local database when we get the success response. It is remove because the import entry is created and no need to have the message in local database. There is one case thaht is not covered. When synchornizating the message, in the process of inserting the import entry we could throw an exception(i.e.: The backend timeout finish before client timeout. Wring development). This exception is send as success with status -1. Wrongly, we are considering it as a success and we remove the message loosing the order. | |||||||
| Steps To Reproduce | - Add an exception in the process of saving the Import Entry. In WebServiceAuthenticatedServlet for example. - Create an order and process it - ERROR! The Exception will be thrown, the import entry is not created and the message is removed from local database. | |||||||
| Proposed Solution | In the success callback, if the status is -1, do not remove the message and call error callback. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0105345) migueldejuana (developer) 2018-06-22 13:36 |
Repository: retail/backports/3.0RR18Q1.4/org.openbravo.mobile.core Changeset: 4afd30b27187169188357f2db94debe4f0e736fe Author: Miguel de Juana <miguel.dejuana <at> openbravo.com> Date: Fri May 25 10:07:56 2018 +0200 URL: http://code.openbravo.com/retail/backports/3.0RR18Q1.4/org.openbravo.mobile.core/rev/4afd30b27187169188357f2db94debe4f0e736fe [^] [^] Fixed issue 0038621: Avoid loosing orders if backend throws exception while inserting import entry - Handle response with status -1 and follow the error flow to do not remove the message which couldn't be saved --- M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js --- |
|
(0106110) plujan (manager) 2018-08-02 10:28 |
Closed as Q1 reached end of support |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2018-05-25 10:12 | migueldejuana | Type | defect => backport |
| 2018-05-25 10:12 | migueldejuana | Target Version | => RR18Q1.4 |
| 2018-06-22 13:36 | migueldejuana | Note Added: 0105345 | |
| 2018-06-22 13:36 | migueldejuana | Status | scheduled => resolved |
| 2018-06-22 13:36 | migueldejuana | Resolution | open => fixed |
| 2018-08-02 10:28 | plujan | Note Added: 0106110 | |
| 2018-08-02 10:28 | plujan | Status | resolved => closed |
| 2018-08-02 10:29 | plujan | Resolution | fixed => out of date |
| 2018-08-02 10:29 | plujan | Target Version | RR18Q1.4 => |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |