From d71e27b0737bd45c4558e934f305b8f0671215a6 Mon Sep 17 00:00:00 2001 From: Matt Blenkinsop 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 Signed-off-by: David Nind --- 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.30.2