Summary: | OPAC ILL form does not use client-side form validation for required fields | ||
---|---|---|---|
Product: | Koha | Reporter: | Pedro Amorim <pedro.amorim> |
Component: | ILL | Assignee: | 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
Created attachment 181649 [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. 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. 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> 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> Nice work everyone! Pushed to main for 25.11 Nice work everyone! Pushed to 25.05.x |