Bug 40178 - Add maxLength to the patron definition
Summary: Add maxLength to the patron definition
Status: Needs documenting
Alias: None
Product: Koha
Classification: Unclassified
Component: REST API (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement
Assignee: Jonathan Druart
QA Contact: Matt Blenkinsop
URL:
Keywords:
Depends on: 40177
Blocks: 40175 40179 40337 40433
  Show dependency treegraph
 
Reported: 2025-06-18 21:19 UTC by Jonathan Druart
Modified: 2025-07-31 12:24 UTC (History)
3 users (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
25.11.00,25.05.02
Circulation function:


Attachments
Bug 40178: Adjust tests (651 bytes, patch)
2025-06-19 09:21 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 40178: Add maxLength to patron spec file (2.43 KB, patch)
2025-06-19 09:21 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 40178: Adjust tests (697 bytes, patch)
2025-07-03 09:29 UTC, David Nind
Details | Diff | Splinter Review
Bug 40178: Add maxLength to patron spec file (2.48 KB, patch)
2025-07-03 09:29 UTC, David Nind
Details | Diff | Splinter Review
Bug 40178: Adjust tests (764 bytes, patch)
2025-07-03 10:10 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 40178: Add maxLength to patron spec file (2.54 KB, patch)
2025-07-03 10:10 UTC, Matt Blenkinsop
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2025-06-18 21:19:08 UTC

    
Comment 1 Jonathan Druart 2025-06-19 09:21:10 UTC
Created attachment 183368 [details] [review]
Bug 40178: Adjust tests
Comment 2 Jonathan Druart 2025-06-19 09:21:12 UTC
Created attachment 183369 [details] [review]
Bug 40178: Add maxLength to patron spec file

Test plan:
Apply the "tests" patch first
  prove xt/api.t
=> fail
Apply this patch
=> pass
Comment 3 David Nind 2025-07-03 09:29:14 UTC
Created attachment 183730 [details] [review]
Bug 40178: Adjust tests

Signed-off-by: David Nind <david@davidnind.com>
Comment 4 David Nind 2025-07-03 09:29:17 UTC
Created attachment 183731 [details] [review]
Bug 40178: Add maxLength to patron spec file

Test plan:
Apply the "tests" patch first
  prove xt/api.t
=> fail
Apply this patch
=> pass

Signed-off-by: David Nind <david@davidnind.com>
Comment 5 Matt Blenkinsop 2025-07-03 10:10:02 UTC
Created attachment 183738 [details] [review]
Bug 40178: Adjust tests

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Matt Blenkinsop <matt.blenkinsop@openfifth.co.uk>
Comment 6 Matt Blenkinsop 2025-07-03 10:10:04 UTC
Created attachment 183739 [details] [review]
Bug 40178: Add maxLength to patron spec file

Test plan:
Apply the "tests" patch first
  prove xt/api.t
=> fail
Apply this patch
=> pass

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Matt Blenkinsop <matt.blenkinsop@openfifth.co.uk>
Comment 7 Lucas Gass (lukeg) 2025-07-08 21:45:35 UTC
Nice work everyone!

Pushed to main for 25.11
Comment 8 Paul Derscheid 2025-07-10 20:50:52 UTC
Nice work everyone!

Pushed to 25.05.x
Comment 9 Fridolin Somers 2025-07-31 12:24:36 UTC
Enhancement not pushed to 24.11.x