This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Property editors can register themselves on PropertyEnv instance and throw PropertyVetoException to prevent property dialog from closing when something is not entered correctly by the user. As a result, the dialog is not closed and the user gets a warning message box. To show the simple warning dialog box the PropertyEnv calls PropertyDialogManager.notify which calls Exceptions.printStackTrace which logs the exception with SEVERE log level which then comes to NbErrorManager log handler and by some magic the SEVERE level is changed to WARNING so the proper dialog is shown to the user in the end. The main problem I have with this is in JDeveloper where logging an exception does not lead to showing a nice warning dialog, SEVERE exception is handled as an IDE error and is given to the user in the Feedback Tool to report a bug. So all property editors that use this mechanism are broken. Note that the purpose of all this in the PropertyEnv context is just to show the simple message dialog. The PropertyVetoException is only a means by which the property editor tells PropertyEnv about the error message. The exception itself has no value and should not be logged. (Besides the warning dialog in NB it also gets printed to console and appears in the log file, which is really pointless.) So can we please make PropertyEnv.setState to just show the message box with the warning message from the PropertyVetoException? Thanks.
Created attachment 119267 [details] suggested patch The fix is easy. If you agree with it, I can integrate it.
I am not sure if the new method will work well when there is no localized message. Instead of calling getLocalizedMessage() you should probably call Exceptions.findLocalizedMessage. I'd prefer to keep current code when there is no localized message and use the new one when a special localized message is found. Feel free to go on.
Kept the original logging if there is no explicit message. http://hg.netbeans.org/jet-main/rev/4399256694cd
Integrated into 'main-golden', will be available in build *201205160400* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress) Changeset: http://hg.netbeans.org/main-golden/rev/4399256694cd User: Tomas Pavek <tpavek@netbeans.org> Log: #212255: simply show dialog with message from PropertyVetoException (if there is any message)