Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0038626 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| backport | [Retail Modules] Web POS | major | always | 2018-05-24 17:44 | 2018-06-25 09:04 | |||
| Reporter | migueldejuana | View Status | public | |||||
| Assigned To | migueldejuana | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | RR18Q2.1 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | |||||
| Projection | none | ETA | none | Target Version | RR18Q2.1 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | marvintm | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0038626: 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 |
|
|
(0105344) migueldejuana (developer) 2018-06-22 13:36 |
Repository: retail/backports/3.0RR18Q2.1/org.openbravo.mobile.core Changeset: 46107eed74ff1fe645269105ce82650fa1dddee2 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.0RR18Q2.1/org.openbravo.mobile.core/rev/46107eed74ff1fe645269105ce82650fa1dddee2 [^] [^] 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 |
|
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 | => RR18Q2.1 |
| 2018-06-22 13:36 | migueldejuana | Note Added: 0105344 | |
| 2018-06-22 13:36 | migueldejuana | Status | scheduled => resolved |
| 2018-06-22 13:36 | migueldejuana | Resolution | open => fixed |
| 2018-06-25 09:04 | marvintm | Review Assigned To | => marvintm |
| 2018-06-25 09:04 | marvintm | Status | resolved => closed |
| 2018-06-25 09:04 | marvintm | Fixed in Version | => RR18Q2.1 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |