Bug 40433

Summary: Missing maxLength in item, patron and library
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: REST APIAssignee: Jonathan Druart <jonathan.druart>
Status: RESOLVED FIXED QA Contact: Martin Renvoize (ashimema) <martin.renvoize>
Severity: normal    
Priority: P5 - low CC: david, fridolin.somers, martin.renvoize, tomascohen
Version: unspecified   
Hardware: All   
OS: All   
GIT URL: Initiative type: ---
Sponsorship status: --- Comma delimited list of Sponsors:
Crowdfunding goal: 0 Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
25.11.00,25.05.03
Circulation function:
Bug Depends on: 40178    
Bug Blocks: 40434, 40430    
Attachments: Bug 40433: Add missing maxLength to item, library and patron
Bug 40433: Add missing maxLength to item, library and patron
Bug 40433: Add missing maxLength to item, library and patron

Description Jonathan Druart 2025-07-17 12:30:56 UTC
The test added in xt/api.t was not fully accurate, we skipped the columns that have the same name as the api attribute (ie. the ones not part of to_api_mapping
Comment 1 Jonathan Druart 2025-07-17 12:32:04 UTC
Created attachment 184192 [details] [review]
Bug 40433: Add missing maxLength to item, library and patron

The test added in xt/api.t was not fully accurate, we skipped the columns that have the same name as the api attribute (ie. the ones not part of to_api_mapping)

Test plan:
prove xt/api.t should pass
Comment 2 Owen Leonard 2025-07-17 16:48:35 UTC
Created attachment 184236 [details] [review]
Bug 40433: Add missing maxLength to item, library and patron

The test added in xt/api.t was not fully accurate, we skipped the
columns that have the same name as the api attribute (ie. the ones not
part of to_api_mapping)

Test plan:
prove xt/api.t should pass

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Martin Renvoize (ashimema) 2025-07-18 11:29:03 UTC
Created attachment 184349 [details] [review]
Bug 40433: Add missing maxLength to item, library and patron

The test added in xt/api.t was not fully accurate, we skipped the
columns that have the same name as the api attribute (ie. the ones not
part of to_api_mapping)

Test plan:
prove xt/api.t should pass

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 4 Lucas Gass (lukeg) 2025-07-21 20:04:03 UTC
Nice work everyone!

Pushed to main for 25.11
Comment 5 Paul Derscheid 2025-08-22 08:29:57 UTC
Nice work everyone!

Pushed to 25.05.x
Comment 6 Fridolin Somers 2025-09-17 13:39:53 UTC
Not pushed to 24.11.x
Comment 7 David Nind 2025-10-25 01:01:18 UTC
Bug fix, and also related to tests. No changes to the manual required.