From 7b6b7a4a3ab6190d385fceee56e11294ad6f6d58 Mon Sep 17 00:00:00 2001
From: Lucas Gass <lucas@bywatersolutions.com>
Date: Mon, 24 Oct 2022 17:45:08 +0000
Subject: [PATCH] Bug 31955: Add page-section div to additional-fields.tt

To test:
1. Apply patch
2. Look at Administration > Additional fields.
3. Each of the 4 tables should have a page-section div around it:

    Order baskets (aqbasket)
    Subscriptions (subscription)
    Account lines (credit) (accountlines:credit)
    Account lines (debit) (accountlines:debit)

Note: There are some indentation changes.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
---
 .../en/modules/admin/additional-fields.tt     | 60 ++++++++++---------
 1 file changed, 31 insertions(+), 29 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/additional-fields.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/additional-fields.tt
index 31acd80d05..858e5a7751 100755
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/additional-fields.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/additional-fields.tt
@@ -134,41 +134,43 @@
     [% ELSIF op == 'list' %]
         <h1>Additional fields for '[% tablename | html %]'</h1>
         [% IF fields %]
-            <table id="fieldst">
-                <thead>
-                    <tr>
-                        <th>Name</th>
-                        <th>Authorized value category</th>
-                        [% IF show_marcfield %]
-                            <th>MARC field</th>
-                        [% END %]
-                        [% IF show_searchable %]
-                            <th>Searchable</th>
-                        [% END %]
-                        <th class="noExport">Actions</th>
-                    </tr>
-                </thead>
-                <tbody>
-                    [% FOR field IN fields %]
+            <div class="page-section">
+                <table id="fieldst">
+                    <thead>
                         <tr>
-                            <td>[% field.name | html %]</td>
-                            <td>[% field.authorised_value_category | html %]</td>
+                            <th>Name</th>
+                            <th>Authorized value category</th>
                             [% IF show_marcfield %]
-                                <td>[% field.marcfield | html %]</td>
+                                <th>MARC field</th>
                             [% END %]
                             [% IF show_searchable %]
-                                <td>
-                                    [% IF field.searchable %]Yes[% ELSE %]No[% END %]
-                                </td>
+                                <th>Searchable</th>
                             [% END %]
-                            <td class="actions">
-                                <a class="btn btn-default btn-xs" href="?op=add_form&amp;field_id=[% field.id | html %]"><i class="fa fa-pencil"></i> Edit</a>
-                                <a class="confirmdelete btn btn-default btn-xs" href="?op=delete&amp;field_id=[% field.id | uri %]&amp;tablename=[% tablename | uri %]"><i class="fa fa-trash"></i> Delete</a>
-                            </td>
+                            <th class="noExport">Actions</th>
                         </tr>
-                    [% END %]
-                </tbody>
-            </table>
+                    </thead>
+                    <tbody>
+                        [% FOR field IN fields %]
+                            <tr>
+                                <td>[% field.name | html %]</td>
+                                <td>[% field.authorised_value_category | html %]</td>
+                                [% IF show_marcfield %]
+                                    <td>[% field.marcfield | html %]</td>
+                                [% END %]
+                                [% IF show_searchable %]
+                                    <td>
+                                        [% IF field.searchable %]Yes[% ELSE %]No[% END %]
+                                    </td>
+                                [% END %]
+                                <td class="actions">
+                                    <a class="btn btn-default btn-xs" href="?op=add_form&amp;field_id=[% field.id | html %]"><i class="fa fa-pencil"></i> Edit</a>
+                                    <a class="confirmdelete btn btn-default btn-xs" href="?op=delete&amp;field_id=[% field.id | uri %]&amp;tablename=[% tablename | uri %]"><i class="fa fa-trash"></i> Delete</a>
+                                </td>
+                            </tr>
+                        [% END %]
+                    </tbody>
+                </table>
+            </div><!-- /.page-section -->
         [% ELSE %]
             [% IF tablename %]There are no additional fields defined for this table.[% END %]
         [% END %]
-- 
2.39.5