Attachment 142299 Details for Bug 31644 – Bug 31644: Check if subfield defined, not for truth

[patch] Bug 31644: Check if subfield defined, not for truth

Bug-31644-Check-if-subfield-defined-not-for-truth.patch (text/plain), 1.65 KB, created by Marcel de Rooy on 2022-10-21 07:46:20 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2022-10-21 07:46:20 UTC

Size: 1.65 KB

patch

obsolete

>From 626bcc8f1c478c26be8fd483f323292619e74c34 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Wed, 28 Sep 2022 18:53:28 +0000
>Subject: [PATCH] Bug 31644: Check if subfield defined, not for truth
>MIME-Version: 1.0
>Content-Type: text/plain; charset=utf-8
>Content-Transfer-Encoding: 8bit
>Content-Type: text/plain; charset=utf-8
>
>To test:
>1 - Define a new MARC Modification template with actions:
>        Copy field 600$a to 942$0
>2 - Define a new record like:
>    LDR 00334nam a22001217a 4500
>    003 ff
>    005 20201102111604.0
>    008 201102b        xxu||||| |||| 00| 0 eng d
>    040 _ _ â¡cvsd
>    100 1 _ â¡012345â¡aKnuth, Donal Ervinâ¡d1938
>    245 _ _ â¡012345â¡aThe aty of computer programmingâ¡cDonald E. Knuth
>    600 _ 0 â¡042â¡aComputer programmingâ¡9462
>3 - Modify this record using the template above
>4 - Note that entire  field is copied to 942
>5 - Apply patch
>6 - Now only subfield 0 is copied
>
>Signed-off-by: David Nind <david@davidnind.com>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> Koha/SimpleMARC.pm | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/Koha/SimpleMARC.pm b/Koha/SimpleMARC.pm
>index 6fd08b1013..5aab1285d7 100644
>--- a/Koha/SimpleMARC.pm
>+++ b/Koha/SimpleMARC.pm
>@@ -92,9 +92,9 @@ sub copy_field {
>     if ( ! ( $record && $fromFieldName && $toFieldName ) ) { return; }
> 
> 
>-    if (   not $fromSubfieldName
>+    if (   not defined $fromSubfieldName
>         or $fromSubfieldName eq ''
>-        or not $toSubfieldName
>+        or not defined $toSubfieldName
>         or $toSubfieldName eq '' ) {
>         _copy_move_field(
>             {   record        => $record,
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 31644: 141066 | 141091 | 141092 | 141093 | 141936 | 141937 | 141938 | 142298 | 142299 | 142300 | 142301