Attachment 157949 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.31 KB, created by Tomás Cohen Arazi (tcohen) on 2023-10-26 18:59:39 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2023-10-26 18:59:39 UTC

Size: 1.31 KB

patch

obsolete

>From a007c9f6526dd2e7260199d0e954de415bf929ad 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>
>---
> 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.42.0

Actions: View | Diff | Splinter Review

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