You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pluto-scm@portals.apache.org by cd...@apache.org on 2007/07/06 13:59:01 UTC

svn commit: r553855 - /portals/pluto/branches/pluto-1.1.x/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java

Author: cdoremus
Date: Fri Jul  6 04:59:00 2007
New Revision: 553855

URL: http://svn.apache.org/viewvc?view=rev&rev=553855
Log:
Fix for PLUTO-369. The getAttributeNames() returned portlet request attribute names including java.* and javax.* ones.

Modified:
    portals/pluto/branches/pluto-1.1.x/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java

Modified: portals/pluto/branches/pluto-1.1.x/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java
URL: http://svn.apache.org/viewvc/portals/pluto/branches/pluto-1.1.x/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java?view=diff&rev=553855&r1=553854&r2=553855
==============================================================================
--- portals/pluto/branches/pluto-1.1.x/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java (original)
+++ portals/pluto/branches/pluto-1.1.x/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java Fri Jul  6 04:59:00 2007
@@ -417,8 +417,11 @@
 
         while (attributes.hasMoreElements()) {
             String attribute = (String) attributes.nextElement();
-
-            String portletAttribute = mapper.decode(
+            
+            //Fix for PLUTO-369
+            String portletAttribute = isNameReserved(attribute) ?
+            		attribute :
+            	mapper.decode(
                 internalPortletWindow.getId(), attribute);
 
             if (portletAttribute != null) { // it is in the portlet's namespace