Bug 25022

Summary: Display problem in authority editor with repeatable field
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: Staff interfaceAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: minor    
Priority: P5 - low CC: 1joynelson, gmcharlt, lucas, oleonard
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.05.00, 19.11.06, 19.05.11
Attachments: Screenshot authority editor
Bug 25022: Display problem in authority editor with repeatable field
Bug 25022: Display problem in authority editor with repeatable field

Description Marcel de Rooy 2020-03-31 12:20:28 UTC
See attached image. I open an authority record with two 024 fields (that do not have a $a subfield btw). The second field is not displayed correctly. Looks like the unordered list is not closed or so?
On current master (writing this 31-03).
Comment 1 Marcel de Rooy 2020-03-31 12:21:20 UTC
Created attachment 102125 [details]
Screenshot authority editor
Comment 2 Marcel de Rooy 2020-03-31 12:25:47 UTC
Owen, any idea ?
Comment 3 Marcel de Rooy 2020-03-31 12:30:03 UTC
Also comes up with 014 and using $a.
So no specific issue with 024 or not using $a.
Comment 4 Owen Leonard 2020-03-31 14:43:12 UTC
Created attachment 102143 [details] [review]
Bug 25022: Display problem in authority editor with repeatable field

This patch corrects an error in the structure of the authority editor
template so that repeating subfields do not generate invalid markup. The
correction was found by comparing with addbiblio.tt.

If you are viewing the diff, please note that some minor whitespace
changes have been made.

To reproduce, clone and and populate any repeatable subfield, save, and
open the authority for editing again. The clones subfield will be
formatting incorrectly, showing bullet points on each line.

To test, apply the patch and reload the authority editor showing the
record with the cloned subfield. It should display correctly.
Comment 5 Marcel de Rooy 2020-04-01 12:13:24 UTC
Created attachment 102215 [details] [review]
Bug 25022: Display problem in authority editor with repeatable field

This patch corrects an error in the structure of the authority editor
template so that repeating subfields do not generate invalid markup. The
correction was found by comparing with addbiblio.tt.

If you are viewing the diff, please note that some minor whitespace
changes have been made.

To reproduce, clone and and populate any repeatable subfield, save, and
open the authority for editing again. The clones subfield will be
formatting incorrectly, showing bullet points on each line.

To test, apply the patch and reload the authority editor showing the
record with the cloned subfield. It should display correctly.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 6 Marcel de Rooy 2020-04-01 12:14:40 UTC
Thanks Owen!
Taking this to PQA right away. Small template correction.
Comment 7 Martin Renvoize 2020-04-03 13:24:50 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 8 Joy Nelson 2020-04-28 21:43:18 UTC
backported to 19.11.x branch for 19.11.06
Comment 9 Lucas Gass 2020-05-11 19:48:47 UTC
backported to 19.05.x for 19.05.11