X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Fremovepeopletag.php;h=20fd80bc7a7e3e9d65bdf63240347f37da96067c;hb=aa004d03aab4ef8035cc204bc8718c9d2296100a;hp=6943f35ed1cbb3eb694737ba69b62ffbe544c6b9;hpb=69e1bd3a58da8a2f9f8eefe8b61d94c9e2f6f71c;p=quix0rs-gnu-social.git diff --git a/actions/removepeopletag.php b/actions/removepeopletag.php index 6943f35ed1..20fd80bc7a 100644 --- a/actions/removepeopletag.php +++ b/actions/removepeopletag.php @@ -66,7 +66,7 @@ class RemovepeopletagAction extends Action * * @return boolean success flag */ - function prepare($args) + function prepare(array $args = array()) { parent::prepare($args); @@ -78,7 +78,6 @@ class RemovepeopletagAction 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 false; } // Only for logged-in users @@ -88,40 +87,25 @@ class RemovepeopletagAction extends Action if (empty($this->user)) { // TRANS: Error message displayed when trying to perform an action that requires a logged in user. $this->clientError(_('Not logged in.')); - return false; } // Profile to subscribe to $tagged_id = $this->arg('tagged'); - $this->tagged = Profile::staticGet('id', $tagged_id); + $this->tagged = Profile::getKV('id', $tagged_id); if (empty($this->tagged)) { // TRANS: Client error displayed when referring to a non-existing profile. $this->clientError(_('No such profile.')); - return false; } $id = $this->arg('peopletag_id'); - $this->peopletag = Profile_list::staticGet('id', $id); + $this->peopletag = Profile_list::getKV('id', $id); if (empty($this->peopletag)) { // TRANS: Client error displayed trying to reference a non-existing list. $this->clientError(_('No such list.')); - return false; - } - - // OMB 0.1 doesn't have a mechanism for local-server- - // originated tag. - - $omb01 = Remote_profile::staticGet('id', $tagged_id); - - if (!empty($omb01)) { - // TRANS: Client error displayed when trying to (un)list an OMB 0.1 remote profile. - $this->clientError(_('You cannot (un)list an OMB 0.1 '. - 'remote profile with this action.')); - return false; } return true; @@ -136,7 +120,7 @@ class RemovepeopletagAction extends Action * * @return void */ - function handle($args) + function handle() { // Throws exception on error @@ -144,7 +128,7 @@ class RemovepeopletagAction extends Action $this->peopletag->tag); if (!$ptag) { - $user = User::staticGet('id', $this->tagged->id); + $user = User::getKV('id', $this->tagged->id); if ($user) { $this->clientError( // TRANS: Client error displayed when an unknown error occurs while delisting a user. @@ -158,7 +142,6 @@ class RemovepeopletagAction extends Action 'The remote server is probably not responding correctly, ' . 'please try retrying later.'), $this->profile->profileurl)); } - return false; } if ($this->boolean('ajax')) { $this->startHTML('text/xml;charset=utf-8'); @@ -170,7 +153,7 @@ class RemovepeopletagAction extends Action $unsubscribe = new TagButton($this, $this->tagged, $this->peopletag); $unsubscribe->show(); $this->elementEnd('body'); - $this->elementEnd('html'); + $this->endHTML(); } else { $url = common_local_url('subscriptions', array('nickname' => $this->user->nickname));