From b6c8015ab3d6e69963b7c91f516c4fa1e42fc6e4 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Tue, 23 May 2017 16:11:06 -0400 Subject: [PATCH] [16.11.x rebase] Bug 18478 - QA Followup Make sure to build necessary letters Fix awkward construction Signed-off-by: Marcel de Rooy --- C4/Letters.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Letters.pm b/C4/Letters.pm index c53c63e..07ef1f7 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.1.4