Changeset 29265

Timestamp:

2014-09-11T12:53:17+12:00 (10 years ago)

Author:

kjdon

Message:

check for null coll_config_xml before trying to modify it for debug, otherwise get null pointer exception. this will happen if you have a folder in collect which is not actually a collection

File:

• main/trunk/greenstone3/src/java/org/greenstone/gsdl3/collection/OAICollection.java (modified) (1 diff)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/collection/OAICollection.java

@@ -84 +84 @@
 
         Element coll_config_xml = loadCollConfigFile();
+        if (coll_config_xml == null) {
+          logger.error("Collection: couldn't configure collection: " + this.cluster_name + ", " + "Couldn't load collection config file");
+
+          return false;
+        }
+        Element build_config_xml = loadBuildConfigFile();
+        if (build_config_xml == null)
+        {
+          logger.error("Collection: couldn't configure collection: " + this.cluster_name + ", " + "Couldn't load build config file");
+
+          return false;
+        }
         GSXSLT.modifyCollectionConfigForDebug(coll_config_xml);
-        Element build_config_xml = loadBuildConfigFile();
-
-        if (coll_config_xml == null || build_config_xml == null)
-        {
-            return false;
-        }
 
         // process the metadata and display items and default library params