Bug 11759: Batch checkout - Add the new template
This patch is the main patch. It adds: - the new template circ/circulation_batch_checkouts.tt - the change into circ/circulation.pl === Why this feature is useful for some libraries? === Currently Koha does not allow to checkout several items at once to a user. This can cause a waste of time when a user want to checkout more than 10 items. This development has been asked to have statistics on in-house use. Every night, a librarian will enter a barcode list (from a file or a scanner : the librarian scans the barcode of the documents that users let on the tables). Barcodes will be added to a statistics patron in order to count the number of items consulted. The development has been expanded to cover another need: some special libraries (BDP for "Bibliotheque departementale de pret": "Loan Regional Library"). These libraries loan items to smaller libraries (municipal). The number of items could be very important and the current way offered by Koha is not adequate. === Test plan === 0/ Apply all patches and execute the DB change entry. 1/ Verify you are abble to do some checkouts (with confirmations, errors). = Statistics way = 2/ Create a statistical patron (e.g. category code STATS). 3/ Update the batch_checkouts pref with the category code (STATS). 4/ Go on the member page and verify the new "Batch check out" tab appears (and only for this user). 5/ Check some items out on the new page using the textarea or a file. 6/ Verify the table contains the item's information and the "Information" column contains "Local use recorded". = Real checkouts = 7/ Create a non statistical patron and update the pref. 8/ Check some items out on the new page using the textarea or a file. 9/ Verify the information in the table is consistent. 10/ Verify item without confirmation needed or impossible alert is checked out. 11/ If the logged in user has the "force_checkout" permission, all checkouts with a "confirmation needed" message can be done. If the logged in user does not have it, there is no way to force the checkout. Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Jonathan Druart
parent
78fedb52e3
commit
76ceaa9f2b
2 changed files with 308 additions and 5 deletions
|
|
@ -90,13 +90,33 @@ if (!C4::Context->userenv && !$branch){
|
|||
}
|
||||
|
||||
my $barcodes = [];
|
||||
my $batch = $query->param('batch');
|
||||
if ( my $barcode = $query->param('barcode') ) {
|
||||
$barcodes = [ $barcode ];
|
||||
} else {
|
||||
my $filefh = $query->upload('uploadfile');
|
||||
if ( $filefh ) {
|
||||
while ( my $content = <$filefh> ) {
|
||||
$content =~ s/[\r\n]*$//g;
|
||||
push @$barcodes, $content if $content;
|
||||
}
|
||||
} elsif ( my $list = $query->param('barcodelist') ) {
|
||||
push @$barcodes, split( /\s\n/, $list );
|
||||
$barcodes = [ map { $_ =~ /^\s*$/ ? () : $_ } @$barcodes ];
|
||||
} else {
|
||||
@$barcodes = $query->param('barcodes');
|
||||
}
|
||||
}
|
||||
|
||||
$barcodes = [ uniq @$barcodes ];
|
||||
|
||||
my $template_name = $batch
|
||||
? q|circ/circulation_batch_checkouts.tt|
|
||||
: q|circ/circulation.tt|;
|
||||
|
||||
my ( $template, $loggedinuser, $cookie ) = get_template_and_user (
|
||||
{
|
||||
template_name => 'circ/circulation.tt',
|
||||
template_name => $template_name,
|
||||
query => $query,
|
||||
type => "intranet",
|
||||
authnotrequired => 0,
|
||||
|
|
@ -325,6 +345,7 @@ if (@$barcodes) {
|
|||
|
||||
if ( $error->{'UNKNOWN_BARCODE'}
|
||||
&& C4::Context->preference("itemBarcodeFallbackSearch")
|
||||
&& not $batch
|
||||
)
|
||||
{
|
||||
$template_params->{FALLBACK} = 1;
|
||||
|
|
@ -404,9 +425,16 @@ if (@$barcodes) {
|
|||
}
|
||||
push @$checkout_infos, $template_params;
|
||||
}
|
||||
|
||||
$template->param( %{$checkout_infos->[0]} );
|
||||
$template->param( barcode => $barcodes->[0] );
|
||||
unless ( $batch ) {
|
||||
$template->param( %{$checkout_infos->[0]} );
|
||||
$template->param( barcode => $barcodes->[0] );
|
||||
} else {
|
||||
my $confirmation_needed = grep { $_->{NEEDSCONFIRMATION} } @$checkout_infos;
|
||||
$template->param(
|
||||
checkout_infos => $checkout_infos,
|
||||
confirmation_needed => $confirmation_needed,
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
# reload the borrower info for the sake of reseting the flags.....
|
||||
|
|
@ -527,6 +555,9 @@ if (C4::Context->preference('ExtendedPatronAttributes')) {
|
|||
extendedattributes => $attributes
|
||||
);
|
||||
}
|
||||
my $view = $batch
|
||||
?'batch_checkout_view'
|
||||
: 'circview';
|
||||
|
||||
my @relatives = GetMemberRelatives( $borrower->{'borrowernumber'} );
|
||||
my $relatives_issues_count =
|
||||
|
|
@ -559,7 +590,7 @@ $template->param(
|
|||
totaldue => sprintf('%.2f', $total),
|
||||
inprocess => $inprocess,
|
||||
is_child => ($borrowernumber && $borrower->{'category_type'} eq 'C'),
|
||||
circview => 1,
|
||||
$view => 1,
|
||||
soundon => C4::Context->preference("SoundOn"),
|
||||
fast_cataloging => $fast_cataloging,
|
||||
CircAutoPrintQuickSlip => C4::Context->preference("CircAutoPrintQuickSlip"),
|
||||
|
|
|
|||
|
|
@ -0,0 +1,272 @@
|
|||
[% USE Branches %]
|
||||
[% USE KohaDates %]
|
||||
[% IF ( export_remove_fields OR export_with_csv_profile ) %]
|
||||
[% SET exports_enabled = 1 %]
|
||||
[% END %]
|
||||
[% USE AuthorisedValues %]
|
||||
[% INCLUDE 'doc-head-open.inc' %]
|
||||
[% SET destination = "circ" %]
|
||||
<title>Koha › Circulation
|
||||
[% IF borrowernumber %]
|
||||
› Checking out a batch to [% INCLUDE 'patron-title.inc' invert_name = 1 %]
|
||||
[% END %]
|
||||
</title>
|
||||
[% INCLUDE 'doc-head-close.inc' %]
|
||||
[% INCLUDE 'calendar.inc' %]
|
||||
[% IF ( UseTablesortForCirc ) %]<link rel="stylesheet" type="text/css" href="[% themelang %]/css/datatables.css" />
|
||||
<script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.dataTables.min.js"></script>
|
||||
[% INCLUDE 'datatables-strings.inc' %]
|
||||
<script type="text/javascript" src="[% themelang %]/js/datatables.js"></script>[% END %]
|
||||
<script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.checkboxes.min.js"></script>
|
||||
<script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery-ui-timepicker-addon.js"></script>
|
||||
<script type="text/javascript" src="[% themelang %]/js/pages/circulation.js"></script>
|
||||
<script type="text/javascript">
|
||||
//<![CDATA[
|
||||
$(document).ready(function() {
|
||||
$("#issuest").dataTable($.extend(true, {}, dataTablesDefaults, {
|
||||
"sDom": 't',
|
||||
"aaSorting": [],
|
||||
"aoColumnDefs": [
|
||||
{ "aTargets": [ -1, -2[% IF ( exports_enabled ) %], -3[% END %] ], "bSortable": false, "bSearchable": false }
|
||||
],
|
||||
"aoColumns": [
|
||||
{ "sType": "title-string" },{ "sType": "html" },null,{ "sType": "title-string" },null,null,null,null,null,null[% IF ( exports_enabled ) %],null[% END %]
|
||||
],
|
||||
"bPaginate": false
|
||||
}));
|
||||
});
|
||||
//]]>
|
||||
</script>
|
||||
</head>
|
||||
<body id="circ_circulation_batch_checkouts" 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/circulation.pl">Checkouts a batch</a> › [% INCLUDE 'patron-title.inc' %]
|
||||
</div>
|
||||
<div id="doc3" class="yui-t2">
|
||||
|
||||
<div id="bd">
|
||||
<div id="yui-main">
|
||||
<div class="yui-b">
|
||||
|
||||
<div class="yui-g">
|
||||
|
||||
[% UNLESS checkout_infos %]
|
||||
<form method="post" enctype="multipart/form-data" action="/cgi-bin/koha/circ/circulation.pl">
|
||||
<fieldset id="circ_circulation_issue">
|
||||
<label for="barcode">Checking out a batch to [% INCLUDE 'patron-title.inc' %]</label>
|
||||
<fieldset class="rows">
|
||||
<legend>Use a file</legend>
|
||||
<ol>
|
||||
<li><label for="uploadfile">File: </label> <input type="file" id="uploadfile" name="uploadfile" /></li>
|
||||
</ol>
|
||||
</fieldset>
|
||||
<fieldset class="rows">
|
||||
<legend>Or list barcodes one by one</legend>
|
||||
<ol>
|
||||
<li>
|
||||
<label for="barcodelist">Barcode list (one barcode per line): </label>
|
||||
<textarea rows="10" cols="30" id="barcodelist" name="barcodelist"></textarea>
|
||||
</li>
|
||||
</ol>
|
||||
</fieldset>
|
||||
<input type="hidden" name="op" value="show" />
|
||||
<fieldset class="action">
|
||||
<input type="hidden" name="borrowernumber" id="borrowernumber" value="[% borrowernumber %]" />
|
||||
<input type="hidden" name="branch" value="[% branch %]" />
|
||||
<input type="hidden" name="batch" value="1" />
|
||||
<input type="submit" value="Check out" class="button" />
|
||||
</fieldset>
|
||||
</fieldset>
|
||||
</form>
|
||||
|
||||
[% ELSE %]
|
||||
[% IF confirmation_needed && CAN_user_circulate_force_checkout %]
|
||||
<form method="post" action="/cgi-bin/koha/circ/circulation.pl" id="mainform" name="mainform" autocomplete="off">
|
||||
[% END %]
|
||||
<table id="checkout_infos">
|
||||
<thead>
|
||||
<tr>
|
||||
[% IF confirmation_needed && CAN_user_circulate_force_checkout %]
|
||||
<th></th>
|
||||
[% END %]
|
||||
<th>Barcode</th>
|
||||
<th>Title</th>
|
||||
<th>Information</th>
|
||||
</tr>
|
||||
</thead>
|
||||
<tbody>
|
||||
[% FOR checkout_info IN checkout_infos %]
|
||||
<tr>
|
||||
[% IF confirmation_needed && CAN_user_circulate_force_checkout %]
|
||||
<td>
|
||||
[% IF checkout_info.NEEDSCONFIRMATION %]
|
||||
<input type="checkbox" name="barcodes" value="[% checkout_info.barcode %]" checked="checked" />
|
||||
[% END %]
|
||||
</td>
|
||||
[% END %]
|
||||
<td>[% checkout_info.barcode %]</td>
|
||||
<td>
|
||||
<a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% checkout_info.item.biblionumber %]&type=intra"><strong>[% checkout_info.item.title |html %][% FOREACH subtitl IN checkout_info.item.subtitle %] [% subtitl.subfield %][% END %]</strong></a>[% IF checkout_info.item.author %], by [% checkout_info.item.author %][% END %][% IF ( checkout_info.itme.itemnotes ) %]- <span class="circ-hlt">[% checkout_info.item.itemnotes %]</span>[% END %] <a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% checkout_info.item.biblionumber %]&itemnumber=[% checkout_info.item.itemnumber %]#item[% checkout_info.item.itemnumber %]">[% checkout_info.item.barcode %]</a>
|
||||
</td>
|
||||
<td>
|
||||
[% IF checkout_info.NEEDSCONFIRMATION %]
|
||||
<p class="warn">
|
||||
[% IF checkout_info.AGE_RESTRICTION %]
|
||||
Age restriction [% checkout_info.AGE_RESTRICTION %]
|
||||
[% END %]
|
||||
[% IF checkout_info.DEBT %]
|
||||
The patron has a debt of [% checkout_info.DEBT %] <!-- Need debt_confirmed -->
|
||||
[% END %]
|
||||
[% IF checkout_info.RENEW_ISSUE %]
|
||||
This Item is currently checked out to this patron. Renew?
|
||||
[% END %]
|
||||
[% IF checkout_info.RESERVE_WAITING %]
|
||||
This Item has been waiting for another patron.
|
||||
[% END %]
|
||||
[% IF checkout_info.RESERVED %]
|
||||
This Item has been on hold for another patron.
|
||||
[% END %]
|
||||
[% IF checkout_info.ISSUED_TO_ANOTHER %]
|
||||
This Item is checked out to another patron.
|
||||
[% IF CAN_user_circulate_force_checkout %]
|
||||
Check in and check out?
|
||||
[% END %]
|
||||
[% END %]
|
||||
[% IF checkout_info.TOO_MANY %]
|
||||
Too many checked out.
|
||||
[% END %]
|
||||
[% IF checkout_info.BORRNOTSAMEBRANCH %]
|
||||
This patrons is from a different library ([% checkout_info.BORRNOTSAMEBRANCH %])</li>
|
||||
[% END %]
|
||||
[% IF checkout_ino.PATRON_CANT %]
|
||||
This patron can't check out this item per library circulation policy.
|
||||
[% END %]
|
||||
[% IF checkout_info.NOT_FOR_LOAN_FORCING %]
|
||||
[% IF checkout_info.itemtype_notforloan %]
|
||||
Item type is normally not for loan.
|
||||
[% ELSIF checkout_info.item_notforloan %]
|
||||
[% item_notforloan_lib = AuthorisedValues.GetByCode( authvalcode_notforloan, checkout_info.item_notforloan, 0 ) %]
|
||||
Item is normally not for loan [% IF item_notforloan_lib %]([% item_notforloan_lib %])[% END %].
|
||||
[% END %]
|
||||
[% END %]
|
||||
[% IF checkout_info.USERBLOCKEDOVERDUE %]
|
||||
Patron has [% checkout_info.USERBLOCKEDOVERDUE %] overdue item(s).
|
||||
[% END %]
|
||||
[% IF checkout_info.ITEM_LOST %]
|
||||
This item has been lost with a status of "[% checkout_info.ITEM_LOST %]".
|
||||
[% END %]
|
||||
[% IF checkout_info.HIGHHOLDS %]
|
||||
High demand item. Loan period shortened to [% checkout_info.HIGHHOLDS.duration %] days (due [% checkout_info.HIGHHOLDS.returndate %]).
|
||||
[% END %]
|
||||
[% IF checkout_info.HIGHHOLDS %] <!-- FIXME -->
|
||||
<script language="JavaScript" type="text/javascript">
|
||||
$(document).ready(function() {
|
||||
$("input[name=duedatespec]:hidden").val('[% checkout_info.HIGHHOLDS.returndate %]');
|
||||
});
|
||||
</script>
|
||||
[% END %]
|
||||
|
||||
[% IF NOT checkout_info.IMPOSSIBLE && ( CAN_user_circulate_force_checkout or checkout_info.HIGHHOLDS ) %]
|
||||
[% IF checkout_info.RESERVED || checkout_info.RESERVE_WAITING %] <!-- arbitrary choice, revert the reserve is not possible-->
|
||||
<p>The hold will be canceled.</p>
|
||||
[% END %]
|
||||
[% END %]
|
||||
</p>
|
||||
[% END %]
|
||||
|
||||
|
||||
[% IF checkout_info.alert.ITEM_LOST || checkout_info.alert.OTHER_CHARGES %]
|
||||
<p class="info">
|
||||
[% IF checkout_info.alert.ITEM_LOST %]
|
||||
This item has been lost with a status of "[% checkout_info.alert.ITEM_LOST %]".
|
||||
[% END %]
|
||||
[% IF checkout_info.alert.OTHER_CHARGES %]
|
||||
The patron has unpaid charges for reserves, rentals etc of [% checkout_info.alert.OTHER_CHARGES %].
|
||||
[% END %]
|
||||
</p>
|
||||
[% END %]
|
||||
|
||||
|
||||
[% IF checkout_info.IMPOSSIBLE %]
|
||||
<p class="error">
|
||||
[% IF checkout_info.STATS %]
|
||||
Local use recorded
|
||||
[% END %]
|
||||
|
||||
[% IF checkout_info.NOT_FOR_LOAN %]
|
||||
[% IF checkout_info.itemtype_notforloan %]
|
||||
Item type not for loan.
|
||||
[% ELSIF checkout_info.item_notforloan %]
|
||||
[% item_notforloan_lib = AuthorisedValues.GetByCode( checkout_info.authvalcode_notforloan, checkout_info.item_notforloan, 0 ) %]
|
||||
Item not for loan [% IF checkout_info.item_notforloan_lib %]([% checkout_info.item_notforloan_lib %])[% END %].
|
||||
[% END %]
|
||||
[% END %]
|
||||
[% IF checkout_info.WTHDRAWN %]
|
||||
Item has been withdrawn
|
||||
[% END %]
|
||||
[% IF checkout_info.RESTRICTED %]
|
||||
Item is restricted
|
||||
[% END %]
|
||||
[% IF checkout_info.GNA %]
|
||||
Patron's address is in doubt
|
||||
[% END %]
|
||||
[% IF checkout_info.CARD_LOST %]
|
||||
Patron's card is lost
|
||||
[% END %]
|
||||
[% IF checkout_info.DEBARRED %]
|
||||
Patron is restricted
|
||||
[% END %]
|
||||
[% IF checkout_info.NO_MORE_RENEWALS %]
|
||||
No more renewals possible
|
||||
[% END %]
|
||||
[% IF checkout_info.EXPIRED %]
|
||||
Patron's card is expired
|
||||
[% END %]
|
||||
[% IF checkout_info.ITEMNOTSAMEBRANCH %]
|
||||
This item belongs to [% Branches.GetName( checkout_info.itemhomebranch ) %] and cannot be checked out from this location.
|
||||
[% END %]
|
||||
[% IF checkout_info.USERBLOCKEDREMAINING %]
|
||||
Patron has had overdue items and is blocked for [% checkout_info.USERBLOCKEDREMAINING %] day(s).
|
||||
[% END %]
|
||||
[% IF checkout_info.USERBLOCKEDOVERDUE %]
|
||||
Checkouts are BLOCKED because patron has overdue items
|
||||
[% END %]
|
||||
[% IF checkout_info.TOO_MANY %]
|
||||
Too many checked out.
|
||||
[% END %]
|
||||
[% IF checkout_info.UNKNOWN_BARCODE %]
|
||||
<li>The barcode was not found [% checkout_info.barcode |html %]
|
||||
[% END %]
|
||||
</p>
|
||||
[% END %]
|
||||
</td>
|
||||
</tr>
|
||||
[% END %]
|
||||
</tbody>
|
||||
</table>
|
||||
|
||||
[% IF confirmation_needed && CAN_user_circulate_force_checkout %]
|
||||
<h3>Please confirm checkout</h3>
|
||||
<input type="hidden" name="borrowernumber" value="[% borrowernumber %]" />
|
||||
<input type="hidden" name="issueconfirmed" value="1" />
|
||||
<input type="hidden" name="debt_confirmed" value="1" />
|
||||
<input type="hidden" name="branch" value="[% branch %]" />
|
||||
<input type="hidden" name="batch" value="1" />
|
||||
<input type="submit" class="approve" value="Checkout or renew" />
|
||||
</form>
|
||||
[% END %]
|
||||
|
||||
[% END %]
|
||||
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
<div class="yui-b">
|
||||
[% INCLUDE 'circ-menu.inc' %]
|
||||
</div>
|
||||
</div>
|
||||
[% INCLUDE 'intranet-bottom.inc' %]
|
||||
Loading…
Reference in a new issue