Bug 11767

Summary: Use validation plugin when creating new authority framework tag
Product: Koha Reporter: Owen Leonard <oleonard>
Component: System AdministrationAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, gmcharlt
Version: Main   
Hardware: All   
OS: All   
URL: /cgi-bin/koha/admin/auth_tag_structure.pl?op=add_form&authtypecode=XXX
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 11764, 11765, 11766    
Bug Blocks:    
Attachments: Bug 11767 - Use validation plugin when creating new authority framework tag
Bug 11767 - Use validation plugin when creating new authority framework tag
Bug 11767 - Use validation plugin when creating new authority framework tag

Description Owen Leonard 2014-02-14 19:29:43 UTC
The new authority framework tag entry form uses custom validation JavaScript which can be easily replaced with HTML5 validation attributes in conjunction with Koha's built-in validation plugin (http://jqueryvalidation.org/documentation/).

The page also includes markup generated by the script which should be moved to the template.
Comment 1 Owen Leonard 2014-02-14 20:04:22 UTC Comment hidden (obsolete)
Comment 2 Chris Cormack 2014-02-19 05:12:28 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2014-02-19 08:52:49 UTC
Created attachment 25425 [details] [review]
Bug 11767 - Use validation plugin when creating new authority framework tag

The page for adding a new tag to an authority framework includes some
custom form validation JavaScript which can be removed in favor of HTML5
validation attributes and Koha's built-in validation plugin. This patch
does so.

The patch also moves some tag markup out of the script and into the
template where it belongs.

To test, apply the patch and go to Administration -> Authority types ->
MARC structure -> New tag. Try submitting the form without entering a
tag number. This should trigger a validation warning.

Submission of the form with valid data should work correctly. Editing an
existing tag should also work correctly.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 4 Galen Charlton 2014-02-19 17:15:50 UTC
Pushed to master.  Thanks, Owen!
Comment 5 Fridolin Somers 2014-05-19 14:51:11 UTC
Pushed to 3.14.x, will be in 3.14.07