svn commit: r474571 - /db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/ClassDescriptor.java

Author: arminw
Date: Mon Nov 13 15:29:06 2006
New Revision: 474571

URL: http://svn.apache.org/viewvc?view=rev&rev=474571
Log:
add comment

Modified:
    db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/ClassDescriptor.java

Modified: db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/ClassDescriptor.java
URL: http://svn.apache.org/viewvc/db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/ClassDescriptor.java?view=diff&rev=474571&r1=474570&r2=474571
==============================================================================
--- db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/ClassDescriptor.java (original)
+++ db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/ClassDescriptor.java Mon Nov 13 15:29:06 2006
@@ -2185,6 +2185,11 @@
             /*
             arminw: only return no-PK fields, because all PK fields are declared
             in sub-class too.
+            TODO: add all fields with inherited fields, because it could be the case that
+            the field is overridden in sub-class but we have to populate the "real" field of the
+            class object not only all fields reflected by the last sub-class object.
+            Check for side-effects, e.g. if a collection-descriptor is mapped in each sub-class
+            or a collection-descriptor is overridden in a sub-class.
             */
             FieldDescriptor[] superFlds = getSuperClassDescriptor().getFieldDescriptorNonPk(true);
             if(m_FieldDescriptions == null)