Changeset 4774 for trunk/gsdl/src/recpt/documentaction.cpp

Timestamp:

2003-06-24T12:28:40+12:00 (21 years ago)

Author:

sjboddie

Message:

No longer show search page (or search form on "about" page) if a collection
doesn't have at least one searchable index. Note that this change includes
the addition of an is_searchable() function to the protocol.

File:

• trunk/gsdl/src/recpt/documentaction.cpp (modified) (7 diffs)

trunk/gsdl/src/recpt/documentaction.cpp

@@ -246 +246 @@
 // set_widthtspace calculates how wide the spaces in the nav bar should
 // be and sets the appropriate macro
-void documentaction::set_spacemacro (displayclass &disp, FilterResponse_t &response) {
+void documentaction::set_spacemacro (displayclass &disp, FilterResponse_t &response,
+                     bool has_search_button) {
 
   text_t width;
@@ -258 +259 @@
   twidth = width.getint();
       
-  disp.expandstring ("query", "_searchwidth_", width);
-  iwidth += width.getint();
-  
-  
+  if (has_search_button) {
+    disp.expandstring ("query", "_searchwidth_", width);
+    iwidth += width.getint();
+  } else {
+    numc -= 1;
+  }
   
   while (dochere != docend) {
@@ -283 +286 @@
 // reponse contains 1 metadata field (Title)
 void documentaction::set_navbarmacros (displayclass &disp, FilterResponse_t &response, 
-                       cgiargsclass &args) {
+                       bool has_search_button, cgiargsclass &args) {
 
   text_t topparent;
@@ -295 +298 @@
 
   navigationbar += "<nobr>\n";
-  if (args["a"] == "q") {
-    navigationbar += "_icontabsearchgreen_";
-  } else {
-    navigationbar += "_imagesearch_";
-  }
-  
-  
-  if (numc == 0) navigationbar += "_imagespacer_";
-    
+  if (has_search_button) {
+    if (args["a"] == "q") {
+      navigationbar += "_icontabsearchgreen_";
+    } else {
+      navigationbar += "_imagesearch_";
+    }
+  }
+  
+  if (has_search_button || numc == 0) navigationbar += "_imagespacer_";
+  
+  bool first = true;
   while (dochere != docend) {
+
+    if (!first) navigationbar += "_imagespacer_";
+
     text_t title = (*dochere).metadata["Title"].values[0];
 
@@ -318 +326 @@
     // if we're inside a document all the classification buttons should be enabled
     if (arg_d.empty() && ((*dochere).OID == topparent)) {
-      if (unknown) navigationbar += "_imagespacer_&nbsp;" + title + "&nbsp;";
-      else navigationbar += "_imagespacer__icontab" + title + "green_";
+      if (unknown) navigationbar += "&nbsp;" + title + "&nbsp;";
+      else navigationbar += "_icontab" + title + "green_";
     } else {
 
       // set the _httpbrowseXXX_ macro for this classification
-      if (unknown) navigationbar += "_imagespacer_&nbsp;<a href=\"_httpdocument_&cl=" + 
+      if (unknown) navigationbar += "&nbsp;<a href=\"_httpdocument_&cl=" + 
              (*dochere).OID + "\">" + title + "</a>&nbsp;";
       else {
-        navigationbar += "_imagespacer__image" + title + "_";
+        navigationbar += "_image" + title + "_";
         disp.setmacro ("httpbrowse" + title, "Global", "_httpdocument_&cl=" + (*dochere).OID);
       }
     }
     dochere ++;
+    first = false;
   }
   navigationbar += "\n</nobr>\n";
@@ -431 +440 @@
     get_children ("", collection, metadata, getParents, collectproto, response, logout);
     
+    bool has_search_button = true;
+    collectproto->is_searchable(collection, has_search_button, err, logout);
+    if (err != noError) has_search_button = true;
+
     // calculate width of spacers and set _widthtspace_ macro
-    if (args.getintarg("v") == 0) set_spacemacro (disp, response);
+    if (args.getintarg("v") == 0) set_spacemacro (disp, response, has_search_button);
 
     // set _navigationbar_ macro
-    set_navbarmacros (disp, response, args);
+    set_navbarmacros (disp, response, has_search_button, args);
    
       }
@@ -443 +456 @@
          << get_comerror_string (err);
     }
-  
-  
   }
   text_tmap::iterator usability = cinfo.format.find("Usability");