You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by da...@apache.org on 2006/12/07 22:23:07 UTC

svn commit: r483670 - /geronimo/server/trunk/modules/geronimo-deploy-tool/src/main/java/org/apache/geronimo/deployment/cli/CommandListConfigurations.java

Author: dain
Date: Thu Dec  7 13:23:05 2006
New Revision: 483670

URL: http://svn.apache.org/viewvc?view=rev&rev=483670
Log:
GERONIMO-2549 NullPointerException when listing plugins on a bogus server.
Thanks for the patch Mark DeLaFranier

Modified:
    geronimo/server/trunk/modules/geronimo-deploy-tool/src/main/java/org/apache/geronimo/deployment/cli/CommandListConfigurations.java

Modified: geronimo/server/trunk/modules/geronimo-deploy-tool/src/main/java/org/apache/geronimo/deployment/cli/CommandListConfigurations.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-deploy-tool/src/main/java/org/apache/geronimo/deployment/cli/CommandListConfigurations.java?view=diff&rev=483670&r1=483669&r2=483670
==============================================================================
--- geronimo/server/trunk/modules/geronimo-deploy-tool/src/main/java/org/apache/geronimo/deployment/cli/CommandListConfigurations.java (original)
+++ geronimo/server/trunk/modules/geronimo-deploy-tool/src/main/java/org/apache/geronimo/deployment/cli/CommandListConfigurations.java Thu Dec  7 13:23:05 2006
@@ -79,6 +79,11 @@
                 } catch (FailedLoginException e) {
                     throw new DeploymentException("Invalid login for Maven repository '"+repo+"'");
                 }
+                if (data == null) {
+                    out.println();
+                    out.println("No plugins were returned from this site.");
+                    return;
+                }
                 Map categories = new HashMap();
                 List available = new ArrayList();
                 for (int i = 0; i < data.getPlugins().length; i++) {