Bug 35484 - OPACShelfBrowser secondary sort should be biblio.title not item.itemnumber
Summary: OPACShelfBrowser secondary sort should be biblio.title not item.itemnumber
Status: Failed QA
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Lucas Gass (lukeg)
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-12-04 23:07 UTC by Lucas Gass (lukeg)
Modified: 2024-04-12 08:18 UTC (History)
1 user (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 35484: Make secondary sort use biblio.title (3.04 KB, patch)
2024-03-04 16:02 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Lucas Gass (lukeg) 2023-12-04 23:07:48 UTC
The OPACShelfBrowser sorts on cn_sort but also includes a secondary sort on itemnumber. This secondary sort is arbitrary and unhelpful, libraries typically don't arrange things on the shelf based on Koha's itemnumber. 

This secondary sort should be based on title to better refelct how libraries organize shelves. 

To recreate:
1. In k-t-d, set all item itemcallnumbers to the same thing, ie: 'abcdefg';
2. Touch all items to update the cn_sort values 
3. Turn on OPACShelfBrowser and view a record detail page on the OPAC.
4. Browse the shelf and see that all items are organized by itemnumber.
Comment 1 Lucas Gass (lukeg) 2024-03-04 16:02:21 UTC
Created attachment 162717 [details] [review]
Bug 35484: Make secondary sort use biblio.title

To test:
1. In k-t-d, set all item itemcallnumbers to the same thing, ie: 'abcdefg';
2. Touch all items to update the cn_sort values
3. Turn on OPACShelfBrowser and view a record detail page on the OPAC.
4. Browse the shelf and see that all items are organized by itemnumber.
5. APPLY PATCH and restart_services
6. Try again, now the secondary sort should be using biblio.title, not itemnumber.
Comment 2 Philip Orr 2024-04-12 08:18:20 UTC
Hmm, not working for me. After following the test plan, I'm
looking at the OPAC record detail page for "Ruby cookbook" (I'm on ktd).
The next items (to the right) after "Ruby cookbook" start with a P.
(Here: "Pro Javascript design patterns")
The items to the left ("Previous") do seem to be alphabetically sorted now.

Clicking on "Next:" to load the next few items in the shelfbrowser
also only loads items to the right of the original item.
The items to the left (so the "Previous") all stay the same.
Clicking on "Previous" does load all new items.