Facets should not be sticky when users click "Search" user/dbs/no_sticky_facets
authorDan Scott <dscott@laurentian.ca>
Sat, 22 Dec 2012 02:15:49 +0000 (21:15 -0500)
committerDan Scott <dscott@laurentian.ca>
Sat, 22 Dec 2012 02:15:49 +0000 (21:15 -0500)
commitbc684ddc513250021e6c6a48d01af8ab2f1d0172
treeeb973f3acade7e1b6c40a27abb4b662537f9d5fc
parentbff532c317f2ae7bf5157159b0f732ac74483c26
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>
Open-ILS/src/templates/opac/results.tt2