[jboss-cvs] JBossAS SVN: r76144 - projects/jboss-mdr/trunk/src/main/org/jboss/metadata/plugins/loader/reflection.

[jboss-cvs-commits at lists.jboss.org]

Author: adrian at jboss.org
Date: 2008-07-23 14:16:59 -0400 (Wed, 23 Jul 2008)
New Revision: 76144

Modified:
   projects/jboss-mdr/trunk/src/main/org/jboss/metadata/plugins/loader/reflection/AnnotatedElementMetaDataLoader.java
Log:
[JBMDR-29] - Add some logging for stuff not found

Modified: projects/jboss-mdr/trunk/src/main/org/jboss/metadata/plugins/loader/reflection/AnnotatedElementMetaDataLoader.java
===================================================================
--- projects/jboss-mdr/trunk/src/main/org/jboss/metadata/plugins/loader/reflection/AnnotatedElementMetaDataLoader.java	2008-07-23 18:02:02 UTC (rev 76143)
+++ projects/jboss-mdr/trunk/src/main/org/jboss/metadata/plugins/loader/reflection/AnnotatedElementMetaDataLoader.java	2008-07-23 18:16:59 UTC (rev 76144)
@@ -28,6 +28,7 @@
 import java.lang.reflect.Member;
 import java.lang.reflect.Method;
 
+import org.jboss.logging.Logger;
 import org.jboss.metadata.plugins.loader.BasicMetaDataLoader;
 import org.jboss.metadata.plugins.loader.SimpleMetaDataLoader;
 import org.jboss.metadata.spi.retrieval.AnnotationItem;
@@ -56,6 +57,9 @@
  */
 public class AnnotatedElementMetaDataLoader extends BasicMetaDataLoader 
 {
+   /** The log */
+   private static final Logger log = Logger.getLogger(AnnotatedElementMetaDataLoader.class);
+   
    /** The annotated element */
    private AnnotatedElement annotated;
    
@@ -127,6 +131,12 @@
             Constructor<?> constructor = constructorSignature.getConstructor();
             if (constructor == null)
                constructor = ReflectionUtils.findConstructor(clazz, signature.getParametersTypes(clazz));
+            if (constructor == null)
+            {
+               if (log.isTraceEnabled())
+                  log.trace("Constructor with signature " + signature + " does not exist on class " + clazz.getName());
+               return null;
+            }
             return new AnnotatedElementMetaDataLoader(constructor);
          }
          else if (signature instanceof MethodSignature)
@@ -135,6 +145,12 @@
             Method method = methodSignature.getMethod();
             if (method == null)
                method = ReflectionUtils.findMethod(clazz, signature.getName(), signature.getParametersTypes(clazz));
+            if (method == null)
+            {
+               if (log.isTraceEnabled())
+                  log.trace("Method with signature " + signature + " does not exist on class " + clazz.getName());
+               return null;
+            }
             return new AnnotatedElementMetaDataLoader(method);
          }
          else if (signature instanceof MethodParametersSignature)
@@ -143,6 +159,12 @@
             Method method = methodParametersSignature.getMethod();
             if (method == null)
                method = ReflectionUtils.findMethod(clazz, signature.getName(), signature.getParametersTypes(clazz));
+            if (method == null)
+            {
+               if (log.isTraceEnabled())
+                  log.trace("Method with signature " + signature + " does not exist on class " + clazz.getName());
+               return null;
+            }
             Annotation[][] paramAnnotations = method.getParameterAnnotations();
             return new SimpleMetaDataLoader(paramAnnotations[methodParametersSignature.getParam()]);
          }
@@ -152,6 +174,12 @@
             Constructor<?> constructor = constructorParametersSignature.getConstructor();
             if (constructor == null)
                constructor = ReflectionUtils.findConstructor(clazz, signature.getParametersTypes(clazz));
+            if (constructor == null)
+            {
+               if (log.isTraceEnabled())
+                  log.trace("Constructor with signature " + signature + " does not exist on class " + clazz.getName());
+               return null;
+            }
             Annotation[][] paramAnnotations = constructor.getParameterAnnotations();
             return new SimpleMetaDataLoader(paramAnnotations[constructorParametersSignature.getParam()]);
          }
@@ -161,7 +189,13 @@
             Field field = fieldSignature.getField();
             if (field == null)
                field = ReflectionUtils.findField(clazz, signature.getName());
-            return field != null ? new AnnotatedElementMetaDataLoader(field) : null;
+            if (field == null)
+            {
+               if (log.isTraceEnabled())
+                  log.trace("Field " + signature.getName() + " does not exist on class " + clazz.getName());
+               return null;
+            }
+            return new AnnotatedElementMetaDataLoader(field);
          }
       }