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.

Bug 50821 - Remove dependency of editor on projectapi
Summary: Remove dependency of editor on projectapi
Status: CLOSED FIXED
Alias: None
Product: editor
Classification: Unclassified
Component: -- Other -- (show other bugs)
Version: 4.x
Hardware: All All
: P3 blocker (vote)
Assignee: issues@editor
URL:
Keywords: ARCH
Depends on:
Blocks:
 
Reported: 2004-10-25 18:18 UTC by Jesse Glick
Modified: 2007-11-05 13:44 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jesse Glick 2004-10-25 18:18:05 UTC
Should be P2 but leaving as P3 temporarily to
avoid needing to waive for 4.0.

The editor module should not depend on the project
API. Cf. similar issue #50815.

If the editor module is refactored into smaller
pieces it would perhaps be OK to have some
GUI-only piece depend on projectapi if the
ultimate user goal is really to behave differently
depending on whether a CP root is owned by a
project or not.

But I think this is probably incorrect anyway;
this is probably a hack and should be solved
properly at a different level. The editor module
should not care that projects are used to specify
classpath information; other modules should be
free to have special ClassPathProvider's that are
fully on par with project implementations.
Comment 1 Roman Strobl 2005-08-15 13:31:11 UTC
Target milestone is obsolete, please re-evaluate. Thanks.
Comment 2 Miloslav Metelka 2005-10-05 10:44:22 UTC
Currently the editor/bookmarks uses saving of bookmarks into project.
In addition java/editor uses FileOwnerQuery.getOwner() to satisfy "Show
Non-project classes" in "Goto Class" dialog.
After the editor module split the editor's core (editor and editor/lib) does not
depend on projects API anymore.
Closing as fixed. Please reopen if additional action is needed.
Comment 3 Jesse Glick 2005-10-06 00:17:46 UTC
Sounds right. The usage of FOQ from Goto Class seems justified.