From 15c85e3216de83767ee88b6b2bb5237a5366cb41 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 | 9 ++++++++ koha-tmpl/intranet-tmpl/prog/js/holds.js | 21 +++++++++++++++++++ reserve/modrequest.pl | 16 +++++++++++++- reserve/request.pl | 11 ++++++++++ 4 files changed, 56 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..706a321695 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,15 @@ No barcode [%- END -%] + [%- IF hold.change_hold_type_allowed -%] + [%- IF hold.itemtype -%] + Next available [% ItemTypes.GetDescription( hold.itemtype ) | html %] item + [%- ELSE -%] + Next available + [%- END -%] + + + [%- 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 58ecb622c2..27df855190 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/holds.js +++ b/koha-tmpl/intranet-tmpl/prog/js/holds.js @@ -441,4 +441,25 @@ $(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 reserve_id = $(this).data('reserve-id'); + let change_type = $("input#change_hold_type_" + reserve_id).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_" + reserve_id).val("0"); + next_available.hide(); + only_itemlink.show(); + only_item.show(); + } else { + $("input#change_hold_type_" + reserve_id).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..bd733dc38e 100755 --- a/reserve/modrequest.pl +++ b/reserve/modrequest.pl @@ -94,7 +94,21 @@ else { } else { $_->rethrow; } - } + }; + + if ( $query->param( "change_hold_type_" . $reserve_id[$i] ) ) { + 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 2c28f2c58f..93abe66a4b 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