do the sms query the dumb way
darcs-hash:20080721093401-84dde-6c370b1901ea3f548bec0d7e69cf21e63c3ade97.gz
This commit is contained in:
parent
2bdc6f0643
commit
30caf568e6
38
lib/mail.php
38
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) {
|
||||
|
|
Loading…
Reference in New Issue
Block a user