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 vk...@apache.org on 2009/01/05 17:17:33 UTC

svn commit: r731609 - /portals/pluto/trunk/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java

Author: vkumar
Date: Mon Jan  5 08:17:33 2009
New Revision: 731609

URL: http://svn.apache.org/viewvc?rev=731609&view=rev
Log:
Fixing isPortletModeAllowedByPortlet method, 
Only VIEW mode is mandatory, as per spec.

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

Modified: portals/pluto/trunk/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java
URL: http://svn.apache.org/viewvc/portals/pluto/trunk/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java?rev=731609&r1=731608&r2=731609&view=diff
==============================================================================
--- portals/pluto/trunk/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java (original)
+++ portals/pluto/trunk/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletRequestImpl.java Mon Jan  5 08:17:33 2009
@@ -788,7 +788,7 @@
     
     private boolean isPortletModeAllowedByPortlet(PortletMode mode) 
     {
-        if (isPortletModeMandatory(mode)) 
+        if(PortletMode.VIEW.equals(mode))
         {
             return true;
         }
@@ -826,12 +826,6 @@
         return false;
     }
 
-    private boolean isPortletModeMandatory(PortletMode mode) 
-    {
-        return PortletMode.VIEW.equals(mode) || PortletMode.EDIT.equals(mode) || PortletMode.HELP.equals(mode);
-    }
-
-
 // InternalRenderRequest Impl ----------------------------------------------
 
     public Cookie[] getCookieProperties()