Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 355302 - organize import deletes required base import
Summary: organize import deletes required base import
Status: VERIFIED FIXED
Alias: None
Product: Objectteams
Classification: Tools
Component: OTDT (show other bugs)
Version: 2.0   Edit
Hardware: Other Linux
: P3 normal (vote)
Target Milestone: 2.1 M5   Edit
Assignee: Stephan Herrmann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 356003
  Show dependency tree
 
Reported: 2011-08-20 17:42 EDT by Stephan Herrmann CLA
Modified: 2012-09-22 13:35 EDT (History)
0 users

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 Stephan Herrmann CLA 2011-08-20 17:42:33 EDT 
Inside the new team CheckUniqueCallinCapture, when invoking organize imports
one of three required base imports (JavaBuilder) is deleted.
Comment 1 Stephan Herrmann CLA 2011-12-25 06:31:32 EST 
I could not reproduce the removing of required imports,
but organize imports was indeed unable to *add* a 
missing import for JavaBuilder.

Analysis yields that this happened for all base classes of
roles that have the same name as their base class,
which wrongly caused the role binding to occur also in
the playedBy clause.

Fixed by nulling the wrongly resolved base class binding.
Now organize import recognizes this reference as 
unresolved and adds the required base import.

Released for 2.1 M5 via commit 563b3091966ce9871ab99feebba66c805a29c628
Comment 2 Stephan Herrmann CLA 2012-09-22 13:35:56 EDT 
Verified using build 2.2.1.201209182002