Bugzilla – Attachment 179614 Details for
Bug 39408
Cannot add patron via API if AutoEmailNewUser and WELCOME content blank
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 39408: Don't attempt to send a message if one is not generated
Bug-39408-Dont-attempt-to-send-a-message-if-one-is.patch (text/plain), 1.21 KB, created by
Nick Clemens (kidclamp)
on 2025-03-21 14:00:20 UTC
(
hide
)
Description:
Bug 39408: Don't attempt to send a message if one is not generated
Filename:
MIME Type:
Creator:
Nick Clemens (kidclamp)
Created:
2025-03-21 14:00:20 UTC
Size:
1.21 KB
patch
obsolete
>From 0c32bea4259765f39fe570f843a72060e268a7b5 Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@bywatersolutions.com> >Date: Fri, 21 Mar 2025 13:58:24 +0000 >Subject: [PATCH] Bug 39408: Don't attempt to send a message if one is not > generated > >The routine to create the message can return nothing if there is no content, >we should handle this case and not attempt to send. > >To test: >1 - Test plan from last patch >2 - Apply this patch >3 - POST to patron API again >4 - Success! >--- > Koha/REST/V1/Patrons.pm | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > >diff --git a/Koha/REST/V1/Patrons.pm b/Koha/REST/V1/Patrons.pm >index 3ed1081d898..1fb35b59e1e 100644 >--- a/Koha/REST/V1/Patrons.pm >+++ b/Koha/REST/V1/Patrons.pm >@@ -139,7 +139,9 @@ sub add { > message_transport_type => 'email' > } > ); >- SendQueuedMessages( { message_id => $message_id } ); >+ >+ # Don't send a message if we didn't generate one for some reason >+ SendQueuedMessages( { message_id => $message_id } ) if $message_id; > } > } > } >-- >2.39.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 39408
:
179613
|
179614
|
180255
|
180256
|
182165
|
182166
|
182167