From 178c5d6e6406887059f035ee7da988b5ba3603f4 Mon Sep 17 00:00:00 2001 From: Lari Taskula Date: Wed, 5 Oct 2022 16:04:59 +0000 Subject: [PATCH] Bug 31692: Allow librarians to change hold type in staff client To test: 1. Apply patch 2. Add item level hold to a record/item, make sure patron has no other holds on that record 3. Go to /cgi-bin/koha/reserve/request.pl?biblionumber=xxx where xxx is the record you placed the hold for 4. Under "Existing holds" table, in "Details" column you should see "Only item X". Click the red X and click Update hold(s) 5. Observe "Only item " has changed to "Next available" 6. Cancel the hold and add two item level holds 7. Under "Existing holds" table, in "Details" column you should see "Only item ", but no red X for either holds. --- .../prog/en/includes/holds_table.inc | 5 +++++ koha-tmpl/intranet-tmpl/prog/js/holds.js | 20 +++++++++++++++++++ reserve/modrequest.pl | 18 ++++++++++++++++- reserve/request.pl | 11 ++++++++++ 4 files changed, 53 insertions(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc index 536bd16484..fe8c9e3626 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc @@ -175,6 +175,11 @@ No barcode [%- END -%] + [%- IF hold.change_hold_type_allowed -%] + Next available + + + [%- END -%] [%- ELSE -%] [%- IF hold.itemtype -%] diff --git a/koha-tmpl/intranet-tmpl/prog/js/holds.js b/koha-tmpl/intranet-tmpl/prog/js/holds.js index 964528a33d..9e9ae3499a 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/holds.js +++ b/koha-tmpl/intranet-tmpl/prog/js/holds.js @@ -441,4 +441,24 @@ $(document).ready(function() { window.location.href='request.pl?action=toggleSuspend&reserve_id=' + reserve_id + '&biblionumber=' + biblionumber + '&suspend_until=' + suspend_until; return false; }); + + $("a#change_hold_type_icon").on('click', function(e) { + e.preventDefault(); + let change_type = $("input#change_hold_type").val(); + let only_item = $(this).siblings("span:first"); + let only_itemlink = $(this).siblings("a:first"); + let next_available = $(this).siblings("span:last"); + if (change_type==="1") { + $("input#change_hold_type").val("0"); + next_available.hide(); + only_itemlink.show(); + only_item.show(); + } else { + $("input#change_hold_type").val("1"); + only_itemlink.hide(); + only_item.hide(); + next_available.show(); + } + return false; + }); }); diff --git a/reserve/modrequest.pl b/reserve/modrequest.pl index ec78f683be..7772b5655e 100755 --- a/reserve/modrequest.pl +++ b/reserve/modrequest.pl @@ -59,6 +59,8 @@ my $CancelBiblioNumber = $query->param('CancelBiblioNumber'); my $CancelBorrowerNumber = $query->param('CancelBorrowerNumber'); my $CancelItemnumber = $query->param('CancelItemnumber'); +my $change_hold_type = $query->param('changeHoldType'); + # 2 possibilitys : cancel an item reservation, or modify or cancel the queded list # 1) cancel an item reservation by function ModReserveCancelAll (in reserves.pm) @@ -94,7 +96,21 @@ else { } else { $_->rethrow; } - } + }; + + if ( $change_hold_type ) { + my $hold = Koha::Holds->find( $reserve_id[$i] ); + + try { + $hold->change_type; + } catch { + if ($_->isa('Koha::Exceptions::Hold::CannotChangeHoldType')){ + warn $_; + } else { + $_->rethrow; + } + } + }; } my @biblio_ids = uniq @biblionumber; Koha::BackgroundJob::BatchUpdateBiblioHoldsQueue->new->enqueue( diff --git a/reserve/request.pl b/reserve/request.pl index 5fcb7ad0b2..21306e0236 100755 --- a/reserve/request.pl +++ b/reserve/request.pl @@ -593,6 +593,13 @@ if ( ( $findborrower && $borrowernumber_hold || $findclub && $club_hold ) $template->param( always_show_holds => $always_show_holds ); my $show_holds_now = $input->param('show_holds_now'); unless( (defined $always_show_holds && $always_show_holds eq 'DONT') && !$show_holds_now ){ + my $holds_count_per_patron = { map { $_->{borrowernumber} => $_->{hold_count} } + @{ Koha::Holds->search( { biblionumber=> $biblionumber }, { + select => [ "borrowernumber", { count => { distinct => "reserve_id" } } ], + as => [ qw( borrowernumber hold_count ) ], + group_by => [ qw( borrowernumber ) ] } + )->unblessed + } }; my @reserves = Koha::Holds->search( { biblionumber => $biblionumber }, { order_by => 'priority' } )->as_list; foreach my $res ( sort { @@ -643,6 +650,10 @@ if ( ( $findborrower && $borrowernumber_hold || $findclub && $club_hold ) $reserve{non_priority} = $res->non_priority(); $reserve{object} = $res; + if ( $holds_count_per_patron->{ $reserve{'borrowernumber'} } == 1 ) { + $reserve{'change_hold_type_allowed'} = 1; + } + push( @reserveloop, \%reserve ); } } -- 2.25.1