summaryrefslogtreecommitdiff
path: root/actions/subscribe.php
diff options
context:
space:
mode:
authorCiaranG <ciaran@ciarang.com>2008-08-22 15:10:32 -0400
committerCiaranG <ciaran@ciarang.com>2008-08-22 15:10:32 -0400
commit92645bbc578fafeb6816a44b8cf9cd594ffb5bd7 (patch)
treeb539bb071b2a037e77d219f0bb0a8eaae1bdc22b /actions/subscribe.php
parentce3cdb20c05b5ad7321d109dc5e0f3b83d828f88 (diff)
XMPP sub/unsub and help commands
darcs-hash:20080822191032-f6e2c-a3a7efbbaad1ec7c48ef132a8ba34fc8b8651969.gz
Diffstat (limited to 'actions/subscribe.php')
-rw-r--r--actions/subscribe.php37
1 files changed, 3 insertions, 34 deletions
diff --git a/actions/subscribe.php b/actions/subscribe.php
index e292fdd6b..71452e46c 100644
--- a/actions/subscribe.php
+++ b/actions/subscribe.php
@@ -38,45 +38,14 @@ class SubscribeAction extends Action {
$other_nickname = $this->arg('subscribeto');
- $other = User::staticGet('nickname', $other_nickname);
-
- if (!$other) {
- common_user_error(_('No such user.'));
- return;
- }
-
- if ($user->isSubscribed($other)) {
- common_user_error(_('Already subscribed!.'));
- return;
- }
-
- if (!$user->subscribeTo($other)) {
- $this->server_error(_('Could not subscribe.'));
+ $result=subs_subscribe_user($user, $other_nickname);
+ if($result != true) {
+ common_user_error($result);
return;
}
-
- $this->notify($other, $user);
-
- if ($other->autosubscribe && !$other->isSubscribed($user)) {
- if (!$other->subscribeTo($user)) {
- $this->server_error(_('Could not subscribe other to you.'));
- return;
- }
- $this->notify($user, $other);
- }
common_redirect(common_local_url('subscriptions', array('nickname' =>
$user->nickname)));
}
- function notify($listenee, $listener) {
- # XXX: add other notifications (Jabber, SMS) here
- # XXX: queue this and handle it offline
- # XXX: Whatever happens, do it in Twitter-like API, too
- $this->notify_email($listenee, $listener);
- }
-
- function notify_email($listenee, $listener) {
- mail_subscribe_notify($listenee, $listener);
- }
} \ No newline at end of file