From d1ce272dd2a5c75862db16d2b0682b0a9d654565 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 4 Aug 2015 10:38:29 +0100 Subject: [PATCH] Bug 14470: Do not allow renew for on-site checkouts At the opac, the renew checkbox should not be displayed if it's an on-site checkout (same on the intranet). On the way, this patch adds a specific message to the intranet if the librarian try to renew an on-site checkout. Indeed before this patch a renew was allowed if the barcode was scanned. Test plan: 1/ Create an on-site checkout for a patron 2/ Confirm that the checkbox 'renew' is not displayed on the checkout list tables 3/ At the OPAC, the renew should not be allowed (no checkbox) 4/ Try to check the item out to the same patron, confirm that you get a specifig message to inform you the renew is not allowed for on-site checkouts. Signed-off-by: Mirko Tietgen Signed-off-by: Katrin Fischer Changed 'issue' to 'item' in the error message. Signed-off-by: Tomas Cohen Arazi (cherry picked from commit ea92a92f5384c3256de55b770d31342f796edc47) Signed-off-by: Chris Cormack Conflicts: t/db_dependent/Circulation.t --- C4/Circulation.pm | 8 +++- .../intranet-tmpl/prog/en/js/checkouts.js | 26 ++++++------ .../prog/en/modules/circ/circulation.tt | 4 ++ t/db_dependent/Circulation.t | 42 ++++++++++++++++++- 4 files changed, 65 insertions(+), 15 deletions(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index b127eead5d..ce586f9785 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -938,7 +938,12 @@ sub CanBookBeIssued { $item->{'itemnumber'} ); if ( $CanBookBeRenewed == 0 ) { # no more renewals allowed - $issuingimpossible{NO_MORE_RENEWALS} = 1; + if ( $renewerror eq 'onsite_checkout' ) { + $issuingimpossible{NO_RENEWAL_FOR_ONSITE_CHECKOUTS} = 1; + } + else { + $issuingimpossible{NO_MORE_RENEWALS} = 1; + } } else { $needsconfirmation{RENEW_ISSUE} = 1; @@ -2662,6 +2667,7 @@ sub CanBookBeRenewed { my $item = GetItem($itemnumber) or return ( 0, 'no_item' ); my $itemissue = GetItemIssue($itemnumber) or return ( 0, 'no_checkout' ); + return ( 0, 'onsite_checkout' ) if $itemissue->{onsite_checkout}; $borrowernumber ||= $itemissue->{borrowernumber}; my $borrower = C4::Members::GetMember( borrowernumber => $borrowernumber ) diff --git a/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js b/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js index 23a84264d5..485c29216e 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js +++ b/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js @@ -328,6 +328,8 @@ $(document).ready(function() { span_style = "display: none"; span_class = "renewals-allowed"; + } else if ( oObj.can_renew_error == "onsite_checkout" ) { + // Don't display something if it's an onsite checkout } else { content += "" + oObj.can_renew_error @@ -339,20 +341,18 @@ $(document).ready(function() { var can_force_renew = ( oObj.onsite_checkout == 0 ) && ( oObj.can_renew_error != "on_reserve" ); var can_renew = ( oObj.renewals_remaining > 0 && !oObj.can_renew_error ); - if ( oObj.onsite_checkout == 0 ) { - if ( can_renew || can_force_renew ) { - content += "" - + "" - + ""; - - content += "(" - + RENEWALS_REMAINING.format( oObj.renewals_remaining, oObj.renewals_allowed ) - + ")"; + if ( can_renew || can_force_renew ) { + content += "" + + "" + + ""; + + content += "(" + + RENEWALS_REMAINING.format( oObj.renewals_remaining, oObj.renewals_allowed ) + + ")"; } content += ""; diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt index 2a2f43dfa8..ba452b71f6 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt @@ -459,6 +459,10 @@ $(document).ready(function() {
  • No more renewals possible
  • [% END %] + [% IF NO_RENEWAL_FOR_ONSITE_CHECKOUTS %] +
  • This item can not be renewed, it's an on-site checkout
  • + [% END %] + [%IF ( AGE_RESTRICTION ) %]
  • Age restriction [% AGE_RESTRICTION %].
  • [% END %] diff --git a/t/db_dependent/Circulation.t b/t/db_dependent/Circulation.t index 30a5fe78e6..bb453eaf6f 100755 --- a/t/db_dependent/Circulation.t +++ b/t/db_dependent/Circulation.t @@ -27,7 +27,7 @@ use C4::Overdues qw(UpdateFine); use Koha::DateUtils; use Koha::Database; -use Test::More tests => 61; +use Test::More tests => 69; BEGIN { use_ok('C4::Circulation'); @@ -596,4 +596,44 @@ C4::Context->dbh->do("DELETE FROM accountlines"); $dbh->rollback; +<<<<<<< HEAD +======= +{ + # Don't allow renewing onsite checkout + my $barcode = 'R00000XXX'; + my $branch = 'CPL'; + + #Create another record + my $biblio = MARC::Record->new(); + $biblio->append_fields( + MARC::Field->new('100', ' ', ' ', a => 'Anonymous'), + MARC::Field->new('245', ' ', ' ', a => 'A title'), + ); + my ($biblionumber, $biblioitemnumber) = AddBiblio($biblio, ''); + + my (undef, undef, $itemnumber) = AddItem( + { + homebranch => $branch, + holdingbranch => $branch, + barcode => $barcode, + }, + $biblionumber + ); + + my $borrowernumber = AddMember( + firstname => 'fn', + surname => 'dn', + categorycode => 'S', + branchcode => $branch, + ); + + my $borrower = GetMember( borrowernumber => $borrowernumber ); + my $issue = AddIssue( $borrower, $barcode, undef, undef, undef, undef, { onsite_checkout => 1 } ); + my ( $renewed, $error ) = CanBookBeRenewed( $borrowernumber, $itemnumber ); + is( $renewed, 0, 'CanBookBeRenewed should not allow to renew on-site checkout' ); + is( $error, 'onsite_checkout', 'A correct error code should be returned by CanBookBeRenewed for on-site checkout' ); +} + +$schema->storage->txn_rollback(); +>>>>>>> ea92a92... Bug 14470: Do not allow renew for on-site checkouts 1; -- 2.39.5