[CAY-2693] Abstract subentities do not have rows materialized correctly

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 4.1
• Fix Version/s: 4.2.M3
• Component/s: Database integration
• Labels: query-lifecycle query_generator
• Flags: Patch

Docs Text

I haven't forked Cayenne or made a pull request. I just found and hashed this out yesterday afternoon. I have monkey-patched it via the attached file and seems to work. The recursion termination condition in both qualifierForEntityAndSubclasses and entityMatchingRow includes entity's declaredQualifier being null. An abstract subentity can have a null declared qualifier and still have implied qualifiers defined by concrete subentities.

• Assignee: Nikita Timofeev
• Reporter: John Larson
• Created: 17/Dec/20 14:21
• Updated: 12/Mar/21 06:59
• Resolved: 12/Mar/21 06:59

Description

Given an inheritance tree such as (qualifiers are in parentheses):

abstract class A

concrete class B extends A (type = 0)

abstract class C extends A

concrete class D extends C (type = 1)

concrete class E extends C (type=2)

and single-table inheritance.

When querying on class A and results include a row of type D, EntityInheritanceTree will return C as the matching entity. In fact, even if the results include a row of type B, if entityMatchingRow will return C if it is checked before entity B.

Also, qualifierForEntityAndSubclasses will return a null qualifier for type C when it should really return (type =1) or (type = 2).