Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0014973Openbravo ERPA. Platformpublic2010-10-21 16:492010-10-30 11:24
mtaal 
mtaal 
normalmajorhave not tried
closedfixed 
5
pi 
pi 
Core
No
0014973: DalWebServiceServlet does not report errors which occur at commit time
The DalWebServiceServlet does not report errors at commit time. This because the commit is postponed to the end of the request after the response has already been generated.
The system should do a commit earlier before the result is written to the response.

See this forum post:
http://forge.openbravo.com/plugins/espforum/view.php?group_id=100&forumid=549512&topicid=7019828 [^]
No tags attached.
Issue History
2010-10-21 16:49mtaalNew Issue
2010-10-21 16:49mtaalAssigned To => mtaal
2010-10-21 17:52hgbotCheckin
2010-10-21 17:52hgbotNote Added: 0032075
2010-10-21 17:52hgbotStatusnew => resolved
2010-10-21 17:52hgbotResolutionopen => fixed
2010-10-21 17:52hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/594ab4625cfa8cac34c8b60bd88f4569176dbc72 [^]
2010-10-22 17:31iperdomoNote Added: 0032124
2010-10-22 17:31iperdomoStatusresolved => closed
2010-10-22 22:43hudsonbotCheckin
2010-10-22 22:43hudsonbotNote Added: 0032134
2010-10-30 11:24anonymoussf_bug_id0 => 3098752

Notes
(0032075)
hgbot   
2010-10-21 17:52   
Repository: erp/devel/pi
Changeset: 594ab4625cfa8cac34c8b60bd88f4569176dbc72
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Oct 21 17:50:13 2010 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/594ab4625cfa8cac34c8b60bd88f4569176dbc72 [^]

fixes issue 14973: DalWebServiceServlet does not report errors which occur at commit time

---
M src-test/org/openbravo/test/webservice/WSUpdateTest.java
M src/org/openbravo/service/rest/DalWebService.java
---
(0032124)
iperdomo   
2010-10-22 17:31   
Tested on pi @ rev f4f7acfab40b
(0032134)
hudsonbot   
2010-10-22 22:43   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/594ab4625cfa [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/98214a9e7372 [^]
Tests: http://builds.openbravo.com/view/int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.18589.obx [^]