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 cz...@apache.org on 2009/03/30 17:19:32 UTC

svn commit: r760004 - /portals/pluto/trunk/pluto-container/src/main/java/org/apache/pluto/container/driver/impl/PortletContextManager.java

Author: cziegeler
Date: Mon Mar 30 15:19:31 2009
New Revision: 760004

URL: http://svn.apache.org/viewvc?rev=760004&view=rev
Log:
Throw exception if application is null instead of printing out "hello"

Modified:
    portals/pluto/trunk/pluto-container/src/main/java/org/apache/pluto/container/driver/impl/PortletContextManager.java

Modified: portals/pluto/trunk/pluto-container/src/main/java/org/apache/pluto/container/driver/impl/PortletContextManager.java
URL: http://svn.apache.org/viewvc/portals/pluto/trunk/pluto-container/src/main/java/org/apache/pluto/container/driver/impl/PortletContextManager.java?rev=760004&r1=760003&r2=760004&view=diff
==============================================================================
--- portals/pluto/trunk/pluto-container/src/main/java/org/apache/pluto/container/driver/impl/PortletContextManager.java (original)
+++ portals/pluto/trunk/pluto-container/src/main/java/org/apache/pluto/container/driver/impl/PortletContextManager.java Mon Mar 30 15:19:31 2009
@@ -132,9 +132,8 @@
             classLoaders.put(portletApp.getName(), Thread.currentThread().getContextClassLoader());
             for (PortletDefinition portlet: portletApp.getPortlets()) {
                 String appName = portletContext.getApplicationName();
-                if (appName == null)
-                {
-                    System.out.println("HELLLO");
+                if (appName == null) {
+                    throw new PortletContainerException("Portlet application name should not be null.");
                 }
                 portletConfigs.put(
                     portletContext.getApplicationName() + "/" + portlet.getPortletName(),