tpac: facet is maintained when new search is launched from search bar

Bug #1091659 reported by Kathy Lussier
34
This bug affects 7 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
2.2
Fix Released
Medium
Unassigned
2.3
Fix Released
Medium
Unassigned

Bug Description

Evergreen version: 2.2.3
Also tested in several other 2.2, 2.3 and master catalogs

After limiting a search by a facet, if the user launches a new search from the search bar at the top of the screen, the facet limiter is maintained.

To replicate this problem:

1. Do a catalog search, for example, "paulsen hatchet"
2. Click on one of the facets in the left sidebar, for example, "Paulsen, Gary"
3. Use the searchbox at the top of your list of hits to do a different search. The gary paulsen facet limiter is maintained in the URL, leading to an unsuccessful search.

Tags: tpac
Revision history for this message
Ben Shum (bshum) wrote :

Confirmed to occur in my testing as well.

Changed in evergreen:
status: New → Confirmed
importance: Undecided → Medium
milestone: none → 2.4.0-alpha
Revision history for this message
Simon Mai (simonmai) wrote :

This doesn't affect our libs now, since we are using Evergreen 2.2 that doesn't maintain the selected facets when you click on Search button. (In the search form, there are not hidden "facet" input)
You can check here -> http://campbelld.mnpals.net/eg/opac/home
But I think this affect others (using ver 2.3) and our libs if we update to use Evergreen 2.3 and so on, since you inserted a hidden facet input into search form to keep the selected facet values.
So now, I have a question. Why do you want to maintain the selected facets when click on Search button again?
Is that important?
Because if you don't have the hidden "facet" input, this bug will not happen.
I will ask our libs about this question after holiday vacation also.
Merry Xmas, eveyone.

Revision history for this message
Dan Scott (denials) wrote :

Merry Christmas - here's a branch that fixes this problem and the related bug 1092946 as well.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbs/no_sticky_facets

Please test and sign-off!

Revision history for this message
Ben Shum (bshum) wrote :

Tested and works well for me. Thanks Dan!

Picked to master, rel_2_3, and rel_2_2. I'll note the fix and targets on the related bug as well.

Changed in evergreen:
status: Confirmed → Fix Committed
Ben Shum (bshum)
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.