LP 1037171 Links within a record after Expert search fixed user/ldw/lp1037171
authorLiam Whalen <whalen.ld@gmail.com>
Wed, 28 Aug 2013 04:47:39 +0000 (21:47 -0700)
committerLiam Whalen <whalen.ld@gmail.com>
Wed, 28 Aug 2013 04:53:59 +0000 (21:53 -0700)
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>
Open-ILS/src/templates/opac/parts/record/authors.tt2
Open-ILS/src/templates/opac/parts/record/body.tt2

index 005b48a..ecbd794 100644 (file)
@@ -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
index 2e74d66..ebed9e6 100644 (file)
@@ -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 %]'>