Jonathan Druart
0ab22e1c7c
In order to simplify and make uniform the code, the controller scripts send a Koha::Patron object to the templates instead of all attributes of a patron. That will make the code much more easier to maintain and will be less error-prone. The variable "patron" sent to the templates is supposed to represent the patron the librarian is editing the detail. In the members module and some scripts of the circulation module, the patron's detail are sent one by one to the template. That leads to frustration from developpers (making sure everything is passed from all scripts) and to regression (we got tone of bugs in the last year because of this way to do). With this patch set it will be easy access patron's detail, passing only 1 variable from the controllers. Test plan: Play with the patron and circulation module and make sur the detail of the patron you are editing/seeing info are correctly displayed. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
208 lines
11 KiB
Text
208 lines
11 KiB
Text
[% USE Koha %]
|
|
[% USE Branches %]
|
|
[% USE ItemTypes %]
|
|
[% USE AuthorisedValues %]
|
|
[% INCLUDE 'doc-head-open.inc' %]
|
|
<title>Koha › Circulation › Transfers</title>
|
|
[% INCLUDE 'doc-head-close.inc' %]
|
|
</head>
|
|
<body id="circ_branchtransfers" 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> › Transfers</div>
|
|
|
|
[% IF Koha.Preference('CircSidebar') %]<div id="doc3" class="yui-t2">[% ELSE %]<div id="doc" class="yui-t7">[% END %]
|
|
<div id="bd">
|
|
<div id="yui-main">
|
|
[% IF Koha.Preference('CircSidebar') %]<div class="yui-b">[% END %]
|
|
[% IF ( found ) %]
|
|
<div class="yui-g"> <h3>Reserve found</h3>
|
|
<table>
|
|
<caption>
|
|
[% IF ( reserved ) %]
|
|
Reserve found for [% name %] (<a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber %]">[% borrowernumber %]</a>).
|
|
[% END %]
|
|
[% IF ( waiting ) %]
|
|
Item is marked waiting at [% branchname %] for [% name %] (<a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber %]">[% borrowernumber %]</a>).
|
|
[% END %]
|
|
</caption>
|
|
<tr>
|
|
<th>
|
|
[% IF ( reserved ) %]Set reserve to waiting and transfer book to [% branchname %]: [% END %]
|
|
[% IF ( waiting ) %]Cancel reservation and then attempt transfer: [% END %]
|
|
</th>
|
|
<td>
|
|
<form method="post" name="mainform" id="mainform" action="branchtransfers.pl">
|
|
[% FOREACH trsfitemloo IN trsfitemloop %]
|
|
<input type="hidden" name="bc-[% trsfitemloo.counter %]" value="[% trsfitemloo.barcode %]" />
|
|
<input type="hidden" name="fb-[% trsfitemloo.counter %]" value="[% trsfitemloo.frombrcd %]" />
|
|
<input type="hidden" name="tb-[% trsfitemloo.counter %]" value="[% trsfitemloo.tobrcd %]" />
|
|
[% END %]
|
|
<input type="hidden" name="itemnumber" value="[% itemnumber %]" />
|
|
<input type="hidden" name="borrowernumber" value="[% borrowernumber %]" />
|
|
[% IF ( waiting ) %]
|
|
<input type="hidden" name="barcode" value="[% barcode | html %]" />
|
|
<input type="hidden" name="request" value="KillWaiting" />
|
|
<input type="submit" value="Cancel" />
|
|
[% END %]
|
|
[% IF ( reserved ) %]
|
|
<input type="hidden" name="request" value="SetWaiting" />
|
|
<input type="submit" value="Waiting" />
|
|
[% END %]
|
|
</form>
|
|
</td>
|
|
</tr>
|
|
[% IF ( reserved ) %]
|
|
<tr>
|
|
<th>Cancel reservation and then attempt transfer:</th>
|
|
<td>
|
|
<form method="post" name="mainform" id="mainform" action="branchtransfers.pl">
|
|
[% FOREACH trsfitemloo IN trsfitemloop %]
|
|
<input type="hidden" name="bc-[% trsfitemloo.counter %]" value="[% trsfitemloo.barcode %]" />
|
|
<input type="hidden" name="fb-[% trsfitemloo.counter %]" value="[% trsfitemloo.frombrcd %]" />
|
|
<input type="hidden" name="tb-[% trsfitemloo.counter %]" value="[% trsfitemloo.tobrcd %]" />
|
|
[% END %]
|
|
<input type="hidden" name="biblionumber" value="[% biblionumber %]" />
|
|
<input type="hidden" name="borrowernumber" value="[% borrowernumber %]" />
|
|
<input type="hidden" name="tobranchcd" value="[% tobranchcd %]" />
|
|
<input type="hidden" name="barcode" value="[% barcode | html %]" />
|
|
<input type="hidden" name="request" value="KillReserved" />
|
|
<input type="submit" value="Cancel" />
|
|
</form>
|
|
</td>
|
|
</tr>
|
|
[% END %]
|
|
<tr>
|
|
<th>Ignore and return to transfers: </th>
|
|
<td>
|
|
<form method="post" name="mainform" id="mainform" action="branchtransfers.pl" />
|
|
<input type="hidden" name="tobranchcd" value="[% tobranchcd %]" />
|
|
<input type="hidden" name="barcode" />
|
|
[% FOREACH trsfitemloo IN trsfitemloop %]
|
|
<input type="hidden" name="bc-[% trsfitemloo.counter %]" value="[% trsfitemloo.barcode %]" />
|
|
<input type="hidden" name="fb-[% trsfitemloo.counter %]" value="[% trsfitemloo.frombrcd %]" />
|
|
<input type="hidden" name="tb-[% trsfitemloo.counter %]" value="[% trsfitemloo.tobrcd %]" />
|
|
[% END %]
|
|
<input type="submit" value="Ignore" />
|
|
</form>
|
|
</td>
|
|
</tr>
|
|
</table></div>
|
|
|
|
[% ELSE %]
|
|
[% IF ( reqmessage ) %]
|
|
<div class="dialog message">
|
|
<ul>
|
|
[% IF ( cancelled ) %]
|
|
<li>Reserve cancelled</li>
|
|
[% END %]
|
|
[% IF ( setwaiting ) %]
|
|
<li>Item should now be waiting at library: [% reqbrchname %]</li>
|
|
[% END %]
|
|
</ul>
|
|
</div>
|
|
[% END %]
|
|
|
|
[% IF ( errmsgloop ) %]
|
|
<div class="dialog message">
|
|
<ul>
|
|
[% FOREACH errmsgloo IN errmsgloop %]
|
|
[% IF ( errmsgloo.errbadcode ) %]
|
|
<li>No Item with barcode: [% errmsgloo.msg | html %]</li>
|
|
[% END %]
|
|
[% IF ( errmsgloo.errispermanent ) %]
|
|
<li>Please return item to home library: [% Branches.GetName( errmsgloo.msg ) %]</li>
|
|
[% END %]
|
|
[% IF ( errmsgloo.errnotallowed ) %]
|
|
<li>Transfer is not allowed for:
|
|
<ol>
|
|
[% IF ( Koha.Preference('BranchTransferLimitsType') == 'itemtype' ) %]
|
|
<li>Item type: <b>[% ItemTypes.GetDescription( errmsgloo.code ) %]</b></li>
|
|
[% ELSE %]
|
|
<li>Collection code: <b>[% AuthorisedValues.GetByCode( 'CCODE', errmsgloo.code ) %]</b></li>
|
|
[% END %]
|
|
<li>Destination library: <b>[% Branches.GetName( errmsgloo.tbr ) %]</b></li>
|
|
</ol>
|
|
</li>
|
|
[% END %]
|
|
[% IF ( errmsgloo.errdesteqholding ) %]
|
|
<li>Item is already at destination library.</li>
|
|
[% END %]
|
|
[% IF ( errmsgloo.errwasreturned ) %]
|
|
<li>Item was on loan to <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% errmsgloo.patron.borrowernumber %]">
|
|
[% errmsgloo.patron.firstname %] [% errmsgloo.patron.surname %]
|
|
([% errmsgloo.patron.cardnumber %])</a> and has been returned.</li>
|
|
[% END %]
|
|
[% END %]
|
|
</ul>
|
|
</div>
|
|
[% END %]
|
|
|
|
<div id="branchtransfers">
|
|
<form method="post" name="mainform" id="mainform" action="/cgi-bin/koha/circ/branchtransfers.pl">
|
|
<fieldset class="brief">
|
|
<legend>Transfer</legend>
|
|
<ol>
|
|
<li>
|
|
<label for="tobranchcd">Destination library: </label>
|
|
<select name="tobranchcd" id="tobranchcd">
|
|
[% PROCESS options_for_libraries libraries => Branches.all( selected => tobranchcd ) %]
|
|
</select>
|
|
</li>
|
|
<li>
|
|
<label for="barcode">Enter barcode: </label>
|
|
<input name="barcode" id="barcode" size="15" class="focus" /> <input type="submit" value="Submit" />
|
|
</li>
|
|
</ol>
|
|
</fieldset>
|
|
<input type="hidden" name="tobranchcd" value="[% tobrancd %]" />
|
|
[% FOREACH trsfitemloo IN trsfitemloop %]
|
|
<input type="hidden" name="bc-[% trsfitemloo.counter %]" value="[% trsfitemloo.barcode %]" />
|
|
<input type="hidden" name="fb-[% trsfitemloo.counter %]" value="[% trsfitemloo.frombrcd %]" />
|
|
<input type="hidden" name="tb-[% trsfitemloo.counter %]" value="[% trsfitemloo.tobrcd %]" />
|
|
[% END %]
|
|
</form>
|
|
</div>
|
|
|
|
[% IF ( trsfitemloop ) %]
|
|
<div class="yui-g">
|
|
<table>
|
|
<caption>Transferred items</caption>
|
|
<tr>
|
|
<th class="tf-title">Title</th>
|
|
<th class="tf-author">Author</th>
|
|
<th class="tf-barcode">Barcode</th>
|
|
<th class="tf-location">Shelving location</th>
|
|
<th class="tf-itemcallnumber">Call number</th>
|
|
<th class="tf-itemtype">Item type</th>
|
|
<th class="tf-ccode">Collection code</th>
|
|
<th class="tf-destination">Destination</th>
|
|
</tr>
|
|
[% FOREACH trsfitemloo IN trsfitemloop %]
|
|
<tr>
|
|
<td class="tf-title"><a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% trsfitemloo.biblionumber %]">[% trsfitemloo.title |html %]</a></td>
|
|
<td class="tf-author">[% trsfitemloo.author %]</td>
|
|
<td class="tf-barcode"><a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% trsfitemloo.biblionumber %]&itemnumber=[% trsfitemloo.itemnumber %]#item[% trsfitemloo.itemnumber %]">[% trsfitemloo.barcode %]</a></td>
|
|
<td class="tf-location">[% trsfitemloo.location %]</td>
|
|
<td class="tf-itemcallnumber">[% trsfitemloo.itemcallnumber %]</td>
|
|
<td class="tf-itemtype">[% ItemTypes.GetDescription( trsfitemloo.itemtype ) %]</td>
|
|
<td class="tf-ccode">[% AuthorisedValues.GetByCode( 'CCODE', trsfitemloo.ccode ) %]</td>
|
|
<td class="tf-destination">[% Branches.GetName( trsfitemloo.tobrcd ) %]</td>
|
|
</tr>
|
|
[% END %]
|
|
</table>
|
|
</div>
|
|
[% END %]
|
|
|
|
|
|
[% END %]
|
|
</div>
|
|
[% IF Koha.Preference('CircSidebar') %]
|
|
</div>
|
|
<div class="yui-b noprint">
|
|
[% INCLUDE 'circ-nav.inc' %]
|
|
</div>
|
|
[% END %]
|
|
</div>
|
|
[% INCLUDE 'intranet-bottom.inc' %]
|