Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 325159

Summary: [type hierarchy] Type hierarchy on package and type should gray out some types
Product: [Eclipse Project] JDT Reporter: Markus Keller <markus.kell.r>
Component: UIAssignee: Raksha Vasisht <raksha.vasisht>
Status: RESOLVED INVALID QA Contact:
Severity: minor    
Priority: P2 CC: daniel_megert, deepakazad
Version: 3.7   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Bug Depends on: 325336    
Bug Blocks:    
Attachments:
Description Flags
patch none

Description Markus Keller CLA 2010-09-13 16:02:54 EDT 
Follow-up to bug 21417

Type hierarchy on package and type should gray out some types. The rendering should be a merge of the hierarchy on m and the hierarchy on the type (types should be colored if they could be both).

package m;
public class A { }
package m;
public class B extends Exception {}

package m2;
public interface IThing { }
package m2;
public class Thing implements IThing { }


- open type hierarchy on m and IThing
=> Object, Throwable, and Exception should be gray

- open type hierarchy on m and Thing
=> Throwable and Exception should be gray
Comment 1 Dani Megert CLA 2010-09-14 02:20:45 EDT 
This should be fixed for M2 since this is a N&N item.
Comment 2 Raksha Vasisht CLA 2010-09-15 08:50:47 EDT 
Created attachment 178925 [details]
patch

The problem was in isInDifferentHierarchyScope(..) which completely has different meaning now with support for multi-selection. Now only the types which belong to the hierarchies of all input elements should be colored, rest should be grayed. Earlier fix colored types which were either input or was contained in one of the java elements which was not fully correct. Markus, could you pls review?
Comment 3 Raksha Vasisht CLA 2010-09-15 18:09:20 EDT 
Works for now with patch for bug 325336 comment #14. Will revisit if Open Type Hierarchy is enabled for combination of types + other selections.
Comment 4 Dani Megert CLA 2010-09-16 04:51:10 EDT 
This bug got invalidated by the fix for bug 325336.