Attachment 165916 Details for Bug 27753 – Bug 27753: Follow up updates to C4/Circulation.pm

[patch] Bug 27753: Follow up updates to C4/Circulation.pm

Bug-27753-Follow-up-updates-to-C4Circulationpm.patch (text/plain), 2.08 KB, created by Laura Escamilla on 2024-04-30 16:06:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Laura Escamilla

Created: 2024-04-30 16:06:06 UTC

Size: 2.08 KB

patch

obsolete

>From a81b3b8e5c55082ad96645eda0eb3553f21dc5ba Mon Sep 17 00:00:00 2001
>From: Laura Escamilla <laura.escamilla@bywatersolutions.com>
>Date: Tue, 30 Apr 2024 14:21:35 +0000
>Subject: [PATCH] Bug 27753: Follow up updates to C4/Circulation.pm
>
>---
> C4/Circulation.pm | 28 ++++++++++++++--------------
> 1 file changed, 14 insertions(+), 14 deletions(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index 5f642c2f90..a83e59671c 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -2535,29 +2535,29 @@ sub AddReturn {
>     }
> 
>     if ( C4::Context->preference('ClaimReturnedLostValue') ) {
>-        my $autoClaimReturnCheckin = C4::Context->preference('AutoClaimReturnStatusOnCheckin');
> 
>-        if ($autoClaimReturnCheckin) {
>             my $claim = Koha::Checkouts::ReturnClaims->find(
>                 {
>                     itemnumber => $item->id,
>                     resolution => undef,
>                 }
>             );
>-
>             if ($claim) {
>-                my $patron_id  = $patron->borrowernumber;
>-                my $resolution = $autoClaimReturnCheckin;
>+                my $autoClaimReturnCheckin = C4::Context->preference('AutoClaimReturnStatusOnCheckin');
>+                if ($autoClaimReturnCheckin) {
> 
>-                $claim->resolve(
>-                    {
>-                        resolution  => $resolution,
>-                        resolved_by => $patron_id,
>-                    }
>-                );
>-                $messages->{ClaimAutoResolved} = $claim;
>-            } else {
>-                $messages->{ReturnClaims} = $claim if $claim;
>+                    my $patron_id  = $patron->borrowernumber;
>+                    my $resolution = $autoClaimReturnCheckin;
>+
>+                    $claim->resolve(
>+                        {
>+                            resolution  => $resolution,
>+                            resolved_by => $patron_id,
>+                        }
>+                    );
>+                    $messages->{ClaimAutoResolved} = $claim;
>+                } else {
>+                    $messages->{ReturnClaims} = $claim;
>             }
>         }
>     }
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 27753: 162517 | 163058 | 163059 | 163060 | 163061 | 163084 | 163085 | 163090 | 163469 | 163470 | 163474 | 163475 | 163476 | 163477 | 163479 | 163480 | 165911 | 165912 | 165914 | 165915 | 165916 | 165917 | 165918 | 165919 | 165920 | 165952 | 165953 | 165958 | 165959 | 166128 | 166129 | 166130 | 166131 | 166132 | 166133 | 166134 | 166135 | 166136 | 166137