[Koha-bugs] [Bug 13990] ILS-DI LookupPatron Requries ID Type

[bugzilla-daemon at bugs.koha-community.org]

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

M. Tompsett <mtompset at hotmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mtompset at hotmail.com

--- Comment #5 from M. Tompsett <mtompset at hotmail.com> ---
(In reply to Jon Knight from comment #2)
> This begs the question: should Koha be constraining the list of id_type's
> that can be passed into this routine from the client? Or is being able to
> search on any column in the borrowers table regarded as a useful feature
> (we've found "email" to be handy for sure).

I think searching on any valid field is handy. Though, some fields are more
useful than others. No error checking on valid fields is in this patch, but
that isn't the tight scope of this bug.

However, if you list it, you should search it. And this patch makes the list
and what is searched matched nicely. I think email was a reasonable addition to
the searchable list that is stated.

-- 
You are receiving this mail because:
You are watching all bug changes.