if not a POST, redirect to subscriptions

darcs-hash:20080705213637-84dde-87ceb18bfc0db5248083bae1b9ce7ef088a68fe2.gz
This commit is contained in:
Evan Prodromou 2008-07-05 17:36:37 -04:00
parent a050b2f1d9
commit 293ad758f7
2 changed files with 15 additions and 4 deletions

View File

@ -28,6 +28,13 @@ class SubscribeAction extends Action {
return; 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_nickname = $this->arg('subscribeto');
$other = User::staticGet('nickname', $other_nickname); $other = User::staticGet('nickname', $other_nickname);
@ -37,8 +44,6 @@ class SubscribeAction extends Action {
return; return;
} }
$user = common_current_user();
if ($user->isSubscribed($other)) { if ($user->isSubscribed($other)) {
common_user_error(_t('Already subscribed!.')); common_user_error(_t('Already subscribed!.'));
return; return;

View File

@ -24,6 +24,14 @@ class UnsubscribeAction extends Action {
common_user_error(_t('Not logged in.')); common_user_error(_t('Not logged in.'));
return; 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_nickname = $this->arg('unsubscribeto');
$other = User::staticGet('nickname', $other_nickname); $other = User::staticGet('nickname', $other_nickname);
if (!$other) { if (!$other) {
@ -31,8 +39,6 @@ class UnsubscribeAction extends Action {
return; return;
} }
$user = common_current_user();
if (!$user->isSubscribed($other)) { if (!$user->isSubscribed($other)) {
common_server_error(_t('Not subscribed!.')); common_server_error(_t('Not subscribed!.'));
} }