From 4665317984a3492efa992f9dba9fe287530c0c48 Mon Sep 17 00:00:00 2001 From: Dan Scott Date: Fri, 21 Dec 2012 21:15:49 -0500 Subject: [PATCH] Facets should not be sticky when users click "Search" We were using a hidden form variable to track which facets were in use, at the same time as relying on the GET params to actually display the facets, and munging the links on the remaining facets to properly add/subtract facets. The result of the hidden form variable was that: 1) clicking the Search button would unexpectedly result in the facet applying to the new search (wat) 2) applying two facets at once and then searching again would result in the facet vars being treated as a raw array. Thanks to Dan Pearl for inspiring this patch with his fix for the latter problem, and Kathy Lussier's recognition of the basic problem. Closes LP# 1092946 and 1091659. Signed-off-by: Dan Scott Signed-off-by: Ben Shum --- Open-ILS/src/templates/opac/results.tt2 | 3 --- 1 file changed, 3 deletions(-) diff --git a/Open-ILS/src/templates/opac/results.tt2 b/Open-ILS/src/templates/opac/results.tt2 index 976c5e9c36..875a8d2cea 100644 --- a/Open-ILS/src/templates/opac/results.tt2 +++ b/Open-ILS/src/templates/opac/results.tt2 @@ -15,9 +15,6 @@ page_count = ctx.page_size == 0 ? 1 : POSIX.ceil(ctx.hit_count / ctx.page_size); %]
- [% IF CGI.param('facet') %] - - [% END %]
[% INCLUDE "opac/parts/searchbar.tt2" took_care_of_form=1 %]
-- 2.11.0