Bug 28996

Summary: SendAlerts crashes on a bad email 'To'
Product: Koha Reporter: Didier Gautheron <didier.gautheron>
Component: Command-line UtilitiesAssignee: Bugs List <koha-bugs>
Status: REOPENED --- QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: dcook, fridolin.somers, jonathan.druart, nick, robin
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28803
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 22343    
Bug Blocks:    
Attachments: Bug 28996: Catch error in mail creation
Bug 28996: Catch error in mail creation

Description Didier Gautheron 2021-09-10 11:56:25 UTC
If there's a bad 'To' field email create die but it isn't caught by SendQueuedMessages.
Comment 1 Didier Gautheron 2021-09-24 12:20:13 UTC
Created attachment 125238 [details] [review]
Bug 28996: Catch error in mail creation
Comment 2 Didier Gautheron 2021-09-24 14:08:10 UTC
Created attachment 125250 [details] [review]
Bug 28996: Catch error in mail creation
Comment 3 Fridolin Somers 2021-09-24 19:49:17 UTC
+1 giga
Comment 4 Jonathan Druart 2021-10-06 15:52:23 UTC

*** This bug has been marked as a duplicate of bug 28803 ***
Comment 5 Jonathan Druart 2021-10-06 15:52:49 UTC
Looks like a duplicate of bug 28803. Please reopen if I am wrong.
Comment 6 David Cook 2024-02-06 23:16:58 UTC
It looks like bug 28803 only covers part of the root problem. C4::Letters::SendAlerts still has the problem. I'll rename the bug title to be more accurate.
Comment 7 David Cook 2024-02-06 23:18:16 UTC
The patch will need to be rewriten to take into account bug 28803 but should be an easy fix overall. (I've applied the fix locally for years. At some point this year I might find the time to upstream it.)