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

Posted to scm@geronimo.apache.org by ga...@apache.org on 2008/09/19 06:33:04 UTC

svn commit: r696919 - /geronimo/server/trunk/plugins/console/plugin-portlets/src/main/java/org/apache/geronimo/console/car/AssemblyListHandler.java

Author: gawor
Date: Thu Sep 18 21:33:03 2008
New Revision: 696919

URL: http://svn.apache.org/viewvc?rev=696919&view=rev
Log:
don't cache plugin or application module lists. Based on patch from Rex Wang (GERONIMO-4306)

Modified:
    geronimo/server/trunk/plugins/console/plugin-portlets/src/main/java/org/apache/geronimo/console/car/AssemblyListHandler.java

Modified: geronimo/server/trunk/plugins/console/plugin-portlets/src/main/java/org/apache/geronimo/console/car/AssemblyListHandler.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/console/plugin-portlets/src/main/java/org/apache/geronimo/console/car/AssemblyListHandler.java?rev=696919&r1=696918&r2=696919&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/console/plugin-portlets/src/main/java/org/apache/geronimo/console/car/AssemblyListHandler.java (original)
+++ geronimo/server/trunk/plugins/console/plugin-portlets/src/main/java/org/apache/geronimo/console/car/AssemblyListHandler.java Thu Sep 18 21:33:03 2008
@@ -110,8 +110,11 @@
         }
 
         listPlugins(request, pluginInstaller, data, appList);
-        request.getPortletSession(true).setAttribute(SERVER_CONFIG_LIST_SESSION_KEY, data);
-        request.getPortletSession(true).setAttribute(SERVER_APP_LIST_SESSION_KEY, appList);
+        
+        // don't cache plugin list or application module list - see GERONIMO-4306
+        //request.getPortletSession(true).setAttribute(SERVER_CONFIG_LIST_SESSION_KEY, data);
+        //request.getPortletSession(true).setAttribute(SERVER_APP_LIST_SESSION_KEY, appList);
+        
         return true;
     }