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/23 10:38:34 UTC

svn commit: r1525536 - in /myfaces/core/trunk/api/src: main/java/javax/faces/component/_ComponentAttributesMap.java test/java/javax/faces/component/UIComponentAttributesTest.java

Author: lu4242
Date: Mon Sep 23 08:38:34 2013
New Revision: 1525536

URL: http://svn.apache.org/r1525536
Log:
MYFACES-3780 _ComponentAttributesMap return null when no property descriptor is available and null value is passed to map 

Modified:
    myfaces/core/trunk/api/src/main/java/javax/faces/component/_ComponentAttributesMap.java
    myfaces/core/trunk/api/src/test/java/javax/faces/component/UIComponentAttributesTest.java

Modified: myfaces/core/trunk/api/src/main/java/javax/faces/component/_ComponentAttributesMap.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/api/src/main/java/javax/faces/component/_ComponentAttributesMap.java?rev=1525536&r1=1525535&r2=1525536&view=diff
==============================================================================
--- myfaces/core/trunk/api/src/main/java/javax/faces/component/_ComponentAttributesMap.java (original)
+++ myfaces/core/trunk/api/src/main/java/javax/faces/component/_ComponentAttributesMap.java Mon Sep 23 08:38:34 2013
@@ -574,10 +574,10 @@ class _ComponentAttributesMap implements
         _PropertyDescriptorHolder propertyDescriptor = getPropertyDescriptor(key);
         if (propertyDescriptor == null)
         {
-            if (value == null)
-            {
-                throw new NullPointerException("value is null for a not available property: " + key);
-            }
+            //if (value == null)
+            //{
+            //    throw new NullPointerException("value is null for a not available property: " + key);
+            //}
         }
         else
         {

Modified: myfaces/core/trunk/api/src/test/java/javax/faces/component/UIComponentAttributesTest.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/api/src/test/java/javax/faces/component/UIComponentAttributesTest.java?rev=1525536&r1=1525535&r2=1525536&view=diff
==============================================================================
--- myfaces/core/trunk/api/src/test/java/javax/faces/component/UIComponentAttributesTest.java (original)
+++ myfaces/core/trunk/api/src/test/java/javax/faces/component/UIComponentAttributesTest.java Mon Sep 23 08:38:34 2013
@@ -51,7 +51,7 @@ public class UIComponentAttributesTest e
     {
         input.getAttributes().put("style", null);
     }
-
+/*
     public void testSetNullAttributeOnInvalidProperty()
     {
         try
@@ -63,5 +63,5 @@ public class UIComponentAttributesTest e
         {
             // expected
         }
-    }
+    }*/
 }
\ No newline at end of file