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.
|Summary:||Enhance ErrorDescription(Factory) with optional ID and more details text|
|Product:||editor||Reporter:||Jan Lahoda <jlahoda>|
|Component:||Hints & Annotations||Assignee:||Milutin Kristofic <mkristofic>|
|Issue Type:||ENHANCEMENT||Exception Reporter:|
|Attachments:||Proposed API change.|
Description Jan Lahoda 2012-03-12 08:43:28 UTC
I would like to enhance ErrorDescription with an optional ID and "more details" text.
Comment 1 Jan Lahoda 2012-03-12 11:25:33 UTC
Created attachment 116594 [details] Proposed API change.
Comment 2 Jan Lahoda 2012-03-12 11:26:12 UTC
Please review. Thanks.
Comment 3 Tomas Zezula 2012-03-13 16:44:17 UTC
The patch is OK.
Comment 4 Jesse Glick 2012-03-14 23:18:38 UTC
[JG01] The createErrorDescription overloads are becoming unreadably long. Consider using the builder pattern for optional args, e.g. ErrorDescription e = ErrorDescriptionFactory.make(WARNING, "Bad stuff!", doc, line).id("bad_stuff).details("There is bad stuff here.").fixes(oneFix, anotherFix);
Comment 5 Jan Lahoda 2012-03-15 22:44:37 UTC
JL01: I am thinking about converting the factory into a builder (possibly deprecating the whole ErrorDescriptionFactory and replacing it with ErrorDescriptionBuilder), but I would rather keep the change simple for now. If/when we move to to the builder pattern, deprecating three additional methods shouldn't be too difficult.
Comment 6 Jesse Glick 2012-03-20 03:08:59 UTC
JG01 - OK.
Comment 7 Martin Balin 2016-07-07 07:28:39 UTC
This old bug may not be relevant anymore. If you can still reproduce it in 8.2 development builds please reopen this issue. Thanks for your cooperation, NetBeans IDE 8.2 Release Boss