You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@myfaces.apache.org by so...@apache.org on 2013/11/14 18:22:31 UTC

svn commit: r1541984 - /myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/context/PartialViewContextImpl.java

Author: sobryan
Date: Thu Nov 14 17:22:31 2013
New Revision: 1541984

URL: http://svn.apache.org/r1541984
Log:
TRINIDAD-2387 - FacesContext.getCurrentInstance().getPartialViewContext().getRenderIds() return immutable when empty

Made returned map mutable.

Modified:
    myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/context/PartialViewContextImpl.java

Modified: myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/context/PartialViewContextImpl.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/context/PartialViewContextImpl.java?rev=1541984&r1=1541983&r2=1541984&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/context/PartialViewContextImpl.java (original)
+++ myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/context/PartialViewContextImpl.java Thu Nov 14 17:22:31 2013
@@ -419,6 +419,7 @@ public class PartialViewContextImpl
   {
     String param =
       _context.getExternalContext().getRequestParameterMap().get(parameter);
+        
     if (param != null)
     {
       String ids[] = param.split("[ \t]+");
@@ -433,7 +434,7 @@ public class PartialViewContextImpl
       }
     }
 
-    return Collections.emptySet();
+    return new HashSet<String>();
   }
 
   private static final class ProcessPhaseCallback