Jonathan Druart
9af6c4e34b
This is a recurrent bug we have over the last years. When a script is called with non-existent borrowernumber it will crashes. We need to handle this gracefully instead of letting the script crashes. On bug 18403 a new subroutine is added to the codebase (output_and_exit_if_error) to handle this kind of errors correctly. Since it is not pushed yet, I propose to just redirect to a script that handle it correctly (circulation.pl) instead of adding this message to all these scripts. Test plan: Hit different scripts from the members module and pass a non-existent borrowernumber. You must be redirected to circulation.pl with a friendly message. Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Signed-off-by: Julian Maurice <julian.maurice@biblibre.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
97 lines
3.4 KiB
Text
97 lines
3.4 KiB
Text
[% USE Koha %]
|
|
[% USE AuthorisedValues %]
|
|
[% USE Branches %]
|
|
[% INCLUDE 'doc-head-open.inc' %]
|
|
<title>Koha › Patrons ›
|
|
Statistics for [% INCLUDE 'patron-title.inc' %]
|
|
</title>
|
|
[% INCLUDE 'doc-head-close.inc' %]
|
|
<link rel="stylesheet" type="text/css" href="[% interface %]/[% theme %]/css/datatables.css" />
|
|
[% INCLUDE 'datatables.inc' %]
|
|
[% INCLUDE 'calendar.inc' %]
|
|
<script type="text/javascript">
|
|
$(document).ready(function() {
|
|
$("#statistics").dataTable($.extend(true, {}, dataTablesDefaults, {
|
|
"sDom": 't',
|
|
"bPaginate": false
|
|
}));
|
|
});
|
|
</script>
|
|
</head>
|
|
|
|
<body id="pat_statistics" class="pat">
|
|
[% INCLUDE 'header.inc' %]
|
|
[% INCLUDE 'patron-search.inc' %]
|
|
|
|
<div id="breadcrumbs">
|
|
<a href="/cgi-bin/koha/mainpage.pl">Home</a>
|
|
› <a href="/cgi-bin/koha/members/members-home.pl">Patrons</a>
|
|
› Statistics for [% firstname %] [% surname %] ([% cardnumber %])
|
|
</div>
|
|
|
|
<div id="doc3" class="yui-t1">
|
|
|
|
<div id="bd">
|
|
<div id="yui-main">
|
|
<div class="yui-b">
|
|
[% INCLUDE 'members-toolbar.inc' %]
|
|
|
|
<h3>Statistics for [% UNLESS ( I ) %][% title %] [% firstname %] [% END %] [% surname %] ([% cardnumber %])</h3>
|
|
[% IF ( datas.size ) %]
|
|
<table id="statistics">
|
|
<thead>
|
|
<tr>
|
|
[% FOREACH cn IN column_names %]
|
|
<th>
|
|
[% SWITCH cn %]
|
|
[% CASE 'itype' %]
|
|
Item type
|
|
[% CASE 'ccode' %]
|
|
Collection code
|
|
[% CASE 'location' %]
|
|
Shelving location
|
|
[% CASE 'homebranch' %]
|
|
Home library
|
|
[% CASE 'holdingbranch' %]
|
|
Holding library
|
|
[% CASE %]
|
|
[% cn %]
|
|
[% END %]
|
|
</th>
|
|
[% END %]
|
|
<th>Total checkouts as of yesterday</th>
|
|
<th>Today's checkouts</th>
|
|
<th>Today's checkins</th>
|
|
<th>Total checkouts</th>
|
|
</tr>
|
|
</thead>
|
|
|
|
<tbody>
|
|
[% FOREACH r IN datas %]
|
|
<tr>
|
|
[% FOREACH c IN r %]
|
|
<td>[% c %]</td>
|
|
[% END %]
|
|
</tr>
|
|
[% END %]
|
|
</tbody>
|
|
<tfoot>
|
|
<tr>
|
|
<td colspan="[% column_names.size %]">TOTAL</td>
|
|
<td>[% count_total_precedent_state %]</td>
|
|
<td>[% count_total_issues %]</td>
|
|
<td>[% count_total_issues_returned %]</td>
|
|
<td>[% count_total_actual_state %]</td>
|
|
</tr>
|
|
</tfoot>
|
|
</table>
|
|
[% ELSE %]
|
|
<div class="dialog message">There are no statistics for this patron.</div>
|
|
[% END %]
|
|
</div>
|
|
</div>
|
|
<div class="yui-b">
|
|
[% INCLUDE 'circ-menu.inc' %]
|
|
</div>
|
|
</div>
|
|
[% INCLUDE 'intranet-bottom.inc' %]
|