[Koha-bugs] [Bug 16330] REST API: add routes to add, update and delete patrons

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Mon Dec 18 15:51:32 CET 2017


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

Jonathan Druart <jonathan.druart at bugs.koha-community.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Passed QA                   |In Discussion

--- Comment #78 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
Please double check there is something wrong: Koha::Patron->store does nothing,
and ->_validate (private) is called from outside the module.

I guess it should be called, and then the expections caught from (at least)
memberentry.

Same for Koha::Patron::Modification->store, I think the exception must be
caught in our code.

And, first of all, _check_branchcode and _check_categorycode sound wrong: we
have a FK on DB level and DBIx::Class will raise an error already if something
is missing. Are we going to create these kind of check method for all the FK we
have at DB level?

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


More information about the Koha-bugs mailing list