From: Liam Whalen <whalen.ld@gmail.com>
Date: Wed, 28 Aug 2013 04:47:39 +0000 (-0700)
Subject: LP 1037171 Links within a record after Expert search fixed
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=refs%2Fheads%2Fuser%2Fldw%2Flp1037171;p=working%2FEvergreen.git

LP 1037171 Links within a record after Expert search fixed

After conducting an Expert search if you view a record from the
search results, the links to Subject and Author searches return
you to the Expert seach because they contain the cgi.param _special
and the cgi.param tag.  WWW/EGCatLoader/Search.pm looks for
these params and directs the search towards an Expert search if
they exist, which is stopping the Subject and Author searches from
being performed.

record/body.tt2 defines a stop_parms variable which is used by mkurl
when consturcting urls in a record display.  By adding all the
variables from the Expert search to stop_parms the Subject links are
created properly.

In order to get the author links to work properly the stop_parms
variable had to be used in the authors.tt2 file.

Signed-off-by: Liam Whalen <whalen.ld@gmail.com>
---

diff --git a/Open-ILS/src/templates/opac/parts/record/authors.tt2 b/Open-ILS/src/templates/opac/parts/record/authors.tt2
index 005b48a0f1..ecbd7943c4 100644
--- a/Open-ILS/src/templates/opac/parts/record/authors.tt2
+++ b/Open-ILS/src/templates/opac/parts/record/authors.tt2
@@ -62,7 +62,7 @@ BLOCK build_author_links;
                 term = term _ ' ' _ sf;
             END;
         END;
-        url = mkurl(ctx.opac_root _ '/results', {query => qterm, qtype => 'author'}, ['page', 'expand']);
+        url = mkurl(ctx.opac_root _ '/results', {query => qterm, qtype => 'author'}, stop_parms);
         author_type = (tlabel || label) | html;
         
         # schema.org changes
diff --git a/Open-ILS/src/templates/opac/parts/record/body.tt2 b/Open-ILS/src/templates/opac/parts/record/body.tt2
index 2e74d66b21..ebed9e6862 100644
--- a/Open-ILS/src/templates/opac/parts/record/body.tt2
+++ b/Open-ILS/src/templates/opac/parts/record/body.tt2
@@ -1,7 +1,7 @@
 [%-  attrs = {marc_xml => ctx.marc_xml};
     PROCESS "opac/parts/misc_util.tt2";
     PROCESS get_marc_attrs args=attrs;
-    stop_parms = ['expand','cnoffset','copy_offset','copy_limit'];
+    stop_parms = ['expand','cnoffset','copy_offset','copy_limit','tag','subfield','term','page','_special'];
     ctx.record_attrs = attrs; # capture for JS
 %]
 <div id='canvas_main' class='canvas' itemscope itemtype='[% args.schema.itemtype %]'>