@@ -, +, @@ --- C4/Suggestions.pm | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) --- a/C4/Suggestions.pm +++ a/C4/Suggestions.pm @@ -503,17 +503,12 @@ sub ModSuggestion { # fetch the entire updated suggestion so that we can populate the letter my $full_suggestion = GetSuggestion( $suggestion->{suggestionid} ); my $patron = Koha::Patrons->find( $full_suggestion->{suggestedby} ); - my $transport; - #If FallbackToSMSIfNoEmail syspref is enabled and the patron has a smsalertnumber but no email address then set message_transport_type of suggestion notice to sms + + my $transport = 'email'; if (C4::Context->preference("FallbackToSMSIfNoEmail")) { - if (($patron->smsalertnumber) && (!$patron->email)) { - $transport="sms"; - } else { - $transport="email"; - } - } else { - $transport = "email"; + $transport = ($patron->smsalertnumber) && (!$patron->email) ? 'sms' : 'email'; } + if ( my $letter = C4::Letters::GetPreparedLetter( module => 'suggestions', --