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.
As written in http://www.netbeans.org/www-nbdev/msg05953.html and related messages in the thread by Josef, it would be nice if ErrorManager could be used for reporting more user friendly messages than the currently used. Josef submitted patches. I applied the one for RenameAction, but I do not think that extending the constants in ErrorManager is necessary, I would rather preffer different behavior (title + icon) for ErrorManager.USER, ErrorManager.WARNING, etc. Also the patch for NbErrorManager is not correct, it should be moved to NotifyException class, otherwise could lead to deadlocks. Last but not least, it would be IMHO better to share same dialog as the exceptions are shown right now. But, of course with different title and buttons + USER exceptions would be have to be displayed first.
I corrected RenameAction.java and NbErrorManager.java as Jesse suggested. So now annotation with USER or WARNING severity with localized message are not printed to console and ide.log because these are user-friendly messages. The piece of code which must be called from AWT thread I moved to NotifyException.java. (see the thread http://www.netbeans.org/www-nbdev/msg05953.html ) So here are the patches...
Created attachment 1186 [details] patch for RenameAction.java
Created attachment 1187 [details] patch for NbErrorManager.java
Created attachment 1188 [details] patch for NotifyException.java
I've applied the Josef's latest patch, NotifyException.java rev. 1.22, but I still think that the UI should be improved to share only one dialog box for exceptions with all kinds of severity (WARNING, USER, EXCEPTION, ...) as described above.
Target milestone -> 3.3
I applied patch for RenameAction.java which I sent last time. It is small change, now only exceptions without USER or WARNING severity are printed to console. The same problem was in org/openide/explorer/view/TreeViewCellEditor.java, see issue #11991. Now it is consistent in both files.
Just a comment: Rosta, please keep in mind, that we would like to move the whole code of NbErrorManager to logger module, when Roger will be ready to take care of it. So the less core features used, the better...
Adjusting target milestone.
remove keyword :3.2.1_CANDIDATE and changed Target Milestone : 3.2.1 -> ---
Ok, patches already exist, while not perfect -> target set to 4.0. I'll reapply the patch, try to make suggested changes and pass to UI for review.
reassigne to Marek, new owner of ui subcomponents
Really old issue, patch is here, but nobody aplied it, why ?
Assigning to Tim
Re-assigning Tim's issues to Dafe.
Jarda, I think we tried to get rid of USER level exceptions and want to migrate to logging API anyway so I am closing this as WONTFIX. Is this correct?