You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "Matthias Weßendorf (JIRA)" <de...@myfaces.apache.org> on 2007/07/19 16:38:06 UTC

[jira] Reopened: (MYFACES-1681) UIComponent.getAttributes() too restrictive

     [ https://issues.apache.org/jira/browse/MYFACES-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Matthias Weßendorf reopened MYFACES-1681:
-----------------------------------------


true for 1.1.x as well

> UIComponent.getAttributes() too restrictive
> -------------------------------------------
>
>                 Key: MYFACES-1681
>                 URL: https://issues.apache.org/jira/browse/MYFACES-1681
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: JSR-252
>    Affects Versions: 1.1.5,  1.2.0
>         Environment: SE 1.6.0_02
> GlassFish V2 admingui
>            Reporter: Ryan Lubke
>            Assignee: Matthias Weßendorf
>             Fix For: 1.2.1-SNAPSHOT
>
>         Attachments: MYFACES-1681.patch
>
>
> The Map returned by UIComponent.getAttributes() is too restrictive with respect
> to null values.
> MyFaces will always throw an NPE for a null value.  Granted, the following is
> in the javadocs:
>  * Any attempt to add a null key or value must throw a NullPointerException.
> However, the following is also in the same section:
>  * put() - If the property is writeable, call the setter method to set the corresponding value (unwrapping primitive values in their    
>    corresponding wrapper classes). If the property is not writeable, or an attempt is made to set a property of primitive type to null, throw 
>    IllegalArgumentException.
> Notice the comment about setting a primitive property to null.  This implies that a null value for this case
> is legal.  The RI will only throw the NPE against a null value *if* there isn't an associated property.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.