Changeset 4971

Timestamp:

2003-07-17T12:11:56+12:00 (21 years ago)

Author:

kjdon

Message:

only add services to the list if they have configured properly

File:

• trunk/gsdl3/src/java/org/greenstone/gsdl3/collection/ServiceCluster.java (modified) (1 diff)

trunk/gsdl3/src/java/org/greenstone/gsdl3/collection/ServiceCluster.java

@@ -267 +267 @@
         s.setMessageRouter(this.router);
         // pass the xml node to the service for configuration
-        s.configure(n, extra_info);
-        
-        // find out the supported service types for this service module
-        Node types = s.process(message);
-        NodeList typenodes = ((Element)types).getElementsByTagName(GSXML.SERVICE_ELEM);    
-        
-        for (int j=0; j<typenodes.getLength();j++) {
-            String service = ((Element) typenodes.item(j)).getAttribute(GSXML.NAME_ATT);        
-            this.service_map.put(service, s);
+        if (s.configure(n, extra_info)) {
             
-            // also add info to the ServiceInfo XML element
-            this.service_list.appendChild(this.doc.importNode(typenodes.item(j), true));
+            // find out the supported service types for this service module
+            Node types = s.process(message);
+            NodeList typenodes = ((Element)types).getElementsByTagName(GSXML.SERVICE_ELEM);    
+            
+            for (int j=0; j<typenodes.getLength();j++) {
+            String service = ((Element) typenodes.item(j)).getAttribute(GSXML.NAME_ATT);        
+            this.service_map.put(service, s);
+            
+            // also add info to the ServiceInfo XML element
+            this.service_list.appendChild(this.doc.importNode(typenodes.item(j), true));
+            }
         }
         } catch (Exception e) {