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:50:06 UTC
[jira] Resolved: (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 resolved MYFACES-1681.
-----------------------------------------
Resolution: Fixed
Fix Version/s: 1.1.6-SNAPSHOT
> 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.1.6-SNAPSHOT, 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.