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."
Created attachment 153490 [details] [review] 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
Created attachment 153587 [details] [review] 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 <david@davidnind.com>
Testing notes (using KTD): 1. For me, the warning only appeared in the logs when editing the contact information for existing patrons. If I created a new patron, the warning message didn't appear in the logs. 2. To view the logs: vi /var/log/koha/kohadev/plack-intranet-error.log
Created attachment 153852 [details] [review] 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 <david@davidnind.com> Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Created attachment 153853 [details] [review] Bug 34280: (QA follow-up) perltidy Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Trivial follow-up patch to make perltidy happy. Otherwise, change makes sense and QA tool passes. Passing QA
Pushed to master for 23.11. Nice work everyone, thanks!
Pushed to 23.05.x for 23.05.03
Looks like this has been added by Bug 29046
Bug 29046 not in 22.11.x. Not pushing.