Attachment 66723 Details for Bug 19004 – Bug 19004: [QA Follow-up] No need to check item-level_itypes again

[patch] Bug 19004: [QA Follow-up] No need to check item-level_itypes again

Bug-19004-QA-Follow-up-No-need-to-check-item-level.patch (text/plain), 1.09 KB, created by Kyle M Hall (khall) on 2017-09-01 12:46:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2017-09-01 12:46:56 UTC

Size: 1.09 KB

patch

obsolete

>From 0aae81a0f8dc062d6ee15d08d0519b2b77e69171 Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Mon, 28 Aug 2017 08:51:24 +0200
>Subject: [PATCH] Bug 19004: [QA Follow-up] No need to check item-level_itypes
> again
>
>As Jonathan pointed out, GetItem already called effective_itemtype.
>So we can just use $item->{itype} here.
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> C4/Circulation.pm | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index b0e26b9..4f4e054 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -1821,9 +1821,7 @@ sub AddReturn {
>     }
> 
>     my $itemnumber = $item->{ itemnumber };
>-    my $itemtype = C4::Context->preference("item-level_itypes")
>-        ? $item->{itype}
>-        : Koha::Biblioitems->find( $item->{biblioitemnumber} )->itemtype;
>+    my $itemtype = $item->{itype}; # GetItem called effective_itemtype
> 
>     my $issue  = Koha::Checkouts->find( { itemnumber => $itemnumber } );
>     if ( $issue ) {
>-- 
>2.10.2

Actions: View | Diff | Splinter Review

Attachments on bug 19004: 65789 | 65790 | 66037 | 66038 | 66486 | 66546 | 66721 | 66722 | 66723