Bug 6090

Summary: empty mails in message_queue are sent
Product: Koha Reporter: Julian Maurice <julian.maurice>
Component: NoticesAssignee: Julian Maurice <julian.maurice>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: minor    
Priority: P3 CC: chris, colin.campbell, henridamien.laurent, katrin.fischer, kyle.m.hall, leila.arkab, m.de.rooy, paul.poulain
Version: 3.8   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Patch
Bug 6090: Do not enqueue empty messages
[SIGNED-OFF] Bug 6090: Do not enqueue empty messages
Bug 6090: Do not enqueue empty messages
Bug 6090: Do not enqueue empty messages
Bug 6090: Do not enqueue empty messages
[SIGNED-OFF] Bug 6090: Do not enqueue empty messages

Description Julian Maurice 2011-04-06 13:41:52 UTC

    
Comment 1 Julian Maurice 2011-04-06 13:54:15 UTC Comment hidden (obsolete)
Comment 2 Katrin Fischer 2011-06-01 05:07:36 UTC
I wonder if we should have messages with empty message body and subject there in the first place? How does this happen?
Comment 3 Colin Campbell 2011-11-04 12:49:27 UTC
Were getting empty messages being reported as well at multiple sites. The bug is surely that these are being added to the queue
Comment 4 laurenthdl 2011-12-23 14:28:41 UTC
Collin, we couldnot find how those empty messages got added to the message queue.
Therefore, we did that workaround.
If you find where the bug that creates those ghost message is, then, that will not be really usefull any longer. But will not harm much the system though.
Comment 5 Julian Maurice 2012-03-08 11:16:03 UTC
The patch doesn't fix the problem, so setting status to assigned until someone send a patch that avoid empty mail to be added to message_queue (if bug still exists)
Comment 6 Julian Maurice 2012-03-08 13:20:45 UTC Comment hidden (obsolete)
Comment 7 Leila 2012-03-21 10:50:17 UTC
Hello,
Need to config the sending of mails, so not as easy as I believed. We don't only have to generate an empty notice.
(server Koha add to send really notices).
Comment 8 Kyle M Hall 2012-06-09 10:10:40 UTC Comment hidden (obsolete)
Comment 9 Paul Poulain 2012-06-10 09:15:17 UTC
QA comment: there is an unconditionnal warn 
+        warn "Trying to add an empty message to the message queue";
that is thrown every time there is an empty letter. This should be issued only if you're in DEBUG mode
Comment 10 Kyle M Hall 2012-06-11 08:32:44 UTC Comment hidden (obsolete)
Comment 11 Marcel de Rooy 2012-06-12 13:53:35 UTC
Typo: +kfpackage C4::Letters;

How was this tested? :-)

Please correct and set back to Needs signoff.
Comment 12 Julian Maurice 2012-06-20 11:42:29 UTC Comment hidden (obsolete)
Comment 13 Julian Maurice 2012-06-20 11:46:54 UTC Comment hidden (obsolete)
Comment 14 Kyle M Hall 2012-06-20 12:51:06 UTC
Created attachment 10381 [details] [review]
[SIGNED-OFF] Bug 6090: Do not enqueue empty messages

This patch avoid enqueuing messages that have an empty body. It can
happen when letter is empty or becomes empty after being processed by
parseletter

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 15 Marcel de Rooy 2012-06-20 13:02:46 UTC
Marking as Passed QA
Comment 16 Julian Maurice 2012-07-06 08:39:24 UTC
Does this patch have to be pushed to stable?