Changeset 3569 for trunk/gsdl3/src/java

Timestamp:

2002-11-26T13:58:26+13:00 (22 years ago)

Author:

kjdon

Message:

now handles multiple requests in the same message properly

File:

• trunk/gsdl3/src/java/org/greenstone/gsdl3/core/MessageRouter.java (modified) (2 diffs)

trunk/gsdl3/src/java/org/greenstone/gsdl3/core/MessageRouter.java

@@ -205 +205 @@
 
     // for now, just process each request one by one, and append the results to mainResult
-    for (int i=0; i<requests.getLength(); i++) {
+    // Note: if you add an element to another node in the same document, it
+    // gets removed from where it was. This changes the node list - you cant iterate over the node list in a normal manner if you are moving elements out of it
+    int num_requests = requests.getLength();
+    for (int i=0; i< num_requests; i++) {
         Node result=null;
-        Element req = (Element)requests.item(i);
+        Element req = (Element)requests.item(0);
         String path = req.getAttribute(GSXML.TO_ATT); // returns "" if no att of this name
         if (path.equals("")) {
@@ -224 +227 @@
             // append the contents of the message to the mainResult - there will only be one response at this stage
             mainResult.appendChild(doc_.importNode(GSXML.getChildByTagName(result, GSXML.RESPONSE_ELEM), true));
+            } else {
+            System.err.println("MessageRouter Error: request had null result!");
             }
         } else {    
-            System.err.println("MessageRouter: request has illegal module name in:\n"+converter_.getString(req));
+            System.err.println("MessageRouter Error: request has illegal module name in:\n"+converter_.getString(req));
         }
         }
         
     } // for each request
-    System.out.println("MR returning result");
-    System.out.println(converter_.getString(mainResult));
 
     return mainResult;