svn commit: r375263 - /myfaces/core/trunk/api/src/main/java/javax/faces/component/_ComponentAttributesMap.java

[mm...@apache.org]

Author: mmarinschek
Date: Mon Feb  6 04:11:31 2006
New Revision: 375263

URL: http://svn.apache.org/viewcvs?rev=375263&view=rev
Log:
value == null should be allowed, http://issues.apache.org/jira/browse/MYFACES-458

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

Modified: myfaces/core/trunk/api/src/main/java/javax/faces/component/_ComponentAttributesMap.java
URL: http://svn.apache.org/viewcvs/myfaces/core/trunk/api/src/main/java/javax/faces/component/_ComponentAttributesMap.java?rev=375263&r1=375262&r2=375263&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 Feb  6 04:11:31 2006
@@ -298,7 +298,7 @@
      */
     public Object put(Object key, Object value)
     {
-        checkKeyAndValue(key, value);
+        checkKeyAndValue(key);
 
         PropertyDescriptor propertyDescriptor = getPropertyDescriptor((String)key);
         if (propertyDescriptor != null)
@@ -420,9 +420,12 @@
     }
 
 
-    private void checkKeyAndValue(Object key, Object value)
+    private void checkKeyAndValue(Object key)
     {
-        if (value == null) throw new NullPointerException("value");
+        //http://issues.apache.org/jira/browse/MYFACES-458: obviously, the spec is a little unclear here,
+        // but value == null should be allowed - if there is a TCK-test failing due to this, we should
+        // apply for getting the TCK-test dropped
+        // if (value == null) throw new NullPointerException("value");
         checkKey(key);
     }