Attachment 123693 Details for Bug 25619 – Bug 25619: Add ability to adjust expiration date for waiting holds

[patch] Bug 25619: Add ability to adjust expiration date for waiting holds

Bug-25619-Add-ability-to-adjust-expiration-date-fo.patch (text/plain), 2.10 KB, created by Katrin Fischer on 2021-08-10 13:59:55 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2021-08-10 13:59:55 UTC

Size: 2.10 KB

patch

obsolete

>From e073afc3e6c28c3015c9ccda8f57e6e968a6e3ef Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Fri, 16 Jul 2021 13:50:29 -0400
>Subject: [PATCH] Bug 25619: Add ability to adjust expiration date for waiting
> holds
>
>There are times when an item that is waiting for pickup needs to have the expiration date extended. This would give staff the ability to modify one by one, as needed, the reserves.expirationdate for a given item awaiting pickup.
>
>Test Plan:
>1) Place a hold, trap an item for it such that is is waiting
>2) Attempt to update the expiration date
>3) Note the new date is not saved
>4) Apply this patch, restart all the things!
>5) Attempt to update the expiration date
>6) The new date should be saved!
>
>Signed-off-by: Abbey Holt <aholt@dubuque.lib.ia.us>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> C4/Reserves.pm | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
>diff --git a/C4/Reserves.pm b/C4/Reserves.pm
>index 1d5ec22db2..e778031098 100644
>--- a/C4/Reserves.pm
>+++ b/C4/Reserves.pm
>@@ -1031,7 +1031,6 @@ sub ModReserve {
>     my $biblionumber = $params->{'biblionumber'};
>     my $cancellation_reason = $params->{'cancellation_reason'};
> 
>-    return if $rank eq "W";
>     return if $rank eq "n";
> 
>     return unless ( $reserve_id || ( $borrowernumber && ( $biblionumber || $itemnumber ) ) );
>@@ -1049,6 +1048,16 @@ sub ModReserve {
>     if ( $rank eq "del" ) {
>         $hold->cancel({ cancellation_reason => $cancellation_reason });
>     }
>+    elsif ($hold->found && $hold->priority eq '0') {
>+        logaction( 'HOLDS', 'MODIFY', $hold->reserve_id, Dumper($hold->unblessed) )
>+            if C4::Context->preference('HoldsLog');
>+
>+        # The only column that can be updated for a found hold is the expiration date
>+        my $date = $params->{expirationdate};
>+        if ($date) {
>+            $hold->expirationdate(dt_from_string($date))->store();
>+        }
>+    }
>     elsif ($rank =~ /^\d+/ and $rank > 0) {
>         logaction( 'HOLDS', 'MODIFY', $hold->reserve_id, Dumper($hold->unblessed) )
>             if C4::Context->preference('HoldsLog');
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 25619: 122922 | 122923 | 122924 | 122929 | 122930 | 123043 | 123693 | 123694 | 123695 | 123779 | 124120 | 124121 | 124122 | 124123 | 124124