Attachment 157955 Details for Bug 35167 – Bug 35167: Make 'effective_not_for_loan_status' fallback to 0 if itype has it undef

[patch] Bug 35167: Make 'effective_not_for_loan_status' fallback to 0 if itype has it undef

Bug-35167-Make-effectivenotforloanstatus-fallback-.patch (text/plain), 1.36 KB, created by Lucas Gass (lukeg) on 2023-10-26 20:20:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Lucas Gass (lukeg)

Created: 2023-10-26 20:20:57 UTC

Size: 1.36 KB

patch

obsolete

>From a78dea9b5c5a0ec865e2f6428bde8f7c8430b94a Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Thu, 26 Oct 2023 14:58:40 -0300
>Subject: [PATCH] Bug 35167: Make 'effective_not_for_loan_status' fallback to 0
> if itype has it undef
>
>This patch makes the effective not for loan status be set the item value
>if not defined at itype level.
>
>To test:
>1. Apply the regressions tests patch
>2. Run:
>   $ ktd --shell
>  k$ prove t/db_dependent/api/v1/items.t
>=> FAIL: Tests fail!
>3. Apply this patch
>4. Repeat 2
>=> SUCCESS: Tests pass!
>5. Sign off :-D
>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
>---
> Koha/Item.pm | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
>diff --git a/Koha/Item.pm b/Koha/Item.pm
>index 4d84ab0981f..5bcc917afa5 100644
>--- a/Koha/Item.pm
>+++ b/Koha/Item.pm
>@@ -1470,7 +1470,10 @@ sub to_api {
>     my $overrides = {};
> 
>     $overrides->{effective_item_type_id} = $self->effective_itemtype;
>-    $overrides->{effective_not_for_loan_status} = $self->notforloan ? $self->notforloan : $self->itemtype->notforloan;
>+
>+    my $itype_notforloan = $self->itemtype->notforloan;
>+    $overrides->{effective_not_for_loan_status} =
>+        ( defined $itype_notforloan && !$self->notforloan ) ? $itype_notforloan : $self->notforloan;
> 
>     return { %$response, %$overrides };
> }
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 35167: 157948 | 157949 | 157954 | 157955 | 157968 | 157969