Bug 330674 - Organize imports shortcut doesn't work in aspect editor
Summary: Organize imports shortcut doesn't work in aspect editor
Status: RESOLVED FIXED
Alias: None
Product: AJDT
Classification: Tools
Component: Core (show other bugs)
Version: 2.1.1   Edit
Hardware: PC Windows Vista
: P3 normal (vote)
Target Milestone: 2.1.2   Edit
Assignee: AJDT-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-19 10:43 EST by Jon Steinich CLA
Modified: 2010-11-23 23:07 EST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jon Steinich CLA 2010-11-19 10:43:08 EST 
Build Identifier: 20100917-0705

When using the aspect editor (java or aj file), the keyboard shortcut for organizing imports doesn't work.  The menu option does work though.

The issue is worse when connected to https://bugs.eclipse.org/bugs/show_bug.cgi?id=295477

Here is an example:
public class ImportsTest {
	
	private Map<Long, String> val;

}

Reproducible: Always

Steps to Reproduce:
1.Create a new class
2.Open in aspect editor
3.Add a line that requires an import
4.Try the organize imports shortcut
Comment 1 Andrew Eisenberg CLA 2010-11-23 18:44:57 EST 
Thanks for the test case.  bug 295477 is not easy to fix, but I was able to fix this one.

Added a few regression tests.  Will be available in next dev build.
Comment 2 Andrew Eisenberg CLA 2010-11-23 19:39:48 EST 
Do you have a way to reliably reproduce bug 295477?
Comment 3 Jon Steinich CLA 2010-11-23 21:13:14 EST 
(In reply to comment #2)
> Do you have a way to reliably reproduce bug 295477?

I don't know of anything at the moment, but I will try to keep a closer watch of it.
Comment 4 Andrew Eisenberg CLA 2010-11-23 23:07:37 EST 
Thanks.  And attach anything you find to that bug.