From 49905bf6f96341750abe4d7b9cfd977a619744ed Mon Sep 17 00:00:00 2001 From: Julian Maurice Date: Thu, 8 Mar 2012 13:50:47 +0100 Subject: [PATCH] [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 --- C4/Letters.pm | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/C4/Letters.pm b/C4/Letters.pm index 89213fe..3d27101 100644 --- a/C4/Letters.pm +++ b/C4/Letters.pm @@ -672,6 +672,13 @@ sub EnqueueLetter ($) { return unless exists $params->{'borrowernumber'}; return unless exists $params->{'message_transport_type'}; + my $content = $params->{letter}->{content}; + $content =~ s/\s+//g if(defined $content); + if ( not defined $content or $content eq '' ) { + warn "Trying to add an empty message to the message queue"; + return; + } + # If we have any attachments we should encode then into the body. if ( $params->{'attachments'} ) { $params->{'letter'} = _add_attachments( -- 1.7.2.5