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 at...@apache.org on 2008/10/22 03:03:34 UTC

svn commit: r706829 - /portals/pluto/branches/2.0-spi-refactoring/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java

Author: ate
Date: Tue Oct 21 18:03:34 2008
New Revision: 706829

URL: http://svn.apache.org/viewvc?rev=706829&view=rev
Log:
Fix for a *spec* required null argument check which got lost on a commit for PLUTO-489, r679515

Modified:
    portals/pluto/branches/2.0-spi-refactoring/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java

Modified: portals/pluto/branches/2.0-spi-refactoring/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java
URL: http://svn.apache.org/viewvc/portals/pluto/branches/2.0-spi-refactoring/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java?rev=706829&r1=706828&r2=706829&view=diff
==============================================================================
--- portals/pluto/branches/2.0-spi-refactoring/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java (original)
+++ portals/pluto/branches/2.0-spi-refactoring/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java Tue Oct 21 18:03:34 2008
@@ -498,6 +498,7 @@
     }
 
     public void setAttribute(String name, Object value) {
+        ArgumentUtility.validateNotNull("attributeName", name);
         final OptionalContainerServices optionalContainerServices = container.getOptionalContainerServices();
         final RequestAttributeService requestAttributeService = optionalContainerServices.getRequestAttributeService();
         requestAttributeService.setAttribute(this, this.getHttpServletRequest(), this.portletWindow, name, value);