From d391a380ae637f23e326659a81c097bae4d2c34b Mon Sep 17 00:00:00 2001
From: Kathy Lussier <kmlussier@masslnc.org>
Date: Fri, 6 Jul 2012 17:01:51 -0400
Subject: [PATCH] When a user views holds in their account, clicking "only
 available" on the holds screen or "Ready for Pickup" in the navbar will add
 an "available=1" parameter. However, if the user then attempts to return to a
 list of all holds, the "available" parameter is not cleared, and the user
 will only see holds available for pickup.

This commit will clear those parameters.

Signed-off-by: Kathy Lussier <kmlussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
---
 Open-ILS/src/templates/opac/myopac/holds.tt2 | 2 +-
 Open-ILS/src/templates/opac/parts/topnav.tt2 | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Open-ILS/src/templates/opac/myopac/holds.tt2 b/Open-ILS/src/templates/opac/myopac/holds.tt2
index 5a5be5b92d..1a6f20954f 100644
--- a/Open-ILS/src/templates/opac/myopac/holds.tt2
+++ b/Open-ILS/src/templates/opac/myopac/holds.tt2
@@ -62,7 +62,7 @@
                 <td align="right">
                     [% l("Show") %] &nbsp; &nbsp;
                     [% IF CGI.param("available") %]
-                    <a href="[% mkurl('holds') %]">[% l('all') %]</a> |
+                    <a href="[% mkurl('holds', {}, ['available']) %]">[% l('all') %]</a> |
                     <strong>[% l("only available") %]</strong>
                     [% ELSE %]
                     <strong>[% l("all") %]</strong> |
diff --git a/Open-ILS/src/templates/opac/parts/topnav.tt2 b/Open-ILS/src/templates/opac/parts/topnav.tt2
index de9aeaaedd..beaccb894a 100644
--- a/Open-ILS/src/templates/opac/parts/topnav.tt2
+++ b/Open-ILS/src/templates/opac/parts/topnav.tt2
@@ -33,7 +33,7 @@
                 </span>
                 <span class="dash_divider">|</span>
                 <span class="dash-align">
-                    <a class="dash-link" href="[% mkurl(ctx.opac_root _ '/myopac/holds')
+                    <a class="dash-link" href="[% mkurl(ctx.opac_root _ '/myopac/holds', {}, ['available'])
                         %]"><span id="dash_holds">[% ctx.user_stats.holds.total
                         %]</span> [% l("On Hold") %]</a>
                 </span>
-- 
2.11.0