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)



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to