You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by bo...@apache.org on 2009/01/15 11:12:48 UTC

svn commit: r734654 - /myfaces/tobago/branches/tobago-1.0.x/core/src/main/java/org/apache/myfaces/tobago/component/ComponentUtil.java

Author: bommel
Date: Thu Jan 15 02:12:47 2009
New Revision: 734654

URL: http://svn.apache.org/viewvc?rev=734654&view=rev
Log:
(TOBAGO-737) Bug in Facelets inline attribute

Modified:
    myfaces/tobago/branches/tobago-1.0.x/core/src/main/java/org/apache/myfaces/tobago/component/ComponentUtil.java

Modified: myfaces/tobago/branches/tobago-1.0.x/core/src/main/java/org/apache/myfaces/tobago/component/ComponentUtil.java
URL: http://svn.apache.org/viewvc/myfaces/tobago/branches/tobago-1.0.x/core/src/main/java/org/apache/myfaces/tobago/component/ComponentUtil.java?rev=734654&r1=734653&r2=734654&view=diff
==============================================================================
--- myfaces/tobago/branches/tobago-1.0.x/core/src/main/java/org/apache/myfaces/tobago/component/ComponentUtil.java (original)
+++ myfaces/tobago/branches/tobago-1.0.x/core/src/main/java/org/apache/myfaces/tobago/component/ComponentUtil.java Thu Jan 15 02:12:47 2009
@@ -351,9 +351,11 @@
     if (bool instanceof Boolean) {
       return (Boolean) bool;
     } else if (bool instanceof String) {
-      LOG.warn("Searching for a boolean, but find a String. Should not happen. "
-          + "attribute: '" + name + "' id: '" + component.getClientId(FacesContext.getCurrentInstance())
-          + "' comp: '" + component + "'");
+      if (LOG.isInfoEnabled()) {
+        LOG.info("Searching for a boolean, but find a String. Should not happen. "
+            + "attribute: '" + name + "' id: '" + component.getClientId(FacesContext.getCurrentInstance())
+            + "' comp: '" + component + "'");
+      }
       return Boolean.valueOf((String) bool);
     } else {
       LOG.warn("Unknown type '" + bool.getClass().getName()