From df7f2f2ab3d30ab0047328b3a430a36b8d828c4d Mon Sep 17 00:00:00 2001 From: Olli-Antti Kivilahti Date: Thu, 28 Aug 2014 11:30:59 +0300 Subject: [PATCH] Bug 12099 - process_message_queue.pl tells it sends print mails even if it doesn't. Made the C4::Letters::SendQueuedMessages() notify all but 'print' messages in verbose mode. This way verbose mode provides a bit cleaner output. --- C4/Letters.pm | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/C4/Letters.pm b/C4/Letters.pm index 2bb764f..dc6bc00 100644 --- a/C4/Letters.pm +++ b/C4/Letters.pm @@ -755,7 +755,9 @@ sub SendQueuedMessages { warn sprintf( 'sending %s message to patron: %s', $message->{'message_transport_type'}, $message->{'borrowernumber'} || 'Admin' ) - if $params->{'verbose'} or $debug; + if ( ($params->{'verbose'} || $debug) && + ( not($message->{'message_transport_type'} eq 'print') ) + ); #Print messages are dealt with elsewhere so don't log them needlessly # This is just begging for subclassing next MESSAGE if ( lc($message->{'message_transport_type'}) eq 'rss' ); if ( lc( $message->{'message_transport_type'} ) eq 'email' ) { -- 1.7.9.5