From 293ad758f76143cf0e23c8d4a65f66d8a1ba7bab Mon Sep 17 00:00:00 2001
From: Evan Prodromou <evan@prodromou.name>
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!.'));
 		}