https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #18 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Jonathan Druart from comment #17)
+ @sort1 = sort {$a cmp $b} uniq( Koha::Patrons->search($sort_filter)->get_column('sort1') );
This is terrible if you have lot of patrons. You should at least do the unique (distinct) at the DBMS level.
What about the following suggestion: you could display sort1 and sort2 authorised values in the dropdown list, or simply display a search input. You could also add it to the table's headers (filter at the top of the column).
sort1 can be free text or an AV. The AV category always exists, but maybe we could check for existing values. If values exist: show a drop down with the values. If there are no entries for the AV, use a free text input field. That would match the behavior on the patron edit form. -- You are receiving this mail because: You are watching all bug changes.