[Bug 43807] New: BorrowerMandatoryField is not being respected in the API
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43807 Bug ID: 43807 Summary: BorrowerMandatoryField is not being respected in the API Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: REST API Assignee: koha-bugs@lists.koha-community.org Reporter: matt.blenkinsop@openfifth.co.uk QA Contact: tomascohen@gmail.com CC: tomascohen@gmail.com Target Milestone: --- When creating a patron via the API, the BorrowerMandatoryField syspref is not being respected -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43807 Matt Blenkinsop <matt.blenkinsop@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |matt.blenkinsop@openfifth.c |ity.org |o.uk -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43807 Matt Blenkinsop <matt.blenkinsop@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43807 --- Comment #1 from Matt Blenkinsop <matt.blenkinsop@openfifth.co.uk> --- Created attachment 207981 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=207981&action=edit Bug 43807: Enforce BorrowerMandatoryField in the Koha::Patron->store method This commit adds an exception for a missing mandatory field and enforces it using a check_mandatory_fields option passed to the store method. In future this could be enforced without the option but there are a large number of instances, especially in unit tests that would need updating. Test plan: 1) Set a selection of fields in BorrowerMandatoryField 2) Create a Patron via the API - it should succeed even with some of those fields blank 3) Apply patches and restart_all 4) Re-try the request - an exception should be thrown Assisted-by: Sonnet 5 (Anthropic) -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org