Summary: | Default sort preferences ignored on advanced search | ||
---|---|---|---|
Product: | Koha | Reporter: | Nicole C. Engard <nengard> |
Component: | Searching | Assignee: | Chris Cormack <chris> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | normal | ||
Priority: | P5 - low | CC: | chris, koha.sekjal |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 6656 : Fix for advanced search not respecting default sort order
my prefs [SIGNED-OFF] Bug 6656 : Fix for advanced search not respecting default sort order Bug 6656 : Fix for advanced search not respecting default sort order |
Description
Nicole C. Engard
2011-08-01 13:59:46 UTC
Hmm, for me it seems to be ignoring the default order everywhere, will investigate Created attachment 5199 [details] [review] Bug 6656 : Fix for advanced search not respecting default sort order Do I need to do something special to test this? I applied the patch, changed my sort to title and did searches in staff and opac - and it was always using relevance. I then changed to author and the same thing ... it stayed searching relevance. Don't want to fail this in case it's something to do with me forgetting a step. Nicole Did you make sure you changed the other pref to a-z (or z-a) as well. If you leave it title, dsc it wont work. Created attachment 5208 [details]
my prefs
This is what I did - and then I hit save. I then did a search and the sort on the top right in the opac and staff client both say relevance still. Have to head out to do training, but can put a more detailed summary together later.
Yes, ascending wont work (it never has) you need a to z, or z to a for title or author. Try changing that (note this patch does not touch that code at all, it just changes the advanced search, the default sort order has always worked only if you choose the right options, ascending/descending for numeric and a to z, z to a for alphabetical ones like title author) Okay - we need another bug to have asc and desc removed from the list of options if they don't work, but this patch is signed off. Created attachment 5209 [details] [review] [SIGNED-OFF] Bug 6656 : Fix for advanced search not respecting default sort order Template only change, easy to read, marking as Passed QA They do work, just not with title and author,they work with the numeric ones. Like total number of checkouts, date added etc. Patch pushed, please test Created attachment 5301 [details] [review] Bug 6656 : Fix for advanced search not respecting default sort order Rebase for 3.4.x Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> The fix for this bug was published in the 3.4.5 release. If you were the reporter of this bug, please take time to verify the fix and update the status of this bug report accordingly. If the bug is fixed to your satisfaction, please close this report. |