Changeset 14946

Timestamp:

2008-01-24T10:04:41+13:00 (16 years ago)

Author:

shaoqun

Message:

try HitsPerPage and StartPage in service params first to work with does_paging param

File:

• greenstone3/trunk/src/java/org/greenstone/gsdl3/action/QueryAction.java (modified) (3 diffs)

greenstone3/trunk/src/java/org/greenstone/gsdl3/action/QueryAction.java

@@ -107 +107 @@
     mr_query_message.appendChild(format_request);
 
+    logger.info(GSXML.xmlNodeToString(mr_query_message));
+
     // do the query
         Element mr_query_response = (Element)this.mr.process(mr_query_message);
@@ -207 +209 @@
     // check the hits_per_page param - is it a service param??
     String hits_pp = (String) service_params.get("hitsPerPage");
-    if (hits_pp != null) {
+    //if (hits_pp != null) {
         // the service is doing the paging, so we want to display all of the returned docs
-        return (Element)this.doc.importNode(orig_doc_list, true);
-    }
-    
-    hits_pp = (String)params.get("hitsPerPage");
+    //    return (Element)this.doc.importNode(orig_doc_list, true);
+    //}
+    
+    //String hits_pp = (String)params.get("hitsPerPage");
     int hits = 20;
     if (hits_pp != null && !hits_pp.equals("")) {
@@ -235 +237 @@
     // now we need our own doc list
     Element result_list = this.doc.createElement(GSXML.DOC_NODE_ELEM + GSXML.LIST_MODIFIER);
-    String start_p = (String)params.get("startPage");
+    
+    String start_p = (String) service_params.get("startPage");
+    if(start_p == null){
+        start_p = (String)params.get("startPage");
+    }
+
     int start = 1;
     if (start_p != null && !start_p.equals("")) {