---- Reported by colin.campbell@ptfs-europe.com 2009-11-25 18:12:28 ---- Nearly all calls to Mail::Sendmail do not test their return. If they fail they fail without letting anyone it leaves anyone trying to get claims working blind. Purpose of this bug is to get some minimal logging of failure. There is more to fo to get these working reliably (see http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=2301) ---- Additional Comments From gmcharlt@gmail.com 2010-02-08 15:09:15 ---- Submitted patch does not apply because of rework in C4/Suggestions.pm done by BibLibre. Concept is still good, so please resubmit. ---- Additional Comments From colin.campbell@ptfs-europe.com 2010-02-08 16:19:50 ---- Created an attachment Reworked patch submitted Reworked against current version of Letters.pm (Suggestionsno longer calls sendmail directly) ---- Additional Comments From gmcharlt@gmail.com 2010-02-08 16:41:43 ---- Patch pushed. Bug should remain open, as there are a bunch of calls to sendmail scattered throughout various scripts that should either be carp-ified or, better yet, folded into a central wrapper. --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:15 UTC --- This bug was previously known as _bug_ 3810 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3810 Imported an attachment (id=1582) Actual time not defined. Setting to 0.0 Setting qa contact to the default for this product. This bug either had no qa contact or an invalid one. The original submitter of attachment 1582 [details] [review] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz.
Still awaiting follow up patches
I think the need Galen identified for further patches was largely mitigated by the removal of sendmail from some of the C4 libraries where it was not necessarily checking success/failure. I've posted a patch that ensures all calls to sendmail elsewhere give the error return on failure (and used carp in all cases)
Follow up patch has been sent by Colin, thanks Colin. If we could get a quick signoff (it looks fine to me fwiw) then ill push it up
This bug is mentioned in: Bug 3810 : Ensure all calls to Mail::Sendmail handled consistently http://lists.koha-community.org/pipermail/koha-patches/2010-December/013269.html
Created attachment 3300 [details] [review] Signed off patch
Pushed and merged, please test and mark resolved