From 30caf568e6f8f8eeafa97c516cdd8b09fd6d7092 Mon Sep 17 00:00:00 2001
From: Evan Prodromou <evan@prodromou.name>
Date: Mon, 21 Jul 2008 05:34:01 -0400
Subject: [PATCH] do the sms query the dumb way

darcs-hash:20080721093401-84dde-6c370b1901ea3f548bec0d7e69cf21e63c3ade97.gz
---
 lib/mail.php | 38 ++++++++++++++++++++------------------
 1 file changed, 20 insertions(+), 18 deletions(-)

diff --git a/lib/mail.php b/lib/mail.php
index 2f4f502d47..3503be1d14 100644
--- a/lib/mail.php
+++ b/lib/mail.php
@@ -160,28 +160,30 @@ function mail_new_incoming_address() {
 
 function mail_broadcast_notice_sms($notice) {
 
-	$user = new User();
-
-	$user->smsnotify = 1;
-	$user->whereAdd('EXISTS (select subscriber from subscriptions where ' .
-					' subscriber = user.id and subscribed = ' . $notice->profile_id . ')');
-	$user->whereAdd('sms is not null');
-
-	$cnt = $user->find();
-
-	common_log(LOG_INFO, "Sending notice " . $notice->id . " to $cnt subscribers", __FILE__);
+    # Now, get users subscribed to this profile
+	# XXX: use a join here rather than looping through results
 	
-	if ($cnt) {
-		while ($user->fetch()) {
-			$success = mail_send_sms_notice($notice, $user);
-			if (!$success) {
-				common_log(LOG_ERR, 'Could not send SMS message to user', __FILE__);
-				return false;
+	$sub = new Subscription();
+	$sub->subscribed = $notice->profile_id;
+        
+	if ($sub->find()) {
+		while ($sub->fetch()) {
+			$user = User::staticGet($sub->subscriber);
+			if ($user && $user->smsemail && $user->smsnotify) {
+				common_log(LOG_INFO,
+						   'Sending notice ' . $notice->id . ' to ' . $user->smsemail,
+						   __FILE__);
+				$success = mail_send_sms_notice($notice, $user);
+				if (!$success) {
+					# XXX: Not sure, but I think that's the right thing to do
+					common_log(LOG_WARNING,
+							   'Sending notice ' . $notice->id . ' to ' . $user->smsemail . ' FAILED, cancelling.',
+							   __FILE__);
+					return false;
+				}
 			}
 		}
 	}
-	
-	return true;
 }
 
 function mail_send_sms_notice($notice, $user) {