From: Bill Erickson <berick@esilibrary.com>
Date: Tue, 26 Jun 2012 19:36:50 +0000 (-0400)
Subject: kpac : repair Top link
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=7a0bd3f10422836bf37fd0d2bad56b91c2a3058a;p=contrib%2FConifer.git

kpac : repair Top link

Template Toolkit string.replace() does an inline replacement instead of
returning a value.  Use .replace() correctly when building the Top URL,
or the result will be an empty href, which direct to "here" in
Firefox/Chrome, but directs to "here, minus the page name in the URL"
(e.g. http://domain/eg/kpac/) in IE, which ultimately directs the user
to /eg/opac/login (since it doesn't match any configured paths).

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
---

diff --git a/Open-ILS/src/templates/kpac/parts/paginate.tt2 b/Open-ILS/src/templates/kpac/parts/paginate.tt2
index c87c9fa643..225b74ebba 100644
--- a/Open-ILS/src/templates/kpac/parts/paginate.tt2
+++ b/Open-ILS/src/templates/kpac/parts/paginate.tt2
@@ -43,8 +43,10 @@
         </td>
 
         <!-- Top of page -->
-        [% IF showtop; href = mmkurl('').replace('#.*', '') %]
-        <td class="to_top_btn"><a href="[% href %]"><img src="[% ctx.media_prefix %]/images/kpac/to_top_btn.png" /></a></td>
+        [% IF showtop %]
+            <td class="to_top_btn"><a href="[% mkurl('').replace('#.*', '') %]">
+                <img src="[% ctx.media_prefix %]/images/kpac/to_top_btn.png" />
+            </a></td>
         [% END %]
     </tr>
 </table>