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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0037913
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Modules] DB Pool Logging Utilitiesmajoralways2018-02-14 12:562018-03-06 13:27
Reporterjoniturralde93View Statuspublic 
Assigned Toalostale 
PriorityurgentResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisionf327c49b3459
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Regression date
Regression introduced by commit
Regression level
Review Assigned Tocaristu
Regression introduced in release
Summary

0037913: Exception thrown by trigger is logged as UndeclaredThrowableException instead of real one

DescriptionWhen a trigger exception is thrown, the invoke method of StatementInvocationHandler throws an InvocationTargetException and later hibernate is throwing as UndeclaredThrowableException, which is not properly handled by Openbravo.
Steps To Reproduce1. Install and configure DB Pool Logging Utilities to log slow statements
2. With no modules in dev log in as System Admin
3. Open Sales Order window definition
4. Edit its description
5. Save
  -> GOT: Generic error message: "Saving failed"
     EXPECTED: Concrete error message as thrown by trigger: "Saving failed. Cannot update an object in a module not in development and without an active template"
Proposed SolutionThe invoke method of StatementInvocationHandler should catch these exceptions and throw the original ones.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0102505)
hgbot (developer)
2018-02-20 08:15

Repository: erp/mods/org.openbravo.util.db
Changeset: f327c49b345903ca9ed6c77c6675ba2a3ce79c90
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue Feb 20 08:11:26 2018 +0100
URL: http://code.openbravo.com/erp/mods/org.openbravo.util.db/rev/f327c49b345903ca9ed6c77c6675ba2a3ce79c90 [^]

fixed bug 37913: statement exceptions were wrapped by InvocationTargetException

  This caused the code in charge of dealing with them (ie. to show proper messages)
  not to be able to handle it.

  When one of this exceptions occurs, instead of re-trhowing the InvocationTargetException
  we now throw the exception occurred in the underlying method.

---
M src/org/openbravo/util/db/StatementInvocationHandler.java
---
(0102506)
alostale (manager)
2018-02-20 08:17

fix included in version 0.1.200
(0103027)
caristu (manager)
2018-03-06 13:27

Code reviewed + tested OK.

- Issue History
Date Modified Username Field Change
2018-02-14 12:56 joniturralde93 New Issue
2018-02-14 12:56 joniturralde93 Assigned To => platform
2018-02-14 12:56 joniturralde93 Resolution time => 1520377200
2018-02-14 15:03 Practics Issue Monitored: Practics
2018-02-20 08:05 alostale Steps to Reproduce Updated View Revisions
2018-02-20 08:05 alostale Assigned To platform => alostale
2018-02-20 08:05 alostale Review Assigned To => caristu
2018-02-20 08:15 hgbot Checkin
2018-02-20 08:15 hgbot Note Added: 0102505
2018-02-20 08:15 hgbot Status new => resolved
2018-02-20 08:15 hgbot Resolution open => fixed
2018-02-20 08:15 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/mods/org.openbravo.util.db/rev/f327c49b345903ca9ed6c77c6675ba2a3ce79c90 [^]
2018-02-20 08:17 alostale Note Added: 0102506
2018-03-06 13:27 caristu Note Added: 0103027
2018-03-06 13:27 caristu Status resolved => closed


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker