many jabber queue management changes

[quix0rs-gnu-social.git] / lib / mail.php

diff --git a/lib/mail.php b/lib/mail.php
index 73c8ae66b891b2e19e2ad5af09224277206f74f2..3dc1f61e2d79e9df5feafce8ac02fd2b8e2a9d1d 100644 (file)
--- a/lib/mail.php
+++ b/lib/mail.php
@@ -109,9 +109,13 @@ function mail_confirm_address($code, $nickname, $address) {
 }
 
 function mail_subscribe_notify($listenee, $listener) {
+       $other = $listener->getProfile();
+       mail_subscribe_notify_profile($listenee, $other);
+}
+
+function mail_subscribe_notify_profile($listenee, $other) {
        if ($listenee->email && $listenee->emailnotifysub) {
                $profile = $listenee->getProfile();
-               $other = $listener->getProfile();
                $name = $profile->getBestName();
                $long_name = ($other->fullname) ? ($other->fullname . ' (' . $other->nickname . ')') : $other->nickname;
                $recipients = $listenee->email;
@@ -160,33 +164,50 @@ 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();
-
-       while ($user->fetch()) {
-               mail_send_sms_notice($notice, $user);
+    # Now, get users subscribed to this profile
+       # XXX: use a join here rather than looping through results
+       
+       $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_notice($notice, $user) {
+function mail_send_sms_notice($notice, $user) {
        $profile = $user->getProfile();
        $name = $profile->getBestName();
        $to = $name . ' <' . $user->smsemail . '>';
        $other = $notice->getProfile();
 
+       common_log(LOG_INFO, "Sending notice " . $notice->id . " to " . $user->smsemail, __FILE__);
+       
        $headers = array();
-       $headers['From'] = $user->incomingemail;
-       $headers['To'] = $name . ' <' . $user->smsemail . '>';
+       $headers['From'] = (isset($user->incomingemail)) ? $user->incomingemail : mail_notify_from();
+       $headers['To'] = $to;
        $headers['Subject'] = sprintf(_('%s status'),
                                                                  $other->getBestName());
        $body = $notice->content;
-       mail_send($user->smsemail, $headers, $body);
+       
+       return mail_send($user->smsemail, $headers, $body);
 }
 
 function mail_confirm_sms($code, $nickname, $address) {