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>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt
index 55ddb49..76c1df8 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt
@@ -406,20 +406,17 @@ function confirmnotdup(redirect){
                     </a>
                 [% END %]
                 </span>
-                
-            </li>
-            <!-- End of the line -->
-        [% END %]
-
-        </ul> <!-- /.sortable_subfield -->
-    </li>
-    [% END %]<!-- if innerloo.tag -->
-    </ul> <!-- /.sortable_field -->
-    [% END %]<!-- BIG_LOO.innerloop -->
-    </div>
-[% END %]<!-- BIG_LOOP -->
-
-</div><!-- tabs -->
+                                </li>
+                                <!-- End of the line -->
+                            [% END %]
+                            </ul> <!-- /.sortable_subfield -->
+                        </li>
+                    [% END %]<!-- if innerloo.tag -->
+                [% END %]<!-- BIG_LOO.innerloop -->
+                </ul> <!-- /.sortable_field -->
+            </div>
+        [% END %]<!-- BIG_LOOP -->
+    </div><!-- tabs -->
 
 <div name="hidden" id="hidden" class="tab">
 [% FOREACH hidden_loo IN hidden_loop %]