X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Funsubscribe.php;h=4b85ba427ca1b18d611434865ffb0dfce1dcbff3;hb=fd2efbc6f81aba67ff3776712af65b1a8fa73d06;hp=ba9ecd8f00e5d46b3409181f89dc5bfcb4201995;hpb=9948523c33ea0d02681ff1e0cd4fcb839dc9df96;p=quix0rs-gnu-social.git diff --git a/actions/unsubscribe.php b/actions/unsubscribe.php index ba9ecd8f00..4b85ba427c 100644 --- a/actions/unsubscribe.php +++ b/actions/unsubscribe.php @@ -50,15 +50,11 @@ class UnsubscribeAction extends Action if (!common_logged_in()) { // TRANS: Error message displayed when trying to perform an action that requires a logged in user. $this->clientError(_('Not logged in.')); - return; } - $user = common_current_user(); - if ($_SERVER['REQUEST_METHOD'] != 'POST') { common_redirect(common_local_url('subscriptions', - array('nickname' => $user->nickname))); - return; + array('nickname' => $this->scoped->nickname))); } /* Use a session token for CSRF protection. */ @@ -69,7 +65,6 @@ class UnsubscribeAction extends Action // TRANS: Client error displayed when the session token does not match or is not given. $this->clientError(_('There was a problem with your session token. ' . 'Try again, please.')); - return; } $other_id = $this->arg('unsubscribeto'); @@ -77,22 +72,19 @@ class UnsubscribeAction extends Action if (!$other_id) { // TRANS: Client error displayed when trying to unsubscribe without providing a profile ID. $this->clientError(_('No profile ID in request.')); - return; } - $other = Profile::staticGet('id', $other_id); + $other = Profile::getKV('id', $other_id); - if (!$other) { + if (!($other instanceof Profile)) { // TRANS: Client error displayed when trying to unsubscribe while providing a non-existing profile ID. $this->clientError(_('No profile with that ID.')); - return; } - $result = subs_unsubscribe_to($user, $other); - - if (is_string($result)) { - $this->clientError($result); - return; + try { + Subscription::cancel($this->scoped, $other); + } catch (Exception $e) { + $this->clientError($e->getMessage()); } if ($this->boolean('ajax')) { @@ -105,11 +97,9 @@ class UnsubscribeAction extends Action $subscribe = new SubscribeForm($this, $other); $subscribe->show(); $this->elementEnd('body'); - $this->elementEnd('html'); + $this->endHTML(); } else { - common_redirect(common_local_url('subscriptions', - array('nickname' => $user->nickname)), - 303); + common_redirect(common_local_url('subscriptions', array('nickname' => $this->scoped->nickname)), 303); } } }