Changeset 37480 for main/trunk/greenstone3/web/interfaces/default/js/document_scripts.js

Timestamp:

2023-03-13T15:06:58+13:00 (14 months ago)

Author:

davidb

Message:

Changes to get Greenbug back up and running. Theis commit targets two areas: we now need to explicitly specify 'o=xml' in our calls to general action with 'sa' as things like 'XSLTGetTemplateListFromFile'; we now also need to set the AJAX calls explicitly so they return string-type as their response (even if fundamentally the message is XML for example), which is done with dataType: 'text' in the jquery-based AJAX call. We are now using jquery v3.x and it looks like the newer versions of the library auto-sense the return time, where as the older jquery we were using did not to this, hence the reason for the change. In the long run it would be better to keep the returned response in XML, and use browser API to access XML to get to the data we want, however the decision for now was to explicitly set things so it is handled as a string, thereby allowing all our currently written code to operate as it used to.

File:

• main/trunk/greenstone3/web/interfaces/default/js/document_scripts.js (modified) (7 diffs)

main/trunk/greenstone3/web/interfaces/default/js/document_scripts.js

@@ -68 +68 @@
     if (gs.cgiParams.ck && gs.cgiParams.ck.length > 0) {
     url += "&ck=" + gs.cgiParams.ck;
-    }
-    
-    
-    
-    $.ajax(url)
+    }       
+    
+    $.ajax(url, { dataType: "text" })
     .done(function(response)
     {
@@ -109 +107 @@
     template += '<xsl:template match="/">';
     template +=   '<sections>';
-    template +=     '<xsl:for-each select="/page/pageResponse/document//documentNode[@nodeID = \'' + sectionID + '\']/documentNode">';
-    template +=       '<xsl:call-template name="wrapDocumentNodes"/>';
-    template +=     '</xsl:for-each>';
+    template += '<xsl:for-each select="/page/pageResponse/document//documentNode[@nodeID = \'' + sectionID + '\']/documentNode">';
+    template +=   '<xsl:call-template name="wrapDocumentNodes"/>';
+    template += '</xsl:for-each>';
     template +=   '</sections>';
     template += '</xsl:template>';
 
-    template = makeURLComponentSafe(template);
-    var url = gs.xsltParams.library_name + "/collection/" + gs.cgiParams.c + "/document/" + sectionID + "?ilt=" + template;
-
-    if (gs.cgiParams.p_s && gs.cgiParams.p_s.length > 0) {
-    url += "&p.s=" + gs.cgiParams.p_s;
-    }
-
-       if(gs.documentMetadata.docType == "paged")
+    template = makeURLComponentSafe(template);
+    var url = gs.xsltParams.library_name + "/collection/" + gs.cgiParams.c + "/document/" + sectionID + "?ilt=" + template;
+
+    if (gs.cgiParams.p_s && gs.cgiParams.p_s.length > 0) {
+        url += "&p.s=" + gs.cgiParams.p_s;
+    }
+
+    if(gs.documentMetadata.docType == "paged")
     {
         url += "&dt=hierarchy";
     }
     
-    $.ajax(url)
+        $.ajax(url, { dataType: "text" })
     .done(function(response)
     {
@@ -461 +459 @@
     }
 
-    $.ajax(url)
+    $.ajax(url, { dataType: "text" })
     .done(function(response)
     {
@@ -537 +535 @@
     }
     else {  
-    $.ajax(url)
+    $.ajax(url, { dataType: "text" })
         .done(retrieveFullTableOfContentsSuccess)
         .fail(function() {
@@ -704 +702 @@
     }
 
-    $.ajax(url)
+    $.ajax(url, { dataType: "text" })
     .done(function(response)
     {
@@ -968 +966 @@
         template +=   '</html>';
         template += '</xsl:template>';
-    template = makeURLComponentSafe(template);
+        template = makeURLComponentSafe(template);
         var url = href + "?noText=1&ilt=" + template;
 
-        $.ajax(url)
+    $.ajax(url, { dataType: "text" })
         .done(function(text)
              {
@@ -1307 +1305 @@
     template +=   ']</images>';
     template += '</xsl:template>';
-    template = makeURLComponentSafe(template);
+    template = makeURLComponentSafe(template);
     var url = gs.xsltParams.library_name + "/collection/" + gs.cgiParams.c + "/document/" + gs.cgiParams.d + "?ed=1&ilt=" + template;
 
-    $.ajax({
-        url:url
-    }).done(function(data) {
+    $.ajax(url, {dataType: "text"})
+    .done(function(data) {
         var startIndex = data.indexOf(">", data.indexOf(">") + 1) + 1;
         var endIndex = data.lastIndexOf("<");