Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0031824
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajorhave not tried2016-01-05 17:572016-01-19 15:18
ReportermtaalView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision886ed432a5b8
ProjectionnoneETAnoneTarget VersionRR16Q2
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomigueldejuana
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0031824: 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.
TagsNo tags attached.
Attached Filestxt file icon trigger.txt [^] (50 bytes) 2016-01-05 17:57 [Show Content]
txt file icon stack-trace.txt [^] (54,739 bytes) 2016-01-05 17:58 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
depends on backport 0031825RR16Q1 closedmtaal Import entry may not get saved in case of low level database error, webpos sees message as succesfully send 
depends on backport 0031826RR15Q4.2 closedmtaal Import entry may not get saved in case of low level database error, webpos sees message as succesfully send 
causes defect 0037301 closedmarvintm Java Heap Memory can rise on login if batch size is big 

-  Notes
(0083145)
hgbot (developer)
2016-01-05 22:11

Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 727a4f04e3e023624bbdc36cdb682e95304129ad
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Tue Jan 05 18:20:56 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/727a4f04e3e023624bbdc36cdb682e95304129ad [^]

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
---
(0083405)
migueldejuana (developer)
2016-01-19 15:18

Tested and reviewed in RR16Q2

- Issue History
Date Modified Username Field Change
2016-01-05 17:57 mtaal New Issue
2016-01-05 17:57 mtaal Assigned To => mtaal
2016-01-05 17:57 mtaal File Added: trigger.txt
2016-01-05 17:57 mtaal Triggers an Emergency Pack => No
2016-01-05 17:57 mtaal Status new => scheduled
2016-01-05 17:58 mtaal File Added: stack-trace.txt
2016-01-05 17:58 mtaal Review Assigned To => marvintm
2016-01-05 22:11 hgbot Checkin
2016-01-05 22:11 hgbot Note Added: 0083145
2016-01-05 22:11 hgbot Status scheduled => resolved
2016-01-05 22:11 hgbot Resolution open => fixed
2016-01-05 22:11 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/727a4f04e3e023624bbdc36cdb682e95304129ad [^]
2016-01-14 16:15 hgbot Checkin
2016-01-14 16:15 hgbot Note Added: 0083305
2016-01-14 16:15 hgbot Fixed in SCM revision http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/727a4f04e3e023624bbdc36cdb682e95304129ad [^] => http://code.openbravo.com/retail/backports/3.0RR16Q1/org.openbravo.mobile.core/rev/886ed432a5b837bbd459ea148b8c195d9873aef2 [^]
2016-01-14 16:18 mtaal Note Deleted: 0083305
2016-01-18 16:47 mtaal Review Assigned To marvintm => migueldejuana
2016-01-19 15:18 migueldejuana Note Added: 0083405
2016-01-19 15:18 migueldejuana Status resolved => closed
2017-11-16 07:40 guillermogil Relationship added causes 0037301


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker