Attachment 153490 Details for Bug 34280 – Bug 34280: Fix warning in logs when saving patron details

[patch] Bug 34280: Fix warning in logs when saving patron details

Bug-34280-Fix-warning-in-logs-when-saving-patron-d.patch (text/plain), 1.25 KB, created by Matt Blenkinsop on 2023-07-14 14:15:19 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Matt Blenkinsop

Created: 2023-07-14 14:15:19 UTC

Size: 1.25 KB

patch

obsolete

>From e7eb75b18c3037c82811b45ae592a225c6a709db Mon Sep 17 00:00:00 2001
>From: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
>Date: Fri, 14 Jul 2023 14:11:28 +0000
>Subject: [PATCH] Bug 34280: Fix warning in logs when saving patron details
>
>If a patron has no valid email address then a warning message appears in the logs when saving:
>
>"Use of uninitialized value $email in string ne at /kohadevbox/koha/Koha/Patron.pm line 1445."
>
>This patch fixes that error by removing an unnescessary string ne
>
>Test plan:
>1) Create/choose a patron with no email addresses
>2) On the patron record in the page section for Contact information, click edit
>3) Now click save
>4) The warning above should appear in the logs
>5) Apply patch
>6) Repeat steps 2 and 3
>7) The warning should no longer appear
>---
> Koha/Patron.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/Koha/Patron.pm b/Koha/Patron.pm
>index 2a8eecf9f6..4a29c76f80 100644
>--- a/Koha/Patron.pm
>+++ b/Koha/Patron.pm
>@@ -1442,7 +1442,7 @@ sub first_valid_email_address {
>       C4::Context->preference('EmailFieldPrecedence');
>     for my $field (@fields) {
>         $email = $self->$field;
>-        last if ( $email ne q{} );
>+        last if ( $email );
>     }
> 
>     return $email;
>-- 
>2.37.1 (Apple Git-137.1)

Actions: View | Diff | Splinter Review

Attachments on bug 34280: 153490 | 153587 | 153852 | 153853