The $_GET param for the sort should be part of the facet links. Probably applies to D6 as well.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | apachesolr_270604_retain_sort.patch | 2.18 KB | pahariwalla |
The $_GET param for the sort should be part of the facet links. Probably applies to D6 as well.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | apachesolr_270604_retain_sort.patch | 2.18 KB | pahariwalla |
Comments
Comment #1
janusman commentedI think I already fixed this a month ago in #258626: Add "your current search" block! Right now handles solrsort and gids (Organic Groups) arguments... probably could/should autodetect any others and attach them to the links...
Changed functions in the patch:
New functions:
Comment #2
robertdouglass commentedThis actually merits more thought. It would be easiy enough to put the sorts in the facet links. However, how do you unsort? If we had an explicit "score" sort this would allow you to go to the default sort (score), but otherwise you'd get into sort mode and never get out of it again.
Comment #3
janusman commentedIt's not just going to be sorts: for example the OG support (in testing) also depends on extra query arguments... who knows what else will also use them.
Removing or adding a facet should not, IMO, destroy sort, as the user is not explicitly unsorting: he/she is clicking on a "add" or "remove facet" link, and is expecting just that =) A similar case is the Autofilter widget in Excel; filtering/unfiltering doesn't re-sort all your data...
I also think there is no "unsorted" listing in a Solr search, rather it is default-sorted by relevancy...
Hmm, I'd vote for the module to provide this:
Just my .02 =)
Comment #4
robertdouglass commentedUsability bug that I'd like to see fixed for 1.0 release.
Comment #5
pahariwalla commentedI no nothing about organic groups, but this simple-dimple patch (1 new line and 3 changed lines) at least retains the sort when clicking (or unclicking) on a facet.
I added a sortby option "Relevancy" so the user can get out of sort mode...
Comment #6
pahariwalla commentedComment #7
robertdouglass commentedThis patch looks good to me. Looking for another reviewer to RTBC and/or commit.
Comment #8
JacobSingh commentedGood work!
Fixed in Drupal-5 and Drupal-6.
Comment #9
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.