Bug 27713

Summary: Duplicate search field IDs in MARC framework administration template
Product: Koha Reporter: Owen Leonard <oleonard>
Component: System AdministrationAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: andrew, fridolin.somers, gmcharlt, jonathan.druart, martin.renvoize, phil, testopia, victor
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27703
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
21.05.00,20.11.04,20.05.10
Circulation function:
Bug Depends on: 25954    
Bug Blocks:    
Attachments: Bug 27713: Duplicate search field IDs in MARC framework administration template
Bug 27713: Duplicate search field IDs in MARC framework administration template
Bug 27713: Duplicate search field IDs in MARC framework administration template

Description Owen Leonard 2021-02-16 13:43:06 UTC
The "search for tag" field in MARC framework administration has the same ID as "search system preferences" field in the header search box. This is invalid and can cause problems when references elements by ID in JavaScript or CSS.
Comment 1 Owen Leonard 2021-02-16 13:51:28 UTC
Created attachment 116925 [details] [review]
Bug 27713: Duplicate search field IDs in MARC framework administration template

This patch makes a minor change to the MARC framework administration
template so that the ID of a form field on the page isn't a duplicate of
one in the header search form.

To test, apply the patch and go to Administration -> MARC bibliographic
framework -> MARC structure.

- Test the "Search for tag" form field to confirm that it works
  correctly.
- Validate the HTML of the page to confirm that there are no "Duplicate
  ID" errors.
Comment 2 Phil Ringnalda 2021-02-20 23:27:37 UTC
Created attachment 117121 [details] [review]
Bug 27713: Duplicate search field IDs in MARC framework administration template

This patch makes a minor change to the MARC framework administration
template so that the ID of a form field on the page isn't a duplicate of
one in the header search form.

To test, apply the patch and go to Administration -> MARC bibliographic
framework -> MARC structure.

- Test the "Search for tag" form field to confirm that it works
  correctly.
- Validate the HTML of the page to confirm that there are no "Duplicate
  ID" errors.

Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org>
Comment 3 Martin Renvoize (ashimema) 2021-02-22 16:06:22 UTC
Created attachment 117161 [details] [review]
Bug 27713: Duplicate search field IDs in MARC framework administration template

This patch makes a minor change to the MARC framework administration
template so that the ID of a form field on the page isn't a duplicate of
one in the header search form.

To test, apply the patch and go to Administration -> MARC bibliographic
framework -> MARC structure.

- Test the "Search for tag" form field to confirm that it works
  correctly.
- Validate the HTML of the page to confirm that there are no "Duplicate
  ID" errors.

Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Martin Renvoize (ashimema) 2021-02-22 16:06:39 UTC
Always good to improve these things.. Passing QA
Comment 5 Jonathan Druart 2021-02-23 12:22:18 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 6 Fridolin Somers 2021-02-25 09:50:24 UTC
Pushed to 20.11.x for 20.11.04
Comment 7 Andrew Fuerste-Henry 2021-03-05 17:05:08 UTC
Pushed to 20.05.x for 20.05.10
Comment 8 Victor Grousset/tuxayo 2021-03-19 00:07:36 UTC
Not backported to oldoldstable (19.11.x). Feel free to ask if it's needed.