Context Navigation

javascript-global-setup.xsl

Timestamp:

2019-10-03T18:56:15+13:00 (5 years ago)

Author:

ak19

Message:

Dr Bainbridge had the correct fix for solr dealing with phrase searching where clicking on a facet then made search results disappear instead of showing the results within that facet. The problem was that double quotes ended up as html entities for ampersanded-quote and therefore didn't get URL encoded for transfer. Fixed now in java-script-global-setup.xsl, where all values gs.cgiParams[key] are taken care of. An additional fix was needed in facet-scripts.js, where makeURLComponentSafe() needed to be applied to each value of gs.cgiParams[] that got used when generating the searchString, notably the s1.query param value where the earlier omission of this step revealed an obvious problem. The facet portion of the URL was already taken care of in previous bugfixes. 3. Added some important comments and links.

File:

-              r33040
+              r33544
             <xsl:for-each select="/page/pageRequest/paramList/param">
                 <xsl:text disable-output-escaping="yes">name = "</xsl:text><xsl:value-of select="@name"/><xsl:text disable-output-escaping="yes">";</xsl:text>
                 <xsl:text disable-output-escaping="yes">value = "</xsl:text><xsl:value-of select="util:escapeNewLinesAndQuotes(@value)"/><xsl:text disable-output-escaping="yes">";</xsl:text>
+                <xsl:text disable-output-escaping="yes">value = "</xsl:text><xsl:value-of disable-output-escaping="yes" select="util:escapeNewLinesAndQuotes(@value)"/><xsl:text disable-output-escaping="yes">";</xsl:text>
                 <xsl:text disable-output-escaping="yes">name = name.replace(".", "_");</xsl:text>
                 gs.cgiParams[name] = value;
+                gs.cgiParams[name] = value;
             </xsl:for-each>
         </script>

Note: See TracChangeset for help on using the changeset viewer.