You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@juddi.apache.org by ks...@apache.org on 2010/12/13 22:39:46 UTC

svn commit: r1045383 - /juddi/trunk/juddi-core/src/main/java/org/apache/juddi/config/Install.java

Author: kstam
Date: Mon Dec 13 21:39:46 2010
New Revision: 1045383

URL: http://svn.apache.org/viewvc?rev=1045383&view=rev
Log:
JUDDI-437, fixing potential NPE if no custom config files are found.

Modified:
    juddi/trunk/juddi-core/src/main/java/org/apache/juddi/config/Install.java

Modified: juddi/trunk/juddi-core/src/main/java/org/apache/juddi/config/Install.java
URL: http://svn.apache.org/viewvc/juddi/trunk/juddi-core/src/main/java/org/apache/juddi/config/Install.java?rev=1045383&r1=1045382&r2=1045383&view=diff
==============================================================================
--- juddi/trunk/juddi-core/src/main/java/org/apache/juddi/config/Install.java (original)
+++ juddi/trunk/juddi-core/src/main/java/org/apache/juddi/config/Install.java Mon Dec 13 21:39:46 2010
@@ -539,17 +539,20 @@ public class Install {
 						en = new JarFile(new File(paths[0])).entries();
 					}
 				}
-			
-				log.debug("Discovering the Publisher XML data files in jar: " + paths[0]);
-				while (en.hasMoreElements()) {
-					String name = en.nextElement().getName();
-					if (name.endsWith(FILE_PUBLISHER)) {
-						log.debug("Found publisher file=" + name);
-						String publisher = name.substring(basePath.length(),name.indexOf(FILE_PUBLISHER));
-				        if (! rootPublisherStr.equalsIgnoreCase(publisher)) {
-				        	publishers.add(publisher);
-				        } 
+				if (paths.length > 0) {
+					log.debug("Discovering the Publisher XML data files in jar: " + paths[0]);
+					while (en.hasMoreElements()) {
+						String name = en.nextElement().getName();
+						if (name.endsWith(FILE_PUBLISHER)) {
+							log.debug("Found publisher file=" + name);
+							String publisher = name.substring(basePath.length(),name.indexOf(FILE_PUBLISHER));
+					        if (! rootPublisherStr.equalsIgnoreCase(publisher)) {
+					        	publishers.add(publisher);
+					        } 
+						}
 					}
+				} else {
+					log.info("No custom configuration files where found in " + path);
 				}
 			} catch (IOException e) {
 				throw new ConfigurationException(e);



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@juddi.apache.org
For additional commands, e-mail: commits-help@juddi.apache.org