https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726

--- Comment #18 from Katrin Fischer <[email protected]> ---
(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.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to