Bug 39053: Add page-section div to reports results pages
This patch adds a "page-section" wrapping div around reports results
pages where it is missing.
Note: This patch contains indentation changes.
To test, apply the patch and test the following reports, confirming that
the results page looks correct:
- Patrons with the most checkouts
- Acquisitions statistics
- Patrons with no checkouts
- Patrons statistics
- Most-circulated items
- Catalog statistics
- Average loan time
- Circulation statistics
- Lost items
- Catalog by item types
- Holds statistics
Sponsored-by: Athens County Public Libraries
Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
(cherry picked from commit bf1e45e490)
Signed-off-by: Baptiste Wojtkowski <baptiste.wojtkowski@biblibre.com>
This commit is contained in:
parent
38460cbd40
commit
ffcc2a5a9f
|
|
@ -70,7 +70,7 @@
|
|||
</p>
|
||||
[% END %]
|
||||
[% END %]
|
||||
|
||||
<div class="page-section">
|
||||
<table>
|
||||
<tr >
|
||||
<th>[% mainloo.line | html %] / [% mainloo.column | html %]</th>
|
||||
|
|
@ -105,6 +105,8 @@
|
|||
<th>[% mainloo.total | html %]</th>
|
||||
</tr>
|
||||
</table>
|
||||
</div>
|
||||
<!-- /.page-section -->
|
||||
[% END %]
|
||||
[% ELSE %]
|
||||
<form method="get" action="/cgi-bin/koha/reports/acquisitions_stats.pl">
|
||||
|
|
|
|||
|
|
@ -84,6 +84,7 @@
|
|||
[% END %]
|
||||
[% END %]
|
||||
|
||||
<div class="page-section">
|
||||
<table>
|
||||
<tr>
|
||||
<th rowspan="2">Rank</th>
|
||||
|
|
@ -114,6 +115,8 @@
|
|||
</tr>
|
||||
[% END %]
|
||||
</table>
|
||||
</div>
|
||||
<!-- /.page-section -->
|
||||
[% END %]
|
||||
[% ELSE %]
|
||||
<h1>Patrons with the most checkouts</h1>
|
||||
|
|
|
|||
|
|
@ -52,6 +52,7 @@
|
|||
[% END %]
|
||||
[% END %]
|
||||
|
||||
<div class="page-section">
|
||||
<table>
|
||||
<tr>
|
||||
<th>Num/Patrons</th>
|
||||
|
|
@ -69,6 +70,8 @@
|
|||
</tr>
|
||||
[% END %]
|
||||
</table>
|
||||
</div>
|
||||
<!-- /.page-section -->
|
||||
[% END %]
|
||||
[% ELSE %]
|
||||
<h1>Patrons with no checkouts</h1>
|
||||
|
|
|
|||
|
|
@ -63,6 +63,7 @@
|
|||
</p>
|
||||
[% END %]
|
||||
[% END %]
|
||||
<div class="page-section">
|
||||
<table>
|
||||
<tr>
|
||||
<th>[% mainloo.line | html %] / [% mainloo.column | html %]</th>
|
||||
|
|
@ -91,6 +92,8 @@
|
|||
<th>[% mainloo.total | html %]</th>
|
||||
</tr>
|
||||
</table>
|
||||
</div>
|
||||
<!-- /.page-section -->
|
||||
[% END %]
|
||||
[% ELSE %]
|
||||
|
||||
|
|
|
|||
|
|
@ -57,6 +57,7 @@
|
|||
[% END %]
|
||||
[% END %]
|
||||
|
||||
<div class="page-section">
|
||||
<table>
|
||||
<tr> <th>Rank/Biblioitemnumbers</th>
|
||||
[% FOREACH loopco IN mainloo.loopcol %]
|
||||
|
|
@ -88,6 +89,8 @@
|
|||
</tr>
|
||||
[% END %]
|
||||
</table>
|
||||
</div>
|
||||
<!-- /.page-section -->
|
||||
[% END %]
|
||||
[% ELSE %]
|
||||
<h1>Most-circulated items</h1>
|
||||
|
|
|
|||
|
|
@ -56,6 +56,7 @@
|
|||
[% END %]
|
||||
[% END %]
|
||||
|
||||
<div class="page-section">
|
||||
<table id="catalogue_stats">
|
||||
<thead>
|
||||
<tr>
|
||||
|
|
@ -96,6 +97,8 @@
|
|||
</tr>
|
||||
</tfoot>
|
||||
</table>
|
||||
</div>
|
||||
<!-- /.page-section -->
|
||||
[% END %]
|
||||
[% ELSE %]
|
||||
|
||||
|
|
|
|||
|
|
@ -53,6 +53,7 @@
|
|||
[% END %]
|
||||
[% END %]
|
||||
|
||||
<div class="page-section">
|
||||
<table>
|
||||
<tr>
|
||||
<th>[% mainloo.line | html %] / [% mainloo.column | html %]</th>
|
||||
|
|
@ -85,6 +86,8 @@
|
|||
<th>[% mainloo.total | html %]</th>
|
||||
</tr>
|
||||
</table>
|
||||
</div>
|
||||
<!-- /.page-section -->
|
||||
[% END %]
|
||||
[% ELSE %]
|
||||
|
||||
|
|
|
|||
|
|
@ -86,6 +86,7 @@
|
|||
</ul>
|
||||
[% END %]
|
||||
|
||||
<div class="page-section">
|
||||
<table>
|
||||
<tr>
|
||||
<th>[% mainloo.line | html %] / [% mainloo.column | html %]</th>
|
||||
|
|
@ -111,6 +112,8 @@
|
|||
<th>[% mainloo.total | html %]</th>
|
||||
</tr>
|
||||
</table>
|
||||
</div>
|
||||
<!-- /.page-section -->
|
||||
[% END %]
|
||||
[% ELSE %]
|
||||
|
||||
|
|
|
|||
|
|
@ -84,6 +84,7 @@
|
|||
</div>
|
||||
[% END %]
|
||||
|
||||
<div class="page-section">
|
||||
<table id="lostitems-table">
|
||||
<thead>
|
||||
<tr>
|
||||
|
|
@ -140,6 +141,8 @@
|
|||
[% END %]
|
||||
</tbody>
|
||||
</table>
|
||||
</div>
|
||||
<!-- /.page-section -->
|
||||
[% END %]
|
||||
[% ELSE %]
|
||||
|
||||
|
|
|
|||
|
|
@ -51,6 +51,7 @@
|
|||
[% IF ( do_it ) %]
|
||||
[% FOREACH mainloo IN mainloop %]
|
||||
<h1>Reports on item types [% IF branch %] held at [% Branches.GetName( branch ) | html %][% END %]</h1>
|
||||
<div class="page-section">
|
||||
<table id="itemtypest">
|
||||
<thead>
|
||||
<tr>
|
||||
|
|
@ -73,6 +74,8 @@
|
|||
[% END %]
|
||||
</tbody>
|
||||
</table>
|
||||
</div>
|
||||
<!-- /.page-section -->
|
||||
[% END %]
|
||||
[% ELSE %]
|
||||
<h1>View a count of items held at your library grouped by item type</h1>
|
||||
|
|
|
|||
|
|
@ -81,7 +81,7 @@
|
|||
[% END %]
|
||||
|
||||
[% FOREACH mainloo IN mainloop %]
|
||||
|
||||
<div class="page-section">
|
||||
<table>
|
||||
<tr>
|
||||
<th>[% mainloo.line | html %] / [% mainloo.column | html %]</th>
|
||||
|
|
@ -109,6 +109,8 @@
|
|||
<th>[% mainloo.total | html %]</th>
|
||||
</tr>
|
||||
</table>
|
||||
</div>
|
||||
<!-- /.page-section -->
|
||||
[% END %]
|
||||
[% END %]
|
||||
[% ELSE %]
|
||||
|
|
|
|||
Loading…
Reference in New Issue