Attachment 140478 Details for Bug 30921 – Bug 30921: (QA follow-up) Remove unnecessary if

[patch] Bug 30921: (QA follow-up) Remove unnecessary if

Bug-30921-QA-follow-up-Remove-unnecessary-if.patch (text/plain), 1008 bytes, created by Joonas Kylmälä on 2022-09-12 16:30:58 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Joonas Kylmälä

Created: 2022-09-12 16:30:58 UTC

Size: 1008 bytes

patch

obsolete

>From ac3754c7974b8a854a2580cfd01655e64cbead7b Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Joonas=20Kylm=C3=A4l=C3=A4?= <joonas.kylmala@iki.fi>
>Date: Mon, 12 Sep 2022 16:24:31 +0000
>Subject: [PATCH] Bug 30921: (QA follow-up) Remove unnecessary if
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>The statement is already inside an if block with the same condition.
>
>Signed-off-by: Joonas KylmÃ¤lÃ¤ <joonas.kylmala@iki.fi>
>---
> Koha/OAI/Server/Repository.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/Koha/OAI/Server/Repository.pm b/Koha/OAI/Server/Repository.pm
>index 886131318b..0cd0df5dcf 100644
>--- a/Koha/OAI/Server/Repository.pm
>+++ b/Koha/OAI/Server/Repository.pm
>@@ -190,7 +190,7 @@ sub get_biblio_marcxml {
>                 }
>             }
>         );
>-        $record_processor->process($record) if $expanded_avs;
>+        $record_processor->process($record);
>     }
> 
>     return $record ? $record->as_xml_record() : undef;
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 30921: 135778 | 140350 | 140354 | 140477 | 140478