From 1da486da8987ce92299b2fdc8dd43bb1963ed6c4 Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Tue, 23 May 2017 16:11:06 -0400
Subject: [PATCH] Bug 18478 - QA Followup

Make sure to build necessary letters
Fix awkward construction

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
---
 C4/Letters.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/C4/Letters.pm b/C4/Letters.pm
index c53c63e4ff..07ef1f7566 100644
--- a/C4/Letters.pm
+++ b/C4/Letters.pm
@@ -1052,7 +1052,7 @@ sub SendQueuedMessages {
                     _set_message_status( { message_id => $message->{'message_id'}, status => 'failed' } );
                     next MESSAGE;
                 }
-                $message->{to_address} = $message->{to_address} ? $message->{to_address} : $member->{'smsalertnumber'};
+                $message->{to_address} ||= $member->{'smsalertnumber'};
                 unless ( $message->{to_address} && $member->{'smsalertnumber'} ) {
                     _set_message_status( { message_id => $message->{'message_id'}, status => 'failed' } );
                     warn sprintf( "No smsalertnumber found for patron %s!", $message->{'borrowernumber'} ) if $params->{'verbose'} or $debug;
-- 
2.39.5