Opened 16 years ago
Closed 16 years ago
#38 closed enhancement (fixed)
Create a generic error handling method
Reported by: | Henrik Heimbuerger | Owned by: | Florian Roth |
---|---|---|---|
Priority: | critical | Milestone: | MS1: First release (1.0.0) |
Component: | unspecified | Version: | pre-1.0 |
Keywords: | error handling, dialog | Cc: |
Description
Implement a method that can be used to report an error to the user. The error should be logged and the user should be shown some kind of dialog.
I guess de.jacavi.rcp.util would be the right package for the corresponding class. OpenTrackFileAction's handleException() method could probably be used as a template.
Change History (11)
comment:1 Changed 16 years ago by
Owner: | changed from somebody to Fabian Rohn |
---|---|
Status: | new → assigned |
comment:2 Changed 16 years ago by
comment:3 Changed 16 years ago by
Status: | assigned → accepted |
---|
comment:4 Changed 16 years ago by
New commit by frohn (revision [147]):
[Fix for ticket #38] An initial ExceptionHandler? added
For further information see javadoc.
comment:5 Changed 16 years ago by
comment:6 Changed 16 years ago by
Exceptions thrown during event handling are now reported. The eventLoopException() method of ApplicationWorkbenchAdvisor has been overridden to do that. However, some further inspection of the RCP documentation indicates that the correct way would be to override getWorkbenchErrorHandler() in the same class. Further research is necessary.
Also, an additional method has been added to the ExceptionHandler because most of the data required by the existing method is not available here. That method is really just a hack and should be removed.
Generally, the handleException() method's argument list should be simplified. A user should for example not be required to know what IStatus and Status are.
comment:7 Changed 16 years ago by
Owner: | changed from Fabian Rohn to Henrik Heimbuerger |
---|
comment:8 Changed 16 years ago by
Owner: | changed from Henrik Heimbuerger to nobody |
---|---|
Status: | accepted → assigned |
comment:9 Changed 16 years ago by
Owner: | changed from nobody to Florian Roth |
---|---|
Status: | assigned → accepted |
comment:10 Changed 16 years ago by
New commit by froth (revision [327]):
[Related to ticket #38] added an easy to use handleException method to the ErrorHandler? class
Runtime exceptions will not use this error handler therefore see the ApplicationWorkbench?.
comment:11 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
TrackDesigner.handleDeletion() is a good candidate for the first usage.