From 3a8506d549dcf817b077a068314b2ab44f0507e8 Mon Sep 17 00:00:00 2001 From: Leo Stoyanov Date: Mon, 23 Jun 2025 15:46:38 +0000 Subject: [PATCH] Bug 40220: Added Consderation of API Interface for Mandatory Extended Attributes Test Plan: 1. Create an extended borrower attribute with: mandatory = 1 (checked), opac_display = 1, and opac_editable = 0 (unchecked - not editable in OPAC). 2. Setup your Postman, or any other mock API software, to obtain an OAuth token (/api/v1/oauth/token) and to register a patron (/api/v1/patrons). 3. With the OAuth token, send a POST request to /api/v1/patrons with valid patron data and missing mandatory extended attribute data. 4. Observe the API response wtih the 400 error. Despite this attribute not being required in the OPAC, the API requires it. 5. Apply the patch and repeat step 3. Observe the API response with the patron data returend, indicating the patron has been created. --- Koha/Patron.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Koha/Patron.pm b/Koha/Patron.pm index dcfff7aeb0..06c8a86210 100644 --- a/Koha/Patron.pm +++ b/Koha/Patron.pm @@ -2403,7 +2403,7 @@ sub extended_attributes { 'borrower_attribute_types_branches.b_branchcode' => undef, }; - if ( $interface eq 'opac' ) { + if ( $interface eq 'opac' || $interface eq 'api' ) { $params->{opac_editable} = 1; } -- 2.39.5