Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0031825Retail ModulesWeb POSpublic2016-01-05 17:572016-01-19 14:59
Reportermtaal 
Assigned Tomtaal 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR16Q1Fixed in VersionRR16Q1 
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
Summary0031825: 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 => RR16Q1.1
2016-01-14 15:41mtaalTarget VersionRR16Q1.1 => RR16Q1
2016-01-14 16:17mtaalNote Added: 0083306
2016-01-14 16:17mtaalStatusscheduled => resolved
2016-01-14 16:17mtaalFixed in Version => RR16Q1
2016-01-14 16:17mtaalResolutionopen => fixed
2016-01-14 16:18mtaalNote Added: 0083307
2016-01-18 16:46mtaalReview Assigned To => migueldejuana
2016-01-19 14:59migueldejuanaNote Added: 0083404
2016-01-19 14:59migueldejuanaStatusresolved => closed

Notes
(0083306)
mtaal   
2016-01-14 16:17   
Repository: retail/backports/3.0RR16Q1/org.openbravo.mobile.core
Changeset: 886ed432a5b837bbd459ea148b8c195d9873aef2
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Jan 14 16:13:09 2016 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR16Q1/org.openbravo.mobile.core/rev/886ed432a5b837bbd459ea148b8c195d9873aef2 [^]

Fixes issue 31824: 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
---
(0083307)
mtaal   
2016-01-14 16:18   
note comment in fix has wrong issue number
(0083404)
migueldejuana   
2016-01-19 14:59   
Tested and reviewed in RR16Q1