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.
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>