David Cook
58b85e1df1
When you cancel a hold in the "Hold Waiting" tab of Holds Awaiting Pickup, you should be prompted to retain or transfer the hold for the next patron in the queue. However, instead of seeing this prompt, you are just shown white screen that fails to alert you to the fact that there are more holds on that item! There is a missing FOREACH reference in waitingreserves.tt. The template tries to directly reference the hash within the cancel_results array, but we need to access the array before we can access the hash. Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
189 lines
9.2 KiB
Text
189 lines
9.2 KiB
Text
[% INCLUDE 'doc-head-open.inc' %]
|
|
<title>Koha › Circulation › Holds awaiting pickup</title>
|
|
[% INCLUDE 'doc-head-close.inc' %]
|
|
<script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.tablesorter.min.js"></script>
|
|
<script type="text/javascript" language="JavaScript">
|
|
//<![CDATA[
|
|
$.tablesorter.addParser({
|
|
id: 'articles',
|
|
is: function(s) {return false; },
|
|
format: function(s) { return s.toLowerCase().replace(/^(the|an|a) /,''); },
|
|
type: 'text'
|
|
});
|
|
$(document).ready(function() {
|
|
$('#resultlist').tabs();
|
|
$("th a").hide();
|
|
$.tablesorter.defaults.widgets = ['zebra'];
|
|
$("#holdst,#holdso").tablesorter({[% IF ( dateformat == 'metric' ) %]
|
|
dateFormat: 'uk',[% END %]
|
|
sortList: [[3,0]],
|
|
headers: { 1:{sorter:'articles'},3: { sorter: 'articles' },4:{sorter:false}}
|
|
});
|
|
});
|
|
//]]>
|
|
</script>
|
|
</head>
|
|
<body id="circ_waitingreserves" class="circ">
|
|
[% INCLUDE 'header.inc' %]
|
|
[% INCLUDE 'circ-search.inc' %]
|
|
|
|
<div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a>
|
|
› <a href="/cgi-bin/koha/circ/circulation-home.pl">Circulation</a>
|
|
› <a href="/cgi-bin/koha/circ/waitingreserves.pl">Holds awaiting pickup</a>
|
|
</div>
|
|
|
|
<div id="doc" class="yui-t7">
|
|
|
|
<div id="bd">
|
|
<div id="yui-main">
|
|
<div class="yui-g">
|
|
|
|
<h2>Holds awaiting pickup for your library on: [% show_date %]
|
|
[% IF ( all_branches_link ) %]
|
|
<span style="margin-left:20px"><a href="[% all_branches_link %]">
|
|
View all libraries</a></span>
|
|
[% END %]
|
|
</h2>
|
|
[% IF ( cancel_result ) %]
|
|
[% FOREACH cancel_result %]
|
|
[% IF ( messagetransfert ) %]
|
|
<div>
|
|
<h2>Hold find for ([% nextreservtitle %]), must be transferred</h2>
|
|
<p>This hold placed by : <b> [% nextreservsurname %] [% nextreservfirstname %]</b> at the library : <b> [% branchname %] </b>, Please transfer this hold.
|
|
</p>
|
|
<form name="cancelReservewithtransfert" action="waitingreserves.pl" method="post">
|
|
<input type="submit" class="button" />
|
|
</form>
|
|
</div>
|
|
[% END %]
|
|
[% IF ( waiting ) %]
|
|
<div id="bloc25">
|
|
<h2>This hold is waiting</h2>
|
|
<p>This hold ([% nextreservtitle %]) was placed by <b> : [% nextreservsurname %] [% nextreservfirstname %]</b>,
|
|
Please retain this hold.
|
|
</p>
|
|
<form name="cancelReservewithwaiting" action="waitingreserves.pl" method="post">
|
|
<input type="submit" />
|
|
</form>
|
|
</div>
|
|
[% END %]
|
|
[% END %]
|
|
[% ELSE %]
|
|
<div id="resultlist" class="toptabs">
|
|
<ul>
|
|
<li><a href="#holdswaiting">[% reservecount %] Hold(s) waiting</a></li>
|
|
<li><a href="#holdsover">[% overcount %] Hold(s) over</a></li>
|
|
</ul>
|
|
<div id="holdswaiting">
|
|
[% IF ( reserveloop ) %]
|
|
<table id="holdst">
|
|
<thead><tr>
|
|
<th>Available since</th>
|
|
<th>Title</th>
|
|
<th>Patron</th>
|
|
<th>Location</th>
|
|
<th>Copy number</th>
|
|
<th>Enumeration</th>
|
|
<th>Action</th>
|
|
</tr></thead>
|
|
<tbody>[% FOREACH reserveloo IN reserveloop %]
|
|
<tr>
|
|
<td><p>[% reserveloo.waitingdate %]</p></td>
|
|
<td>[% INCLUDE 'biblio-default-view.inc' biblionumber = reserveloo.biblionumber %]
|
|
[% reserveloo.title |html %] [% reserveloo.subtitle |html %]
|
|
</a>
|
|
(<b>[% reserveloo.itemtype %]</b>)
|
|
<br />Barcode: [% reserveloo.barcode %]
|
|
</td>
|
|
<td><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% reserveloo.borrowernum %]">[% reserveloo.borrowername %], [% reserveloo.borrowerfirstname %]</a><br />[% reserveloo.borrowerphone %]<br />
|
|
[% IF ( reserveloo.borrowermail ) %]<a href="mailto:[% reserveloo.email %]?subject=Reservation: [% reserveloo.title %]">
|
|
[% reserveloo.borrowermail %]</a>[% END %]
|
|
</td>
|
|
<td>[% reserveloo.homebranch %]<br />[% reserveloo.itemcallnumber %]</td>
|
|
<td>[% reserveloo.copynumber %]</td>
|
|
<td>[% reserveloo.enumchron %]</td>
|
|
<td>
|
|
<form name="cancelReserve" action="waitingreserves.pl" method="post">
|
|
<input type="hidden" name="borrowernumber" value="[% reserveloo.borrowernum %]" />
|
|
<input type="hidden" name="itemnumber" value="[% reserveloo.itemnumber %]" />
|
|
<input type="hidden" name="fbr" value="[% reserveloo.holdingbranch %]" />
|
|
<input type="hidden" name="tbr" value="[% reserveloo.homebranch %]" />
|
|
[% IF ( reserveloo.dotransfer ) %]
|
|
<input type="submit" value="Cancel hold and return to : [% reserveloo.branchname %]" />
|
|
[% ELSE %]
|
|
<input type="submit" value="Cancel hold" />
|
|
[% END %]
|
|
</form>
|
|
</td>
|
|
</tr>
|
|
[% END %]</tbody>
|
|
</table>
|
|
[% ELSE %]
|
|
<div class="dialog message">No holds found.</div>
|
|
[% END %]
|
|
</div>
|
|
<div id="holdsover">
|
|
<p>Holds listed here have been awaiting pickup for more than [% ReservesMaxPickUpDelay %] days.</p>
|
|
[% IF ( overloop ) %]
|
|
<p>
|
|
<form name="cancelAllReserve" action="waitingreserves.pl" method="post">
|
|
<input type="hidden" name="cancelall" value="1" />
|
|
<input type="hidden" name="allbranches" value="[% allbranches %]" />
|
|
<input type="submit" value="Cancel [% IF TransferWhenCancelAllWaitingHolds %]and Transfer [% END %]All" />
|
|
</form>
|
|
[% UNLESS TransferWhenCancelAllWaitingHolds %]
|
|
Only items that need not be transferred will be cancelled (TransferWhenCancelAllWaitingHolds sypref)
|
|
[% END %]
|
|
</p>
|
|
<br/>
|
|
<table id="holdso">
|
|
<thead><tr>
|
|
<th>Available since</th>
|
|
<th>Title</th>
|
|
<th>Patron</th>
|
|
<th>Location</th>
|
|
<th>Copy number</th>
|
|
<th>Enumeration</th>
|
|
<th>Action</th>
|
|
</tr></thead>
|
|
<tbody>[% FOREACH overloo IN overloop %]
|
|
<tr>
|
|
<td><p>[% overloo.waitingdate %]</p></td>
|
|
<td>[% INCLUDE 'biblio-default-view.inc' biblionumber = overloo.biblionumber %][% overloo.title |html %] [% overloo.subtitle |html %]
|
|
</a>
|
|
[% UNLESS ( item_level_itypes ) %][% IF ( overloo.itemtype ) %] (<b>[% overloo.itemtype %]</b>)[% END %][% END %]
|
|
<br />Barcode: [% overloo.barcode %]
|
|
</td>
|
|
<td><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% overloo.borrowernum %]">[% overloo.borrowername %], [% overloo.borrowerfirstname %]</a><br />[% overloo.borrowerphone %]<br />
|
|
[% IF ( overloo.borrowermail ) %]<a href="mailto:[% overloo.email %]?subject=Reservation: [% overloo.title |url %]">
|
|
[% overloo.borrowermail %]</a>[% END %]
|
|
</td>
|
|
<td>[% overloo.homebranch %] [% overloo.itemcallnumber %]</td>
|
|
<td>[% overloo.copynumber %]</td>
|
|
<td>[% overloo.enumchron %]</td>
|
|
<td><form name="cancelReserve" action="waitingreserves.pl" method="post">
|
|
<input type="hidden" name="borrowernumber" value="[% overloo.borrowernum %]" />
|
|
<input type="hidden" name="itemnumber" value="[% overloo.itemnumber %]" />
|
|
<input type="hidden" name="fbr" value="[% overloo.holdingbranch %]" />
|
|
<input type="hidden" name="tbr" value="[% overloo.homebranch %]" />
|
|
[% IF ( overloo.dotransfer ) %]
|
|
<input type="submit" value="Cancel hold and return to : [% overloo.branchname %]" />
|
|
[% ELSE %]
|
|
<input type="submit" value="Cancel hold" />
|
|
[% END %]
|
|
</form>
|
|
</td>
|
|
</tr>
|
|
[% END %]</tbody>
|
|
</table>
|
|
[% ELSE %]
|
|
<div class="dialog message">No holds found.</div>
|
|
[% END %]
|
|
</div>
|
|
</div>
|
|
[% END %]
|
|
|
|
</div>
|
|
</div>
|
|
</div>
|
|
[% INCLUDE 'intranet-bottom.inc' %]
|