[CAY-1438] refactoring EntityInheritanceTree lookups

• Type: Task
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.1M1
• Fix Version/s: 3.1M1
• Component/s: Core Library
• Assignee: Andrus Adamchik
• Reporter: Andrus Adamchik
• Created: 31/May/10 01:31
• Updated: 31/May/10 01:33
• Resolved: 31/May/10 01:33

Description

Related to CAY-1090 vertical inheritance work, need to make EntityInheritanceTree object easier to use. As a part of that will change EntityResolver.lookupInheritanceTree() logic that would return NULL if a class has no subclasses. Instead this method will return non-null instance for any existing entity while a check for subclasses will be performed by the callers.

After doing some local refactoring based on this change, I foudn that it actually saved some if/else code, and resulted in a cleaner inheritance handling in general. One possible further improvement will be to split EntityInheritanceTree functions into "pure metadata" and "behavior" and assign the later to the ClassDescriptor (in the same way we split ObjEntity and ClassDescriptor responsibilities).

Issue Links

• relates to: CAY-1090 Vertical inheritance