From bca10125a4bd04529c6938488d5e54c0bdc96dbe Mon Sep 17 00:00:00 2001
From: Katrin Fischer <katrin.fischer.83@web.de>
Date: Sat, 7 Jan 2023 01:31:34 +0000
Subject: [PATCH] Bug 32587: Add page-section to 'items with no checkouts'
 report
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This adds page sections to the summary and result tables
for the 'items with no checkouts' report.

To test:
* Run the 'item with no checkouts' report with different settings
* Confirm result tables are missing the white brackground from
  page-section
* Apply patch
* Confirm the result tables now do have the white background

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
  → Confirm both problem and solution
---
 .../prog/en/modules/reports/catalogue_out.tt  | 112 +++++++++---------
 1 file changed, 58 insertions(+), 54 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/catalogue_out.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/catalogue_out.tt
index 12e29249bb..35a62dd0dd 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/catalogue_out.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/catalogue_out.tt
@@ -53,67 +53,71 @@
                         [% END %]
 
                         [% IF ( mainloo.looptables ) %]
-                            <h3><a name="summary" id="summary"></a>Summary</h3>
-                            <table id="summary_table">
-                                <tr>
-                                    <th>Group</th>
-                                    <th>Call number range</th>
-                                    <th>Number of items displayed</th>
-                                    <th>Total items in group</th>
-                                </tr>
-                                [% FOREACH looptable IN mainloo.looptables %]
+                            <div class="page-section">
+                                <h3><a name="summary" id="summary"></a>Summary</h3>
+                                <table id="summary_table">
                                     <tr>
-                                        <td>
-                                            <a href="#table[% looptable.coltitle | uri %]">[% looptable.coltitle | html %]</a>
-                                        </td>
-                                        <td>
-                                            [% IF ( looptable.looptable_count ) %]
-                                                [% looptable.looptable_first | html %] to [% looptable.looptable_last | html %]
-                                            [% END %]
-                                        </td>
-                                        <td>[% looptable.looptable_count | html %]</td>
-                                        <td>[% looptable.coltitle_count | html %]</td>
+                                        <th>Group</th>
+                                        <th>Call number range</th>
+                                        <th>Number of items displayed</th>
+                                        <th>Total items in group</th>
                                     </tr>
-                                [% END %]
-                                <tr>
-                                    <th>TOTAL</th>
-                                    <th></th>
-                                    <th>[% mainloo.total_looptable_count | html %]</th>
-                                    <th>[% mainloo.total_coltitle_count | html %]</th>
-                                </tr>
-                            </table>
-                        [% END %]
-
-                        [% FOREACH looptable IN mainloo.looptables %]
-                            <h3>
-                                <a id="table[% looptable.coltitle | html %]"></a>
-                                [% looptable.coltitle | html %]
-                            </h3>
-                            <table>
-                                <tr>
-                                    <th>#</th>
-                                    <th>Call number</th>
-                                    <th>Barcode</th>
-                                    <th>Item details</th>
-                                </tr>
-                                [% IF ( looptable.looprow ) %]
-                                    [% FOREACH loopro IN looptable.looprow %]
+                                    [% FOREACH looptable IN mainloo.looptables %]
                                         <tr>
-                                            <td>[% loop.count | html %]</td>
-                                            <td>[% IF ( loopro.itemcallnumber ) %][% loopro.itemcallnumber | html %][% ELSE %]No call number[% END %]</td>
-                                            <td>[% IF ( loopro.barcode ) %][% loopro.barcode | html %][% ELSE %]No barcode[% END %]</td>
                                             <td>
-                                                <p><a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% loopro.biblionumber | uri %]">[% IF ( loopro.title ) %][% loopro.title | html %][% ELSE %]NO TITLE[% END %]</a> [% IF ( loopro.author ) %] by [% loopro.author | html %][% END %]</p>
-                                                [% IF ( loopro.branch ) %]at [% Branches.GetName( loopro.branch ) | html %][% END %]
+                                                <a href="#table[% looptable.coltitle | uri %]">[% looptable.coltitle | html %]</a>
                                             </td>
+                                            <td>
+                                                [% IF ( looptable.looptable_count ) %]
+                                                    [% looptable.looptable_first | html %] to [% looptable.looptable_last | html %]
+                                                [% END %]
+                                            </td>
+                                            <td>[% looptable.looptable_count | html %]</td>
+                                            <td>[% looptable.coltitle_count | html %]</td>
                                         </tr>
                                     [% END %]
-                                    [% ELSE %]
-                                        <tr>
-                                            <td colspan="4">No items for [% looptable.coltitle | html %]</td>
-                                        </tr>
-                                    [% END %]
-                            </table>
+                                    <tr>
+                                        <th>TOTAL</th>
+                                        <th></th>
+                                        <th>[% mainloo.total_looptable_count | html %]</th>
+                                        <th>[% mainloo.total_coltitle_count | html %]</th>
+                                    </tr>
+                                </table>
+                            </div>
+                        [% END %]
+
+                        [% FOREACH looptable IN mainloo.looptables %]
+                            <div class="page-section">
+                                <h3>
+                                    <a id="table[% looptable.coltitle | html %]"></a>
+                                    [% looptable.coltitle | html %]
+                                </h3>
+                                <table>
+                                    <tr>
+                                        <th>#</th>
+                                        <th>Call number</th>
+                                        <th>Barcode</th>
+                                        <th>Item details</th>
+                                    </tr>
+                                    [% IF ( looptable.looprow ) %]
+                                        [% FOREACH loopro IN looptable.looprow %]
+                                            <tr>
+                                                <td>[% loop.count | html %]</td>
+                                                <td>[% IF ( loopro.itemcallnumber ) %][% loopro.itemcallnumber | html %][% ELSE %]No call number[% END %]</td>
+                                                <td>[% IF ( loopro.barcode ) %][% loopro.barcode | html %][% ELSE %]No barcode[% END %]</td>
+                                                <td>
+                                                    <p><a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% loopro.biblionumber | uri %]">[% IF ( loopro.title ) %][% loopro.title | html %][% ELSE %]NO TITLE[% END %]</a> [% IF ( loopro.author ) %] by [% loopro.author | html %][% END %]</p>
+                                                    [% IF ( loopro.branch ) %]at [% Branches.GetName( loopro.branch ) | html %][% END %]
+                                                </td>
+                                            </tr>
+                                        [% END %]
+                                        [% ELSE %]
+                                            <tr>
+                                                <td colspan="4">No items for [% looptable.coltitle | html %]</td>
+                                            </tr>
+                                        [% END %]
+                                </table>
+                            </div>
                         [% END %]
                     [% END %]
                 [% ELSE %]
-- 
2.34.1