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

Timestamp:

2000-07-06T09:49:36+12:00 (24 years ago)

Author:

sjboddie

Message:

Receptionist now caches collection information to avoid making multiple
get_collectinfo calls to collection server

File:

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

trunk/gsdl/src/recpt/documentaction.cpp

@@ -28 +28 @@
 /*
    $Log$
+   Revision 1.39  2000/07/05 21:49:31  sjboddie
+   Receptionist now caches collection information to avoid making multiple
+   get_collectinfo calls to collection server
+
    Revision 1.38  2000/05/04 05:18:46  sjboddie
    attempting to get end-user collection building to work under windows
@@ -183 +187 @@
 
 documentaction::documentaction () {
+  recpt = NULL;
+
 
   // this action uses cgi variables "a", "d", "cl",
@@ -500 +506 @@
   if (collectproto == NULL) return;
   
+  if (recpt == NULL) {
+    logout << "ERROR (documentaction::define_external_macros): This action does not contain\n"
+       << "      information about any receptionists. The method set_receptionist was\n"
+       << "      probably not called from the module which instantiated this action.\n";
+    return;
+  }
+
   outconvertclass text_t2ascii;
   comerror_t err;
@@ -506 +519 @@
   text_tset metadata;
 
-
-  // get info on current collection and load up formatinfo
-  // I'd prefer not to do this here as we're getting 
-  // collection info every time (and probably also getting
-  // it in other places some of the time) - One day I'll
-  // fix it ... maybe - Stefan.
-  ColInfoResponse_t cinfo;
-  collectproto->get_collectinfo (collection, cinfo, err, logout);
-  load_formatinfo (cinfo.format, args.getintarg("gt"));
+  ColInfoResponse_t *cinfo = recpt->get_collectinfo_ptr (collectproto, collection, logout);
+  if (cinfo == NULL) {
+    logout << "ERROR (documentaction::define_external_macros): get_collectinfo_ptr returned NULL\n";
+    return;
+  }
+  load_formatinfo (cinfo->format, args.getintarg("gt"));
 
   if (formatinfo.DocumentUseHTML) {
@@ -526 +536 @@
       disp.setmacro ("gsdltop", "Global", "documenttop");
     }
-    text_tmap::iterator it = cinfo.format.find ("homepage");
-    if (it != cinfo.format.end()) {
+    text_tmap::iterator it = cinfo->format.find ("homepage");
+    if (it != cinfo->format.end()) {
       text_t httppagehome;
       if (get_link (args, protos, (*it).second, httppagehome, logout))
@@ -1101 +1111 @@
   textout << outconvert << disp << buffer;
 }
-
-
-
-
-
-
-
-
-
-
-