Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 342528

Summary: DynamicWizard - make error handler overridable
Product: [WebTools] WTP Webservices Reporter: Dimitar Georgiev <dimitar.georgiev>
Component: wst.wsAssignee: Keith Chong <keith.chong.ca>
Status: NEW --- QA Contact: Keith Chong <keith.chong.ca>
Severity: enhancement    
Priority: P3 CC: kaloyan
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Attachments:
Description Flags
Patch against head - make status handler overridable none

Description Dimitar Georgiev CLA 2011-04-12 05:27:52 EDT
Created attachment 193019 [details]
Patch against head - make status handler overridable

Hi Folks,

The DynamicWizard framework (org.eclipse.wst.command.env.ui) is very useful in sparing us from boilerplate code and reinventing the wheel. Therefore it is heavily utilized in one of our Eclipse-based products.

There is an error handler which is isolated from the wizard by means of a Java interface, but still it is hardcoded to EclipseErrorHandler in the DynamicWizard's init() method. However this implementation, as any concrete implementation, can prove unsuitable to some usecases.

Currently there seems to be no way to set the error handler to the extender's own implementation. Therefore I am proposing a small patch to make this possible.

I have set the severity of this bug to major, because currently we have to resort to reflection to work this limitation around. If you feel this is unreasonable, feel free to decrease.

Lastly, if the patch is accepted, I would like to ask if this could get in a maintenance build in the near future.

Thanks!

Best regards, Dimitar
Comment 1 Keith Chong CLA 2011-08-18 13:32:59 EDT
Hi Dimitar, 

Which maintenance build do you want it in?  3.3.1?