https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27565 Sarah Cornell <sbcornell@cityofportsmouth.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sbcornell@cityofportsmouth. | |com --- Comment #1 from Sarah Cornell <sbcornell@cityofportsmouth.com> --- I'm pretty sure these dropdowns used to be sorted alphabetically by description. I'm not sure when they switched, but they all appear to be sorted in the order in which they were added to the system. They really need to go back to alpha by description. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27565 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au --- Comment #3 from David Cook <dcook@prosentient.com.au> --- I can confirm, at the very least, that this is happening in master for Shelving Location on the staff interface's /cgi-bin/koha/catalogue/itemsearch.pl page. It appears to be sorting by the primary key instead of the description. Some relevant lines from the itemsearch.tt file: [% locations = AuthorisedValues.GetDescriptionsByKohaField({ kohafield = 'items.location' }) %] [% FOREACH loc IN locations %] [% loc.value = loc.authorised_value %] [% loc.label = loc.lib %] [% END %] Koha::Template::Plugin::AuthorisedValues leads to Koha::AuthorisedValues->get_descriptions_by_koha_field I'm seeing no sorting... But I also don't see any obvious changes to this code in years... By contrast, the Shelving Location is sorted on /cgi-bin/koha/cataloguing/additem.pl as it uses a different module. It uses C4::Koha::GetAuthorisedValues. Looking at other instances of Koha::AuthorisedValues->get_descriptions_by_koha_field, I see the method is typically used to build an unsorted hashmap, and then that is used to build a sorted array. I'm not sure why Matthias did it that way back in 2016, as it seems inefficient for no gain. Also potentially adds some syntactical issues... So there are 2 solutions: 1. Add sorting to Koha::AuthorisedValues 2. Add sorting to itemsearch.tt and itemsearch.pl (the data is fetched for different dropdowns in different places) The most efficient option would be to sort at the database level, so Koha::AuthorisedValues, but it's the heaviest touch, and would require the most testing. Changing itemsearch.pl would be easier although For what it's worth, I think that this has been an issue for at least 4 years. This is a bit too systemic to write a quick patch, so I'll leave this one for someone with more time. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27565 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |18433 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18433 [Bug 18433] Allow to select results to export in item search -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27565 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com, | |sally.healey@cheshireshared | |services.gov.uk -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27565 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |martin.renvoize@ptfs-europe |ity.org |.com Severity|enhancement |normal -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27565 --- Comment #8 from Fridolin Somers <fridolin.somers@biblibre.com> --- I see some places call get_descriptions_by_koha_field() with : { order_by => ['description'] } https://git.koha-community.org/Koha-community/Koha/src/commit/4a6dd5f82da1f4... Does this even work ? -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27565 --- Comment #9 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- (In reply to Fridolin Somers from comment #7)

We should add tests to t/db_dependent/AuthorisedValues.t no ?

Definitely.. I just wanted some feedback initially to see if the end result was what people were looking for.

And also impact search_by_marc_field() right ?

Yeah, this is pretty low level, its effect could be fairly widespread. (But hopefully in a positive way)

I dont see a call on search_by_koha_field() with category param. Obsolete ?

I also wondered that.. nice to see you noticed that too.. I'll investigate to see where it's hanging around from. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27565 --- Comment #11 from Fridolin Somers <fridolin.somers@biblibre.com> --- (In reply to Martin Renvoize from comment #10)

(In reply to Fridolin Somers from comment #8)

...

I see some places call get_descriptions_by_koha_field() with : { order_by => ['description'] }

https://git.koha-community.org/Koha-community/Koha/src/commit/ 4a6dd5f82da1f45456b953421e699abc179a6fb9/reports/catalogue_stats.pl#L120

Does this even work ?

Yeah, I really don't think that works.. Unless I'm missing something.. 'description' is the field name in the UI, not a field name in the database.

Also get_descriptions_by_koha_field() only uses the first given hash as $param, I think second hash is ignored. Maybe caused by replacement of search_by_koha_field() with get_descriptions_by_koha_field() : https://git.koha-community.org/Koha-community/Koha/commit/a16a75018063477cf1... -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27565 --- Comment #12 from Fridolin Somers <fridolin.somers@biblibre.com> --- Is this a duplicate of Bug 28554 ? -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27565 --- Comment #13 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- (In reply to Fridolin Somers from comment #12)

Is this a duplicate of Bug 28554 ?

Looks like it could be.. it's been a while since I wrote this patch though so would need to investigate a little deeper. -- You are receiving this mail because: You are watching all bug changes.