Changeset 22297

Timestamp:

2010-06-23T14:50:10+12:00 (14 years ago)

Author:

ak19

Message:

1. Location of some Greenstone util classes had changed. 2. Removed duplication of variable value.

File:

• other-projects/gs3-webservices-java-client/trunk/src/GS3DemoClient/org/greenstone/gs3services/QBRSOAPServer.java (modified) (6 diffs)

other-projects/gs3-webservices-java-client/trunk/src/GS3DemoClient/org/greenstone/gs3services/QBRSOAPServer.java

@@ -38 +38 @@
 import org.w3c.dom.Node;
 
+import org.greenstone.util.GlobalProperties;
 import org.greenstone.gsdl3.core.MessageRouter;
-import org.greenstone.gsdl3.util.GlobalProperties;
 import org.greenstone.gsdl3.util.GSFile;
 import org.greenstone.gsdl3.util.GSXML;
@@ -130 +130 @@
     protected static final String serviceSubsetOptions = // services
         "paramList displayItemList";
-    protected static final String docStructureOptions =
+    protected static final String structureOptions =
         "entire ancestors parent siblings children descendants"; // note the spelling
     protected static final String structureInfoOptions =
         "numSiblings siblingPosition numChildren";
-    protected static final String browseStructureOptions = 
-        "ancestors parent siblings children descendants"; // note the spelling
     
         
@@ -481 +479 @@
         // Create the <paramlist></paramlist> element of param elements,
         // if any; and only if values are legal (that is, if they occur in  
-        // static Strings docStructureOptions and structureInfoOptions):
+        // static Strings structureOptions and structureInfoOptions):
         // <param name="structure" value = "structure[i]">
         // <param name="info" value = "info[i]">
@@ -489 +487 @@
         if(structure != null) {
             for(int i = 0; i < structure.length; i++) {
-                if(docStructureOptions.indexOf(structure[i]) != -1) {
+                if(structureOptions.indexOf(structure[i]) != -1) {
                     paramList.appendChild(GSXML.createParameter(
                             this.doc, "structure", structure[i]));                  
@@ -695 +693 @@
         for(int i = 0; i < structureParams.length; i++) {
             // check it is a valid structure parameter
-            if(browseStructureOptions.indexOf(structureParams[i]) != -1) {
+            if(structureOptions.indexOf(structureParams[i]) != -1) {
                 paramList.appendChild(GSXML.createParameter(
                         this.doc, "structure", structureParams[i]));
@@ -701 +699 @@
         }
 
-        if(infoParams != null) {
+        if(infoParams != null && !infoParams[0].equals("")) {
             for(int i = 0; i < infoParams.length; i++) {
             if(structureInfoOptions.indexOf(infoParams[i]) != -1) {
                 paramList.appendChild(GSXML.createParameter(
-                                    this.doc, "info", info[i]));
+                        this.doc, "info", info[i]));
             }
             }