Bug 39773

Summary: OPAC ILL form does not use client-side form validation for required fields
Product: Koha Reporter: Pedro Amorim <pedro.amorim>
Component: ILLAssignee: Pedro Amorim <pedro.amorim>
Status: Needs documenting --- QA Contact: Lucas Gass (lukeg) <lucas>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, lucas, martin.renvoize, pedro.amorim, tomascohen
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
The OPAC interlibrary loan request form now uses browser-based client-side validation for required fields. When creating a new ILL request, required fields (such as request type) are validated before the form is submitted, providing immediate feedback to users if required information is missing. This enhancement improves the user experience by: - Highlighting required fields that are empty when the user attempts to submit - Preventing incomplete forms from being submitted to the server - Reducing unnecessary page reloads for validation errors This validation applies to both authenticated ILL requests and unauthenticated requests (when the ILLOpacUnauthenticatedRequest system preference is enabled).
Version(s) released in:
25.11.00,25.05.03
Circulation function:
Bug Depends on: 38340    
Bug Blocks:    
Attachments: Bug 39773: Add 'required' attribute for required fields
Bug 39773: Add 'required' attribute for required fields
Bug 39773: Add 'required' attribute for required fields
Bug 39773: Add 'required' attribute for required fields

Description Pedro Amorim 2025-04-29 12:23:10 UTC
We're already using a required parameter. We should make use of it in the HTML and add the 'required' attribute to the HTML inputs.
Comment 1 Pedro Amorim 2025-04-29 12:26:08 UTC Comment hidden (obsolete)
Comment 2 Pedro Amorim 2025-04-29 12:57:19 UTC
Created attachment 181650 [details] [review]
Bug 39773: Add 'required' attribute for required fields

Test plan, apply patch:
1) Enable ILLModule
2) Create a new OPAC request, visit:
http://localhost:8080/cgi-bin/koha/opac-illrequests.pl?op=add_form&backend=Standard
3) Press 'Create'. Notice the required 'type 'is now validated before the form is submitted.

Additional testing:
1) Enable ILLOpacUnauthenticatedRequest. Log out. Repeat 2)
2) Press 'Create'. Notice the required unauthenticated fields are highlighted as required.
Comment 3 Roman Dolny 2025-05-06 22:04:44 UTC
Created attachment 182009 [details] [review]
Bug 39773: Add 'required' attribute for required fields

Test plan, apply patch:
1) Enable ILLModule
2) Create a new OPAC request, visit:
http://localhost:8080/cgi-bin/koha/opac-illrequests.pl?op=add_form&backend=Standard
3) Press 'Create'. Notice the required 'type 'is now validated before the form is submitted.

Additional testing:
1) Enable ILLOpacUnauthenticatedRequest. Log out. Repeat 2)
2) Press 'Create'. Notice the required unauthenticated fields are highlighted as required.

Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Comment 4 Lucas Gass (lukeg) 2025-07-25 16:25:25 UTC
Created attachment 184650 [details] [review]
Bug 39773: Add 'required' attribute for required fields

Test plan, apply patch:
1) Enable ILLModule
2) Create a new OPAC request, visit:
http://localhost:8080/cgi-bin/koha/opac-illrequests.pl?op=add_form&backend=Standard
3) Press 'Create'. Notice the required 'type 'is now validated before the form is submitted.

Additional testing:
1) Enable ILLOpacUnauthenticatedRequest. Log out. Repeat 2)
2) Press 'Create'. Notice the required unauthenticated fields are highlighted as required.

Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Comment 5 Lucas Gass (lukeg) 2025-07-25 20:38:01 UTC
Nice work everyone!

Pushed to main for 25.11
Comment 6 Paul Derscheid 2025-08-22 08:28:03 UTC
Nice work everyone!

Pushed to 25.05.x
Comment 7 Fridolin Somers 2025-09-19 09:30:47 UTC
Depends on Bug 38340 not in 24.11.x