From 94452c9d7eea40a102d7f32fe63b8a067269de12 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 5 Nov 2013 12:32:30 +0100 Subject: [PATCH] Bug 11201: Add a in-house use list pages The circulation page has a new entry: a link to a list of the pending in-house use. Bug 10860 introduces a new way for managing in-house uses. This patch adds a new page (from the circulation home page) to list all pending in-house uses. Test plan: Go on the circulation home page and click on the in-house use link. Verify all your in-house uses are listed and information are consistent. --- C4/Circulation.pm | 31 ++++++++ circ/in-house_use.pl | 43 +++++++++++ .../prog/en/modules/circ/circulation-home.tt | 1 + .../prog/en/modules/circ/in-house_use.tt | 80 ++++++++++++++++++++ t/db_dependent/Circulation/GetIssues.t | 36 ++++----- 5 files changed, 169 insertions(+), 22 deletions(-) create mode 100755 circ/in-house_use.pl create mode 100644 koha-tmpl/intranet-tmpl/prog/en/modules/circ/in-house_use.tt diff --git a/C4/Circulation.pm b/C4/Circulation.pm index ec38281..ec313e8 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -70,6 +70,7 @@ BEGIN { &barcodedecode &LostItem &ReturnLostItem + &GetPendingInHouseUses ); # subs to deal with issuing a book @@ -3611,6 +3612,36 @@ sub IsItemIssued { 1; +=head2 GetPendingInHouseUses + +=cut + +sub GetPendingInHouseUses { + my $dbh = C4::Context->dbh; + return $dbh->selectall_arrayref(q| + SELECT + items.barcode, + items.biblionumber, + items.itemnumber, + items.itemnotes, + items.itemcallnumber, + items.location, + issues.date_due, + issues.branchcode, + biblio.author, + biblio.title, + borrowers.firstname, + borrowers.surname, + borrowers.cardnumber, + borrowers.borrowernumber + FROM items + LEFT JOIN issues ON items.itemnumber = issues.itemnumber + LEFT JOIN biblio ON items.biblionumber = biblio.biblionumber + LEFT JOIN borrowers ON issues.borrowernumber = borrowers.borrowernumber + WHERE issues.inhouse_use = 1 + |, { Slice => {} } ); +} + __END__ =head1 AUTHOR diff --git a/circ/in-house_use.pl b/circ/in-house_use.pl new file mode 100755 index 0000000..1d6599e --- /dev/null +++ b/circ/in-house_use.pl @@ -0,0 +1,43 @@ +#!/usr/bin/perl +# This file is part of Koha. +# +# Copyright (C) 2013 BibLibre +# +# Koha is free software; you can redistribute it and/or modify it +# under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# Koha is distributed in the hope that it will be useful, but +# WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with Koha; if not, see . + +use Modern::Perl; + +use C4::Auth; +use C4::Circulation qw( GetPendingInHouseUses ); +use C4::Output; + +my $cgi = new CGI; + +my ( $template, $loggedinuser, $cookie ) = get_template_and_user( + { + template_name => "circ/in-house_use.tt", + query => $cgi, + type => "intranet", + authnotrequired => 0, + flagsrequired => {circulate => "circulate_remaining_permissions"}, + } +); + +my $pending_inhouse_use = C4::Circulation::GetPendingInHouseUses(); + +$template->param( + pending_inhouse_use => $pending_inhouse_use, +); + +output_html_with_http_headers $cgi, $cookie, $template->output; diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation-home.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation-home.tt index e5eaddc..f2f241a 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation-home.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation-home.tt @@ -46,6 +46,7 @@ systems with large numbers of overdue items.[% END %]
  • Overdues with fines - Limited to your library. See report help for other details.
  • +
  • In-house use list
  • diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/in-house_use.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/in-house_use.tt new file mode 100644 index 0000000..331d050 --- /dev/null +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/in-house_use.tt @@ -0,0 +1,80 @@ +[% USE KohaBranchName %] +[% USE KohaDates %] +[% INCLUDE 'doc-head-open.inc' %] +Koha › Circulation › In-house use +[% INCLUDE 'doc-head-close.inc' %] + +[% INCLUDE 'datatables.inc' %] + + + + +[% INCLUDE 'header.inc' %] +[% INCLUDE 'circ-search.inc' %] + + + +
    +
    +
    +
    +

    Pending in-house use list

    + [% IF pending_inhouse_use %] + + + + + + + + [% FOREACH item IN pending_inhouse_use %] + + + + + + + + + + [% END %] + +
    DatePatronTitleCallnumberBarcodeLibraryLocation
    [% item.date_due | $KohaDates %] + [%item.firstname %] [% item.surname %] + + [% item.title |html %][% IF ( item.author ) %], by [% item.author %][% END %][% IF ( item.itemnotes ) %]- [% item.itemnotes %][% END %] + [% item.itemcallnumber %] + [% item.barcode %] + [% item.branchcode | $KohaBranchName%][% item.location %]
    + [% ELSE %] +

    No pending in-house use.

    + [% END %] +
    +
    +
    + [% INCLUDE 'circ-menu.inc' %] +
    +
    +[% INCLUDE 'intranet-bottom.inc' %] diff --git a/t/db_dependent/Circulation/GetIssues.t b/t/db_dependent/Circulation/GetIssues.t index 36607d4..dcd0ec2 100644 --- a/t/db_dependent/Circulation/GetIssues.t +++ b/t/db_dependent/Circulation/GetIssues.t @@ -2,7 +2,7 @@ use Modern::Perl; -use Test::More; +use Test::More tests => 10; use Test::MockModule; use C4::Biblio; use C4::Items; @@ -12,6 +12,12 @@ use C4::Category; use C4::Circulation; use MARC::Record; +my $dbh = C4::Context->dbh; +$dbh->{AutoCommit} = 0; +$dbh->{RaiseError} = 1; + +$dbh->do(q|DELETE FROM issues|); + my $branchcode; my $branch_created; my @branches = keys %{ GetBranches() }; @@ -78,24 +84,10 @@ is(scalar @$issues, 1, "The other is an item from biblio $biblionumber2"); $issues = GetIssues({itemnumber => $itemnumber2}); is(scalar @$issues, 0, "No one has issued the second item of biblio $biblionumber2"); -END { - AddReturn('0101', $branchcode); - AddReturn('0203', $branchcode); - DelMember($borrowernumber); - if ($category_created) { - C4::Context->dbh->do( - "DELETE FROM categories WHERE categorycode = ?", undef, $categorycode); - } - my $dbh = C4::Context->dbh; - C4::Items::DelItem($dbh, $biblionumber1, $itemnumber1); - C4::Items::DelItem($dbh, $biblionumber1, $itemnumber2); - C4::Items::DelItem($dbh, $biblionumber2, $itemnumber3); - C4::Biblio::DelBiblio($biblionumber1); - C4::Biblio::DelBiblio($biblionumber2); - - if ($branch_created) { - DelBranch($branchcode); - } -}; - -done_testing; +my $inhouse_uses = GetPendingInHouseUses; +is( scalar @$inhouse_uses, 0, "No pending in house use" ); + +my $itemnumber4 = AddItem({ barcode => '0104', %item_branch_infos }, $biblionumber1); +AddIssue( $borrower, '0104', undef, undef, undef, undef, { inhouse_use => 1 } ); +$inhouse_uses = GetPendingInHouseUses; +is( scalar @$inhouse_uses, 1, "There is 1 pending in house use" ); -- 1.7.10.4