Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0031826Retail ModulesWeb POSpublic2016-01-05 17:572016-01-19 14:58
Reportermtaal 
Assigned Tomtaal 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR15Q4.2Fixed in Version 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0031826: Import entry may not get saved in case of low level database error, webpos sees message as succesfully send
DescriptionWhen there is a low level error when saving an import entry the WebPOS client will think that the message is saved on the server and will remove it from the local cache.
Steps To Reproduce1- Login to webpos
2- Create a trigger which gives an exception, set as after-insert trigger on c_import_entry table. See the example trigger function attached to the issue.
3- Create a ticket in webpos and submit
4- The ticket is not saved on the server but still removed from webpos

See the attached stack trace for what the console shows.
Proposed Solutionon the server create the correct error json and set a failure status code so that the webpos client correctly signals an error.
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0031824RR16Q2 closed mtaal Import entry may not get saved in case of low level database error, webpos sees message as succesfully send 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-01-05 17:57mtaalTypedefect => backport
2016-01-05 17:57mtaalTarget VersionRR16Q2 => RR15Q4.3
2016-01-14 10:43mtaalTarget VersionRR15Q4.3 => RR15Q4.2
2016-01-14 15:32hgbotCheckin
2016-01-14 15:32hgbotNote Added: 0083302
2016-01-14 15:32hgbotStatusscheduled => resolved
2016-01-14 15:32hgbotResolutionopen => fixed
2016-01-14 15:32hgbotFixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR15Q4.2/org.openbravo.mobile.core/rev/ebf29401c3eefc66e85e0fe4e3034c0d5408b0df [^]
2016-01-18 16:45mtaalReview Assigned To => migueldejuana
2016-01-19 14:58migueldejuanaNote Added: 0083403
2016-01-19 14:58migueldejuanaStatusresolved => closed

Notes
(0083302)
hgbot   
2016-01-14 15:32   
Repository: retail/backports/3.0RR15Q4.2/org.openbravo.mobile.core
Changeset: ebf29401c3eefc66e85e0fe4e3034c0d5408b0df
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Jan 14 15:31:47 2016 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR15Q4.2/org.openbravo.mobile.core/rev/ebf29401c3eefc66e85e0fe4e3034c0d5408b0df [^]

Fixes issue 31826: Import entry may not get saved in case of low level database error, webpos sees message as succesfully send
Create correct error json and return status code 500 on error

---
M src/org/openbravo/mobile/core/process/MobileService.java
---
(0083403)
migueldejuana   
2016-01-19 14:58   
Tested and reviewed in RR15Q4.2