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

--- Comment #7 from Alex Arnaud <[email protected]> ---
(In reply to Kyle M Hall from comment #6)
> Won't this be a breaking change for anyone currently using ILS-DI that
> doesn't have cardnumber and userid set to the same thing? Would it be better
> to try cardnumber and fall back to userid, and simply not allow either to be
> an empty field? That would make it behave like the rest of Koha's login
> functionality, while minimizing breakages.

Not sure to understand how it could be broken. For me this patch make sense
even though i didn't find a bug. We pass a userid to the function, so we
retrieve the patron with the userid. It is unique, and it has been
authenticated by checkpw just before.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://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