[gatein-commits] gatein SVN: r4397 - in exo/portal/branches/3.1.x: examples/portal/war/src/main/webapp/WEB-INF and 1 other directories.

Author: ndkhoiits Date: 2010-09-27 05:37:56 -0400 (Mon, 27 Sep 2010) New Revision: 4397 Modified: exo/portal/branches/3.1.x/component/web/src/main/java/org/exoplatform/web/GenericHttpListener.java exo/portal/branches/3.1.x/examples/portal/war/src/main/webapp/WEB-INF/web.xml exo/portal/branches/3.1.x/web/portal/src/main/webapp/WEB-INF/web.xml Log: EXOGTN-36 Avoid NPE when a portal container has been disabled Modified: exo/portal/branches/3.1.x/component/web/src/main/java/org/exoplatform/web/GenericHttpListener.java =================================================================== --- exo/portal/branches/3.1.x/component/web/src/main/java/org/exoplatform/web/GenericHttpListener.java 2010-09-27 09:07:10 UTC (rev 4396) +++ exo/portal/branches/3.1.x/component/web/src/main/java/org/exoplatform/web/GenericHttpListener.java 2010-09-27 09:37:56 UTC (rev 4397) @@ -19,6 +19,7 @@ package org.exoplatform.web; +import org.exoplatform.commons.utils.PropertyManager; import org.exoplatform.container.ExoContainer; import org.exoplatform.container.ExoContainerContext; import org.exoplatform.container.PortalContainer; @@ -140,7 +141,19 @@ PortalContainer.setInstance(portalContainer); hasBeenSet = true; } - broadcast(portalContainer, CONTEXT_DESTROYED, event); + final String ctxName = event.getServletContext().getServletContextName(); + if (!PortalContainer.isPortalContainerNameDisabled(ctxName) && portalContainer instanceof PortalContainer) + { + // The portal container corresponding to the current servlet context could be found + broadcast(portalContainer, CONTEXT_DESTROYED, event); + } + else if (PropertyManager.isDevelopping()) + { + log.info("The portal environment could not be set for the webapp '" + ctxName + + "' because this servlet context has not been defined as a " + + "dependency of any portal container or it is a disabled portal" + + " container, the contextDestroyed event will be ignored"); + } } catch (Exception ex) { Modified: exo/portal/branches/3.1.x/examples/portal/war/src/main/webapp/WEB-INF/web.xml =================================================================== --- exo/portal/branches/3.1.x/examples/portal/war/src/main/webapp/WEB-INF/web.xml 2010-09-27 09:07:10 UTC (rev 4396) +++ exo/portal/branches/3.1.x/examples/portal/war/src/main/webapp/WEB-INF/web.xml 2010-09-27 09:37:56 UTC (rev 4397) @@ -73,7 +73,15 @@ <filter-name>CacheUserProfileFilter</filter-name> <filter-class>org.exoplatform.web.CacheUserProfileFilter</filter-class> </filter> + <filter> + <filter-name>PortalContainerFilter</filter-name> + <filter-class>org.exoplatform.container.web.PortalContainerFilter</filter-class> + </filter> + <filter-mapping> + <filter-name>PortalContainerFilter</filter-name> + <url-pattern>/*</url-pattern> + </filter-mapping> <filter-mapping> <filter-name>GenericFilter</filter-name> <url-pattern>/*</url-pattern> Modified: exo/portal/branches/3.1.x/web/portal/src/main/webapp/WEB-INF/web.xml =================================================================== --- exo/portal/branches/3.1.x/web/portal/src/main/webapp/WEB-INF/web.xml 2010-09-27 09:07:10 UTC (rev 4396) +++ exo/portal/branches/3.1.x/web/portal/src/main/webapp/WEB-INF/web.xml 2010-09-27 09:37:56 UTC (rev 4397) @@ -90,8 +90,16 @@ <filter-name>ClusteredSSOFilter</filter-name> <filter-class>org.exoplatform.web.login.ClusteredSSOFilter</filter-class> </filter> + <filter> + <filter-name>PortalContainerFilter</filter-name> + <filter-class>org.exoplatform.container.web.PortalContainerFilter</filter-class> + </filter> <filter-mapping> + <filter-name>PortalContainerFilter</filter-name> + <url-pattern>/*</url-pattern> + </filter-mapping> + <filter-mapping> <filter-name>RememberMeFilter</filter-name> <url-pattern>/public/*</url-pattern> </filter-mapping>