From 8f287eebc5a3a3afca5443e8d9f677c293c1ee52 Mon Sep 17 00:00:00 2001 From: Galen Charlton Date: Mon, 28 Apr 2014 21:32:45 +0000 Subject: [PATCH] Bug 12100: (follow-up) fix regression This patch fixes a regression on bug 5603 introduced by the previous patch by ensuring that if no preferred transports are active for a given message, GetMessagingPreferences() won't try to populate the transports hash with an undef hash key. Signed-off-by: Galen Charlton --- C4/Members/Messaging.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/C4/Members/Messaging.pm b/C4/Members/Messaging.pm index 3cb6ae8244..6be8347ee2 100644 --- a/C4/Members/Messaging.pm +++ b/C4/Members/Messaging.pm @@ -98,6 +98,7 @@ END_SQL $return->{'days_in_advance'} = $row->{'days_in_advance'} if defined $row->{'days_in_advance'}; $return->{'wants_digest'} = $row->{'wants_digest'} if defined $row->{'wants_digest'}; $return->{'letter_code'} = $row->{'letter_code'}; + next unless defined $row->{'message_transport_type'}; $return->{'transports'}->{ $row->{'message_transport_type'} } = $row->{'letter_code'}; } return $return; -- 2.39.5