Description Dimitar Georgiev 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