From 715e3323e143186fc261da01c2ab909ec5d6d095 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Wed, 12 Jan 2022 19:05:02 +0000
Subject: [PATCH] Bug 29867: (follow-up) Add markup comments

This patch adds comments to the template to highlight the markup
structure.

This patch should have no effect on the page's appearance or
functionality.

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
---
 .../en/modules/admin/authorised_values.tt     | 30 +++++++++----------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/authorised_values.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/authorised_values.tt
index 2be9bd054d..4aa7da0001 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/authorised_values.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/authorised_values.tt
@@ -76,9 +76,9 @@
                     Authorized values
                 </a>
             </li>
-        [% END %]
+        [% END # /IF op == 'add_form' %]
     </ol>
-</nav>
+</nav> <!-- /#breadcrumbs -->
 
 <div class="main container-fluid">
     <div class="row">
@@ -144,7 +144,6 @@
                                         </select>
                                         <div class="hint">Select 'All libraries' if this authorized value must be displayed all the time. Otherwise select libraries you want to associate with this value.</div>
                                     </li>
-
                                 </ol>
 
                                 <div id="icons" class="toptabs" style="clear:both">
@@ -184,21 +183,22 @@
                                                             [% END %]
                                                         </label>
                                                     </li>
-                                                [% END %]
+                                                [% END # /FOREACH image %]
                                             </ul>
                                             <br class="clear" />
                                         </div>
-                                    [% END %]
-                                </div>
-                            [% END %]
+                                    [% END # /FOREACH imageset %]
+                                </div> <!-- /#icons -->
+                            [% END # /IF action_add_category %]
                         </fieldset>
+
                         <fieldset class="action">
                             <input type="hidden" name="id" value="[% id | html %]" />
                             <input type="submit" value="Save" />
                             <a class="cancel" href="/cgi-bin/koha/admin/authorised_values.pl?searchfield=[% category_name | url %]">Cancel</a>
                         </fieldset>
                     </form>
-                [% END %]
+                [% END # /IF op == 'add_form' %]
 
                 [% IF op == 'list' %]
                     <div id="toolbar" class="btn-toolbar">
@@ -252,7 +252,7 @@
                                 [% m.code | html %]
                             [% END %]
                         </div>
-                    [% END %]
+                    [% END # /FOR m IN messages %]
 
                     [% IF ( searchfield ) %]
 
@@ -269,7 +269,7 @@
                                     [% END %]
                                 </select>
                             </p>
-                        </form>
+                        </form> <!-- /#category -->
 
                         [% IF loop %]
                             <table id="categoriest" class="authorized_values_table">
@@ -309,14 +309,14 @@
                                                         [% END %]
                                                 [% ELSE %]
                                                     No limitation
-                                                [% END %]
+                                                [% END # /IF loo.branches.size > 0 %]
                                             </td>
                                             <td class="actions"><a href="/cgi-bin/koha/admin/authorised_values.pl?op=add_form&amp;id=[% loo.id | uri %]" class="btn btn-default btn-xs"><i class="fa fa-pencil"></i> Edit</a>
                                             <a class="delete btn btn-default btn-xs" href="/cgi-bin/koha/admin/authorised_values.pl?op=delete&amp;searchfield=[% searchfield | uri %]&amp;id=[% loo.id | uri %]"><i class="fa fa-trash"></i> Delete</a></td>
                                         </tr>
                                     [% END # /FOREACH loop %]
                                 </tbody>
-                            </table>
+                            </table> <!-- /#categoriest -->
                         [% ELSE %]
                             <div class="dialog message">
                                 <p>There are no authorized values defined for [% category.category_name | html %]</p>
@@ -333,7 +333,7 @@
                                     </form>
                                 [% END %]
                             </div>
-                        [% END %]
+                        [% END # /IF loop %]
 
                     [% ELSE # category was selected %]
 
@@ -356,9 +356,9 @@
                                             <a id="addauth" class="btn btn-default btn-xs" href= "/cgi-bin/koha/admin/authorised_values.pl?op=add_form&amp;category=[% c | uri %]"><i class="fa fa-plus"> </i> Add</a>
                                         </td>
                                     </tr>
-                                [% END %]
+                                [% END # /FOR c %]
                             </tbody>
-                        </table>
+                        </table> <!-- /#categoriest -->
 
                     [% END # /IF ( searchfield  )%]
 
-- 
2.39.5