From baf3fcb76df1aaf84d7be4062f6fcd1e5fcfba6e Mon Sep 17 00:00:00 2001 From: Julian Maurice Date: Thu, 8 Mar 2012 13:50:47 +0100 Subject: [PATCH] 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 --- C4/Letters.pm | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/C4/Letters.pm b/C4/Letters.pm index 28c6984..0f0e0f6 100644 --- a/C4/Letters.pm +++ b/C4/Letters.pm @@ -534,6 +534,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.9.1