From 36eaf42bbb25ce1a5ca8636777d0057a0713105c Mon Sep 17 00:00:00 2001 From: Evan Prodromou Date: Mon, 10 Aug 2009 16:42:04 -0400 Subject: [PATCH] Revert "UnsubscribeAction: Add LACONICA gate, fix PHPCS errors, fix error handling typo." This reverts commit e1270b26f5eae1c3d7ea4b954f0bc1f51d73494c. --- actions/unsubscribe.php | 41 +++++++---------------------------------- 1 file changed, 7 insertions(+), 34 deletions(-) diff --git a/actions/unsubscribe.php b/actions/unsubscribe.php index 46fbcf6571..19275041a8 100644 --- a/actions/unsubscribe.php +++ b/actions/unsubscribe.php @@ -1,16 +1,5 @@ - * @author Robin Millette - * @license http://www.fsf.org/licensing/licenses/agpl.html AGPLv3 - * @link http://laconi.ca/ - * +/* * Laconica - a distributed open-source microblogging tool * Copyright (C) 2008, 2009, Control Yourself, Inc. * @@ -28,20 +17,6 @@ * along with this program. If not, see . */ -if (!defined('LACONICA')) { - exit(1); -} - -/** - * Unsubscribe handler - * - * @category Action - * @package Laconica - * @author Evan Prodromou - * @author Robin Millette - * @license http://www.fsf.org/licensing/licenses/agpl.html AGPLv3 - * @link http://laconi.ca/ - */ class UnsubscribeAction extends Action { @@ -56,18 +31,16 @@ class UnsubscribeAction extends Action $user = common_current_user(); if ($_SERVER['REQUEST_METHOD'] != 'POST') { - common_redirect(common_local_url('subscriptions', - array('nickname' => $user->nickname))); + common_redirect(common_local_url('subscriptions', array('nickname' => $user->nickname))); return; } - /* Use a session token for CSRF protection. */ + # CSRF protection $token = $this->trimmed('token'); if (!$token || $token != common_session_token()) { - $this->clientError(_('There was a problem with your session token. ' . - 'Try again, please.')); + $this->clientError(_('There was a problem with your session token. Try again, please.')); return; } @@ -80,7 +53,7 @@ class UnsubscribeAction extends Action $other = Profile::staticGet('id', $other_id); - if (!$other) { + if (!$other_id) { $this->clientError(_('No profile with that id.')); return; } @@ -103,8 +76,8 @@ class UnsubscribeAction extends Action $this->elementEnd('body'); $this->elementEnd('html'); } else { - common_redirect(common_local_url('subscriptions', - array('nickname' => $user->nickname)), + common_redirect(common_local_url('subscriptions', array('nickname' => + $user->nickname)), 303); } } -- 2.39.2