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 Henrik Heimbuerger

Owner: changed from somebody to Fabian Rohn
Status: newassigned

comment:2 Changed 16 years ago by Henrik Heimbuerger

TrackDesigner.handleDeletion() is a good candidate for the first usage.

comment:3 Changed 16 years ago by Fabian Rohn

Status: assignedaccepted

comment:4 Changed 16 years ago by SVN

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 SVN

New commit by hheimbuerger (revision [161]):
[Partial fix for ticket #38] Added basic error handling code for exceptions thrown by the event dispatcher thread.

Just a hack in multiple ways, please see the ticket comments for more information.

comment:6 Changed 16 years ago by Henrik Heimbuerger

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 Henrik Heimbuerger

Owner: changed from Fabian Rohn to Henrik Heimbuerger

comment:8 Changed 16 years ago by Henrik Heimbuerger

Owner: changed from Henrik Heimbuerger to nobody
Status: acceptedassigned

comment:9 Changed 16 years ago by Florian Roth

Owner: changed from nobody to Florian Roth
Status: assignedaccepted

comment:10 Changed 16 years ago by SVN

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 Florian Roth

Resolution: fixed
Status: acceptedclosed
Note: See TracTickets for help on using tickets.