You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openjpa.apache.org by "Amy Yang (JIRA)" <ji...@apache.org> on 2008/04/26 08:17:57 UTC
[jira] Created: (OPENJPA-583)
MetaDataInheritanceComparator/InheritanceComparator need some improvement
MetaDataInheritanceComparator/InheritanceComparator need some improvement
-------------------------------------------------------------------------
Key: OPENJPA-583
URL: https://issues.apache.org/jira/browse/OPENJPA-583
Project: OpenJPA
Issue Type: Bug
Components: kernel
Affects Versions: 1.0.2, 1.0.3, 1.1.0, 1.2.0
Environment: Windows XP
java version "1.6.0_05"
Java(TM) SE Runtime Environment (build 1.6.0_05-b13)
Java HotSpot(TM) Client VM (build 10.0-b19, mixed mode)
Reporter: Amy Yang
Priority: Minor
Current MetaDataInheritanceComparator/InheritanceComparator has 2 problems:
1. the support of interfaces seems incorrect.
when calculating the level value of an interface, the logic is
if (to.isInterface())
return to.getInterfaces().length;
we should calculate the level value also based on the inheritance of the interfaces.
2. when comparing 2 objects (o1 and o2) with MetaDataInheritanceComparator, if o1 has a primary key field p1, then we should compare p1 and o2 first, and then compare o1 relative to the result.
current logic seems incomplete:
for (int i = 0; i < fmds.length; i++) {
if (fmds[i].isPrimaryKey() && m2.getDescribedType().
isAssignableFrom(fmds[i].getDeclaredType()))
return 1;
}
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.