svn commit: r558088 - /cayenne/main/trunk/framework/cayenne-jdk1.4-unpublished/src/main/java/org/apache/cayenne/map/EntityResolver.java

[aa...@apache.org]

Author: aadamchik
Date: Fri Jul 20 11:42:37 2007
New Revision: 558088

URL: http://svn.apache.org/viewvc?view=rev&rev=558088
Log:
acting on TODO - adding logging for the map inconsistency

Modified:
    cayenne/main/trunk/framework/cayenne-jdk1.4-unpublished/src/main/java/org/apache/cayenne/map/EntityResolver.java

Modified: cayenne/main/trunk/framework/cayenne-jdk1.4-unpublished/src/main/java/org/apache/cayenne/map/EntityResolver.java
URL: http://svn.apache.org/viewvc/cayenne/main/trunk/framework/cayenne-jdk1.4-unpublished/src/main/java/org/apache/cayenne/map/EntityResolver.java?view=diff&rev=558088&r1=558087&r2=558088
==============================================================================
--- cayenne/main/trunk/framework/cayenne-jdk1.4-unpublished/src/main/java/org/apache/cayenne/map/EntityResolver.java (original)
+++ cayenne/main/trunk/framework/cayenne-jdk1.4-unpublished/src/main/java/org/apache/cayenne/map/EntityResolver.java Fri Jul 20 11:42:37 2007
@@ -40,6 +40,8 @@
 import org.apache.cayenne.reflect.valueholder.ValueHolderDescriptorFactory;
 import org.apache.cayenne.util.Util;
 import org.apache.commons.collections.collection.CompositeCollection;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 
 /**
  * Represents a virtual shared namespace for zero or more DataMaps. Unlike DataMap,
@@ -55,6 +57,8 @@
 public class EntityResolver implements MappingNamespace, Serializable {
 
     static final Object DUPLICATE_MARKER = new Object();
+    
+    protected static final Log logger = LogFactory.getLog(EntityResolver.class);
 
     protected boolean indexedByClass;
 
@@ -511,10 +515,8 @@
                             entityInheritanceCache.put(superOEName, superNode);
                         }
                         else {
-                            // bad mapping?
-                            // TODO (Andrus, 10/18/2005) it would be nice to log something
-                            // here, but since EntityResolver is used on the client, log4J
-                            // is a no-go...
+                            // bad mapping? Or most likely some classloader issue
+                            logger.warn("No super entity mapping for '" + superOEName + "'");
                             continue;
                         }
                     }