[ 
http://issues.apache.org/jira/browse/MYFACES-655?page=comments#action_12331255 
] 

Craig McClanahan commented on MYFACES-655:
------------------------------------------

As Oliver points out, the current MyFaces behavior is definitely spec 
compliant.  Implementation of the clear() and putAll() methods is *optional* in 
the java.util.Map contract.  However, there is a *huge* usability benefit in 
implementing these operations on the relevant maps (i.e. the attributes maps 
for request, session, and application scope).   Therefore, I've encouraged the 
JSF 1.2 expert group to make the implementation of these methods required in 
JSF 1.2, and I can report back that the EG has already considered the 
implementation of clear(), and will now consider the implementation of putAll() 
as well, to be required for a JSF 1.2 implementation.  If both of those come to 
pass, I would hope that both MyFaces and the java.net RI implementation of JSF 
1.1 would both also backport this into their current implementations.


> RequestMap doesn't implement putAll as it should
> ------------------------------------------------
>
>          Key: MYFACES-655
>          URL: http://issues.apache.org/jira/browse/MYFACES-655
>      Project: MyFaces
>         Type: Bug
>     Versions: 1.1.0
>     Reporter: Colin Sampaleanu
>     Assignee: Mathias Broekelmann

>
> The JSF description of for the ExternalContext.getRequestMap() method says:
> """Return a mutable Map representing the request scope attributes for the 
> current application. The returned Map  must implement the entire contract for 
> a modifiable map as described in the JavaDocs for java.util.Map. 
> Modifications made in the Map must cause the corresponding changes in the set 
> of request scope attributes."""
> yet MyFaces returns a RequestMap class which throws 
> UnsupportedOperationException for both the putAll() and the clear() methods.
> This is a major bug which will break apps (like mine) that depend on being 
> able to add params via putAll(), for example.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to