X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Fgroupunblock.php;h=a72486be487ce31acccd6b0616d95dfa01c3fb22;hb=d6b28c64830f632bb2f4b6f3c9369b9e56ad217a;hp=6f26108155b647ebaed33f84bcdda85eee0fd578;hpb=2a4dc77a633cc78907934fd93200ac16d55be78e;p=quix0rs-gnu-social.git diff --git a/actions/groupunblock.php b/actions/groupunblock.php index 6f26108155..a72486be48 100644 --- a/actions/groupunblock.php +++ b/actions/groupunblock.php @@ -52,54 +52,46 @@ class GroupunblockAction extends Action * * @return boolean success flag */ - function prepare($args) + function prepare(array $args=array()) { parent::prepare($args); 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 false; } $token = $this->trimmed('token'); if (empty($token) || $token != common_session_token()) { // 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; } $id = $this->trimmed('unblockto'); if (empty($id)) { // TRANS: Client error displayed when trying to unblock a user from a group without providing a profile. $this->clientError(_('No profile specified.')); - return false; } $this->profile = Profile::getKV('id', $id); if (empty($this->profile)) { // TRANS: Client error displayed when trying to unblock a user from a group without providing an existing profile. $this->clientError(_('No profile with that ID.')); - return false; } $group_id = $this->trimmed('unblockgroup'); if (empty($group_id)) { // TRANS: Client error displayed when trying to unblock a user from a group without providing a group. $this->clientError(_('No group specified.')); - return false; } $this->group = User_group::getKV('id', $group_id); if (empty($this->group)) { // TRANS: Client error displayed when trying to unblock a user from a non-existing group. $this->clientError(_('No such group.')); - return false; } $user = common_current_user(); if (!$user->isAdmin($this->group)) { // TRANS: Client error displayed when trying to unblock a user from a group without being an administrator for the group. $this->clientError(_('Only an admin can unblock group members.'), 401); - return false; } if (!Group_block::isBlocked($this->group, $this->profile)) { // TRANS: Client error displayed when trying to unblock a non-blocked user from a group. $this->clientError(_('User is not blocked from group.')); - return false; } return true; } @@ -111,7 +103,7 @@ class GroupunblockAction extends Action * * @return void */ - function handle($args) + function handle(array $args=array()) { parent::handle($args); if ($_SERVER['REQUEST_METHOD'] == 'POST') { @@ -131,7 +123,6 @@ class GroupunblockAction extends Action if (!$result) { // TRANS: Server error displayed when unblocking a user from a group fails because of an unknown error. $this->serverError(_('Error removing the block.')); - return; } foreach ($this->args as $k => $v) { @@ -145,9 +136,7 @@ class GroupunblockAction extends Action if ($action) { common_redirect(common_local_url($action, $args), 303); } else { - common_redirect(common_local_url('blockedfromgroup', - array('nickname' => $this->group->nickname)), - 303); + common_redirect(common_local_url('blockedfromgroup', array('nickname' => $this->group->nickname)), 303); } } }