From: Dan Scott Date: Wed, 17 Aug 2011 21:18:54 +0000 (-0400) Subject: TPAC: Strip the "page" GET param from author searches X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=16b9e665371dae76fde0db3e0b63387e89a895df;p=contrib%2FConifer.git TPAC: Strip the "page" GET param from author searches mkurl() generates a URL that, by default, retains all of the parameters that the currnt query string includes - useful for retaining library search scope, etc. However, searches for authors that are launched by clicking on the author's name in the search results page / record details were failing in many cases because the "page" parameter came along with it - and if there weren't five pages of results for page=5, for example, then no hits would be shown. This commit removes the page param from the URL. In other places, all of the params are stripped (see cnbrowse) but as I assume we want to retain search scope and potentially other parameters, we'll start with the smallest possible change. Signed-off-by: Dan Scott --- diff --git a/Open-ILS/src/templates/default/opac/parts/record/summary.tt2 b/Open-ILS/src/templates/default/opac/parts/record/summary.tt2 index d77c07fd15..38685edbb5 100644 --- a/Open-ILS/src/templates/default/opac/parts/record/summary.tt2 +++ b/Open-ILS/src/templates/default/opac/parts/record/summary.tt2 @@ -34,7 +34,7 @@ id='rdetail_author' href="[%- authorquery = attrs.author | replace('[,\.:;]', ''); - mkurl(ctx.opac_root _ '/results', {qtype => 'author', query => authorquery}) + mkurl(ctx.opac_root _ '/results', {qtype => 'author', query => authorquery}, ['page']) -%]">[% attrs.author | html %] [% END %] diff --git a/Open-ILS/src/templates/default/opac/parts/result/table.tt2 b/Open-ILS/src/templates/default/opac/parts/result/table.tt2 index a22308d7cc..9884045e6b 100644 --- a/Open-ILS/src/templates/default/opac/parts/result/table.tt2 +++ b/Open-ILS/src/templates/default/opac/parts/result/table.tt2 @@ -62,7 +62,7 @@ name='item_author' href="[%- authorquery = attrs.author | replace('[,\.:;]', ''); - mkurl(ctx.opac_root _ '/results', {qtype => 'author', query => authorquery}) + mkurl(ctx.opac_root _ '/results', {qtype => 'author', query => authorquery}, ['page']) -%]">[% attrs.author | html %]    [% attrs.pubdate | html %]