Changeset 37007

Timestamp:

2022-12-09T22:49:53+13:00 (6 months ago)

Author:

davidb

Message:

Source might be a format such as TIFF that a browser cannot display, however Image can be controlled by the collectinConfig.xml file to be web friendly, hence the reason for the change

File:

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

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

@@ -727 +727 @@
     var liElems = tableOfContents.find("li").each(function()
     {
-        var section = $(this);
+            var section = $(this);
         var add = true;
         for(var j = 0; j < leafSections.length; j++)
@@ -949 +949 @@
         var endHREF = href.indexOf("'", startHREF);
         var nodeID = href.substring(startHREF, endHREF);
-        href = gs.xsltParams.library_name + "/collection/" + gs.cgiParams.c + "/document/" + nodeID;
+            href = gs.xsltParams.library_name + "/collection/" + gs.cgiParams.c + "/document/" + nodeID;
         
         var template = '';
@@ -1295 +1295 @@
     template += '<xsl:template match="/">';
     template +=   '<images>[';
-    template +=     '<xsl:for-each select="//documentNode">';
-    template +=       '<xsl:text disable-output-escaping="yes">{"source":"</xsl:text><gsf:metadata name="Source"/><xsl:text disable-output-escaping="yes">",</xsl:text>';
+        template +=     '<xsl:for-each select="//documentNode">';
+        // ****
+        // Used to be "Source" but now changed to "Image"
+        // This is because Source might not be a web friendly image format (e.g. TIFF original)
+        // Image givdes us a better chance at success, as least with Image, the collectionConfig.xml file can specify a format that is web friendly
+        // **** **** Is it worth changing the "source" and .source JS field to be "image" and .image??? **** ****
+    template +=       '<xsl:text disable-output-escaping="yes">{"source":"</xsl:text><gsf:metadata name="Image"/><xsl:text disable-output-escaping="yes">",</xsl:text>';
     template +=       '<xsl:text disable-output-escaping="yes">"id":"</xsl:text><xsl:value-of select="@nodeID"/><xsl:text disable-output-escaping="yes">"}</xsl:text>';
     template +=       '<xsl:if test="position() != count(//documentNode)">,</xsl:if>';