From 293ad758f76143cf0e23c8d4a65f66d8a1ba7bab Mon Sep 17 00:00:00 2001 From: Evan Prodromou Date: Sat, 5 Jul 2008 17:36:37 -0400 Subject: [PATCH] if not a POST, redirect to subscriptions darcs-hash:20080705213637-84dde-87ceb18bfc0db5248083bae1b9ce7ef088a68fe2.gz --- actions/subscribe.php | 9 +++++++-- actions/unsubscribe.php | 10 ++++++++-- 2 files changed, 15 insertions(+), 4 deletions(-) diff --git a/actions/subscribe.php b/actions/subscribe.php index 9a1d43fa5d..fe683828a7 100644 --- a/actions/subscribe.php +++ b/actions/subscribe.php @@ -28,6 +28,13 @@ class SubscribeAction extends Action { return; } + $user = common_current_user(); + + if ($_SERVER['REQUEST_METHOD'] != 'POST') { + common_redirect(common_local_url('subscriptions', array('nickname' => $user->nickname))); + return; + } + $other_nickname = $this->arg('subscribeto'); $other = User::staticGet('nickname', $other_nickname); @@ -37,8 +44,6 @@ class SubscribeAction extends Action { return; } - $user = common_current_user(); - if ($user->isSubscribed($other)) { common_user_error(_t('Already subscribed!.')); return; diff --git a/actions/unsubscribe.php b/actions/unsubscribe.php index fabb7148e1..1bf95fb612 100644 --- a/actions/unsubscribe.php +++ b/actions/unsubscribe.php @@ -24,6 +24,14 @@ class UnsubscribeAction extends Action { common_user_error(_t('Not logged in.')); return; } + + $user = common_current_user(); + + if ($_SERVER['REQUEST_METHOD'] != 'POST') { + common_redirect(common_local_url('subscriptions', array('nickname' => $user->nickname))); + return; + } + $other_nickname = $this->arg('unsubscribeto'); $other = User::staticGet('nickname', $other_nickname); if (!$other) { @@ -31,8 +39,6 @@ class UnsubscribeAction extends Action { return; } - $user = common_current_user(); - if (!$user->isSubscribed($other)) { common_server_error(_t('Not subscribed!.')); } -- 2.39.5