]> git.evergreen-ils.org Git - Evergreen.git/commit
Facets should not be sticky when users click "Search"
authorDan Scott <dscott@laurentian.ca>
Sat, 22 Dec 2012 02:15:49 +0000 (21:15 -0500)
committerBen Shum <bshum@biblio.org>
Sun, 23 Dec 2012 18:36:29 +0000 (13:36 -0500)
commitb389c8c8593230d3ad58a1184a33509685daa7aa
treef1b770230853b7b3852c4388f7e41e0eec40e487
parentc3125b355fc442689dc2f203bc535bf068d63283
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 <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>
Open-ILS/src/templates/opac/results.tt2