You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lu...@apache.org on 2013/09/15 01:28:22 UTC

svn commit: r1523352 - in /myfaces/core/branches/2.0.x/impl/src/main/java/org/apache/myfaces/view/facelets: FaceletViewDeclarationLanguage.java tag/composite/CompositeComponentResourceTagHandler.java

Author: lu4242
Date: Sat Sep 14 23:28:22 2013
New Revision: 1523352

URL: http://svn.apache.org/r1523352
Log:
MYFACES-3748 Bug in check for required attribute at Composite Components (thanks to Dennis Hoersch for provide this patch)

Modified:
    myfaces/core/branches/2.0.x/impl/src/main/java/org/apache/myfaces/view/facelets/FaceletViewDeclarationLanguage.java
    myfaces/core/branches/2.0.x/impl/src/main/java/org/apache/myfaces/view/facelets/tag/composite/CompositeComponentResourceTagHandler.java

Modified: myfaces/core/branches/2.0.x/impl/src/main/java/org/apache/myfaces/view/facelets/FaceletViewDeclarationLanguage.java
URL: http://svn.apache.org/viewvc/myfaces/core/branches/2.0.x/impl/src/main/java/org/apache/myfaces/view/facelets/FaceletViewDeclarationLanguage.java?rev=1523352&r1=1523351&r2=1523352&view=diff
==============================================================================
--- myfaces/core/branches/2.0.x/impl/src/main/java/org/apache/myfaces/view/facelets/FaceletViewDeclarationLanguage.java (original)
+++ myfaces/core/branches/2.0.x/impl/src/main/java/org/apache/myfaces/view/facelets/FaceletViewDeclarationLanguage.java Sat Sep 14 23:28:22 2013
@@ -987,7 +987,7 @@ public class FaceletViewDeclarationLangu
                                 }
                                 else
                                 {
-                                    required = Boolean.getBoolean(requiredValue.toString());
+                                    required = Boolean.valueOf(requiredValue.toString());
                                 }
 
                                 if (required != null && required.booleanValue())

Modified: myfaces/core/branches/2.0.x/impl/src/main/java/org/apache/myfaces/view/facelets/tag/composite/CompositeComponentResourceTagHandler.java
URL: http://svn.apache.org/viewvc/myfaces/core/branches/2.0.x/impl/src/main/java/org/apache/myfaces/view/facelets/tag/composite/CompositeComponentResourceTagHandler.java?rev=1523352&r1=1523351&r2=1523352&view=diff
==============================================================================
--- myfaces/core/branches/2.0.x/impl/src/main/java/org/apache/myfaces/view/facelets/tag/composite/CompositeComponentResourceTagHandler.java (original)
+++ myfaces/core/branches/2.0.x/impl/src/main/java/org/apache/myfaces/view/facelets/tag/composite/CompositeComponentResourceTagHandler.java Sat Sep 14 23:28:22 2013
@@ -127,7 +127,7 @@ public class CompositeComponentResourceT
                     }
                     else
                     {
-                        required = Boolean.getBoolean(value.toString());
+                        required = Boolean.valueOf(value.toString());
                     } 
                     
                     if (required != null && required.booleanValue())