svn commit: r720368 - /myfaces/extensions/validator/trunk/core/src/main/java/org/apache/myfaces/extensions/validator/core/el/DefaultELHelper.java

[gp...@apache.org]

Author: gpetracek
Date: Mon Nov 24 16:42:11 2008
New Revision: 720368

URL: http://svn.apache.org/viewvc?rev=720368&view=rev
Log:
minor changes

Modified:
    myfaces/extensions/validator/trunk/core/src/main/java/org/apache/myfaces/extensions/validator/core/el/DefaultELHelper.java

Modified: myfaces/extensions/validator/trunk/core/src/main/java/org/apache/myfaces/extensions/validator/core/el/DefaultELHelper.java
URL: http://svn.apache.org/viewvc/myfaces/extensions/validator/trunk/core/src/main/java/org/apache/myfaces/extensions/validator/core/el/DefaultELHelper.java?rev=720368&r1=720367&r2=720368&view=diff
==============================================================================
--- myfaces/extensions/validator/trunk/core/src/main/java/org/apache/myfaces/extensions/validator/core/el/DefaultELHelper.java (original)
+++ myfaces/extensions/validator/trunk/core/src/main/java/org/apache/myfaces/extensions/validator/core/el/DefaultELHelper.java Mon Nov 24 16:42:11 2008
@@ -18,8 +18,6 @@
  */
 package org.apache.myfaces.extensions.validator.core.el;
 
-import org.apache.myfaces.extensions.validator.internal.ToDo;
-import org.apache.myfaces.extensions.validator.internal.Priority;
 import org.apache.myfaces.extensions.validator.internal.UsageInformation;
 import org.apache.myfaces.extensions.validator.internal.UsageCategory;
 import org.apache.myfaces.extensions.validator.util.ExtValUtils;
@@ -49,7 +47,6 @@
  * it's pluggable in order to support special mechanisms of different technologies (than jsp and facelets)
  * so you can plug in your own impl. which implements a custom workaround (like the facelets workaround of this impl.)
  */
-@ToDo(Priority.MEDIUM)
 @UsageInformation(UsageCategory.INTERNAL)
 public class DefaultELHelper implements ELHelper
 {
@@ -106,7 +103,11 @@
         //(e.g. for special component libs -> issue with ExtValRendererWrapper#encodeBegin)
         if(valueBindingExpression == null)
         {
-            //TODO logging
+            if(this.logger.isTraceEnabled())
+            {
+                this.logger.trace(
+                        uiComponent.getClass() + " has no value binding - component id: " + uiComponent.getId());
+            }
             return null;
         }
 
@@ -125,7 +126,7 @@
             {
                 if(logger.isWarnEnabled())
                 {
-                    logger.warn("couldn't resolve expression: " + result.getExpressionString());
+                    logger.warn("couldn't resolve expression: " + valueBindingExpression);
                 }
                 return null;
             }