Bug 237638 - [Quick Fix] Include whole Target Platform into quick-fix for unknown classes
Summary: [Quick Fix] Include whole Target Platform into quick-fix for unknown classes
Status: CLOSED DUPLICATE of bug 381975
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.4   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: PDE-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 352523 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-06-18 10:57 EDT by Philipp Kursawe CLA
Modified: 2013-04-23 11:10 EDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Kursawe CLA 2008-06-18 10:57:25 EDT
When you type the name of an unknown class the ctrl-space quick-assist should provide a list of classes not only from the already imported packages/bundles but from the whole target platform (PDE) and add the package to the bundles imports if not already imported.

This would greatly help reduce clicks to first import a package to use classes from it. Furthermore the "The type <some class> cannot be resolved. It is indirectly referenced from required .class files" could also automatically add the needed import package to the bundle.
Comment 1 Chris Aniszczyk CLA 2008-06-18 11:04:36 EDT
We kind of have a quickfix that does this already Philipp:

http://archive.eclipse.org/eclipse/downloads/drops/S-3.4M2-200709210919/eclipse-news-M2.html

Look at the N&N for 3.4M2

 	To assist plug-in developers, extra validation was added to help with access restriction issues that could be solved by exporting a package. For example, say you have two plug-ins: a and b. If a attempts to use a class from b and b doesn't export the package containing the class, you'll receive a warning and applicable quickfix. 

Does this work for you?

Is it not working for you?
Comment 2 Philipp Kursawe CLA 2008-06-18 11:11:23 EDT
Thanks for the quick response.

This only partially helps my second request. The first one about adding packages from classes contained in packages of the tartget platform would still be nice to have, wouldn't it?
Comment 3 Chris Aniszczyk CLA 2008-06-18 12:26:31 EDT
Ok, fair enough, I see what you're trying to do. We can try to improve what we already have available by hooking into JDT problems.
Comment 4 Philipp Kursawe CLA 2008-06-18 12:35:35 EDT
Thanks! I think you should make this feature an optional one, cause it can cause a big hog on the memory consumption if you have a big target-platform. Thanks for considering!
Comment 5 Darin Wright CLA 2009-04-22 11:39:43 EDT
not planned for 3.5
Comment 6 Curtis Windatt CLA 2011-07-20 09:06:23 EDT
*** Bug 352523 has been marked as a duplicate of this bug. ***
Comment 7 Curtis Windatt CLA 2013-04-23 11:10:07 EDT
Marking as a duplicate of a more recent fix that adds import package/require bundle proposals for unknown classes.

*** This bug has been marked as a duplicate of bug 381975 ***