From 6fe74607d1119e4afe4aa4056a3c14028df41f4a Mon Sep 17 00:00:00 2001
From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Date: Tue, 24 May 2022 12:32:45 +0100
Subject: [PATCH] Bug 30838: Don't fallback through phone numbers for sms

This patch removes the fallback handling for smsalartnumber as the
to_address in notices. We ignore the to_address field in the message
queue at send time for sms anyway and use smsalertnumber exclusively so
having this field populated is just confusing to the end user

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 5e388ed509c58b5f7398baf6f7bb027b4dc46c1c)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit 20b17b51679ac6884da2550d952a75c786e81692)
Signed-off-by: Arthur Suzuki <arthur.suzuki@biblibre.com>
---
 C4/Message.pm | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/C4/Message.pm b/C4/Message.pm
index 3a3216657b..3ac8f21dc3 100644
--- a/C4/Message.pm
+++ b/C4/Message.pm
@@ -185,9 +185,6 @@ sub _to_address {
             || $borrower->{B_email};
     } elsif ($transport eq 'sms') {
         $address = $borrower->{smsalertnumber}
-            || $borrower->{phone}
-            || $borrower->{phonepro}
-            || $borrower->{B_phone};
     } else {
         warn "'$transport' is an unknown message transport.";
     }
-- 
2.39.5