Attachment 174560 Details for Bug 37398 – Bug 37398: Make sure subfield and subfield_date are defined

[patch] Bug 37398: Make sure subfield and subfield_date are defined

Bug-37398-Make-sure-subfield-and-subfielddate-are-.patch (text/plain), 2.01 KB, created by Marcel de Rooy on 2024-11-15 10:41:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2024-11-15 10:41:57 UTC

Size: 2.01 KB

patch

obsolete

>From f075515199bb5f92efbfb5528e1ef37290bff1a4 Mon Sep 17 00:00:00 2001
>From: Lucas Gass <lucas@bywatersolutions.com>
>Date: Thu, 12 Sep 2024 16:43:53 +0000
>Subject: [PATCH] Bug 37398: Make sure subfield and subfield_date are defined
>Content-Type: text/plain; charset=utf-8
>
>Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> Koha/UI/Form/Builder/Item.pm | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/Koha/UI/Form/Builder/Item.pm b/Koha/UI/Form/Builder/Item.pm
>index 50db1a3e3a..f426ece8dd 100644
>--- a/Koha/UI/Form/Builder/Item.pm
>+++ b/Koha/UI/Form/Builder/Item.pm
>@@ -147,11 +147,11 @@ sub generate_subfield_form {
>         }
>     }
> 
>-    if ($subfield->{kohafield} =~ /items\.(replacementpricedate|dateaccessioned|datelastborrowed|onloan)/) {
>+    if ( defined $subfield->{kohafield}  && $subfield->{kohafield} =~ /items\.(replacementpricedate|dateaccessioned|datelastborrowed|onloan)/) {
>         $subfield_data{datetype} = 'date';
>     }
> 
>-    if ($subfield->{kohafield} =~ /items\.(datelastseen|damaged_on|itemlost_on|withdrawn_on|deleted_on)/) {
>+    if ( defined $subfield->{kohafield} && $subfield->{kohafield} =~ /items\.(datelastseen|damaged_on|itemlost_on|withdrawn_on|deleted_on)/) {
>         $subfield_data{datetype} = 'datetime';
>     }
> 
>@@ -323,14 +323,14 @@ sub generate_subfield_form {
>             authtypecode => $subfield->{authtypecode},
>         };
>     }
>-    elsif ( $subfield_data{datetype} eq 'date' ) {
>+    elsif ( defined $subfield_data{datetype} && $subfield_data{datetype} eq 'date' ) {
>         $subfield_data{marc_value} = {
>             type => 'date_field',
>             id => $subfield_data{id},
>             value => $value,
>         }
>     }
>-    elsif ( $subfield_data{datetype} eq 'datetime' ) {
>+    elsif ( defined $subfield_data{datetype} && $subfield_data{datetype} eq 'datetime' ) {
>         $subfield_data{marc_value} = {
>             type => 'datetime_field',
>             id => $subfield_data{id},
>-- 
>2.39.5

Actions: View | Diff | Splinter Review

Attachments on bug 37398: 171416 | 171417 | 171419 | 171420 | 171421 | 171422 | 171423 | 171424 | 171881 | 171882 | 171919 | 171963 | 172187 | 172188 | 172189 | 172190 | 172191 | 172192 | 172193 | 172194 | 172195 | 172196 | 174558 | 174559 | 174560 | 174561 | 174562 | 174563 | 174564 | 174565 | 174566