X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Fnewgroup.php;h=ef8346c03ef053b37868da8933d0e10db7216a14;hb=2a8ab1c6cae572d324339b211625c4d219db0c89;hp=9682b875cb0c666a058c0b241b4310d939cd2225;hpb=2cdba23df418245e9981753e473b2eeaf1420dad;p=quix0rs-gnu-social.git diff --git a/actions/newgroup.php b/actions/newgroup.php index 9682b875cb..ef8346c03e 100644 --- a/actions/newgroup.php +++ b/actions/newgroup.php @@ -24,13 +24,12 @@ * @author Evan Prodromou * @author Sarven Capadisli * @copyright 2008-2009 StatusNet, Inc. + * @copyright 2013 Free Software Foundation, Inc. * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0 * @link http://status.net/ */ -if (!defined('STATUSNET') && !defined('LACONICA')) { - exit(1); -} +if (!defined('GNUSOCIAL')) { exit(1); } /** * Add a new group @@ -43,9 +42,15 @@ if (!defined('STATUSNET') && !defined('LACONICA')) { * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0 * @link http://status.net/ */ -class NewgroupAction extends Action +class NewgroupAction extends FormAction { - var $msg; + protected $group; + + protected $form = 'GroupEdit'; + + function getGroup() { + return $this->group; + } function title() { @@ -53,120 +58,54 @@ class NewgroupAction extends Action return _('New group'); } - /** - * Prepare to run - */ - function prepare($args) + protected function doPreparation() { - parent::prepare($args); - - if (!common_logged_in()) { - // TRANS: Client error displayed trying to create a group while not logged in. - $this->clientError(_('You must be logged in to create a group.')); - return false; - } - - $user = common_current_user(); - $profile = $user->getProfile(); - if (!$profile->hasRight(Right::CREATEGROUP)) { + // $this->scoped is the current user profile + if (!$this->scoped->hasRight(Right::CREATEGROUP)) { // TRANS: Client exception thrown when a user tries to create a group while banned. - throw new ClientException(_('You are not allowed to create groups on this site.'), 403); - } - - return true; - } - - /** - * Handle the request - * - * On GET, show the form. On POST, try to save the group. - * - * @param array $args unused - * - * @return void - */ - function handle($args) - { - parent::handle($args); - if ($_SERVER['REQUEST_METHOD'] == 'POST') { - $this->trySave(); - } else { - $this->showForm(); + $this->clientError(_('You are not allowed to create groups on this site.'), 403); } } - function showForm($msg=null) + protected function getInstructions() { - $this->msg = $msg; - $this->showPage(); + // TRANS: Form instructions for group create form. + return _('Use this form to create a new group.'); } - function showContent() - { - $form = new GroupEditForm($this); - $form->show(); - } - - function showPageNotice() - { - if ($this->msg) { - $this->element('p', 'error', $this->msg); - } else { - $this->element('p', 'instructions', - // TRANS: Form instructions for group create form. - _('Use this form to create a new group.')); - } - } - - function trySave() + protected function doPost() { if (Event::handle('StartGroupSaveForm', array($this))) { - try { - $nickname = Nickname::normalize($this->trimmed('nickname')); - } catch (NicknameException $e) { - $this->showForm($e->getMessage()); - } + $nickname = Nickname::normalize($this->trimmed('newnickname'), true); + $fullname = $this->trimmed('fullname'); $homepage = $this->trimmed('homepage'); $description = $this->trimmed('description'); $location = $this->trimmed('location'); + $private = $this->boolean('private'); $aliasstring = $this->trimmed('aliases'); - if ($this->nicknameExists($nickname)) { - // TRANS: Group create form validation error. - $this->showForm(_('Nickname already in use. Try another one.')); - return; - } else if (!User_group::allowedNickname($nickname)) { - // TRANS: Group create form validation error. - $this->showForm(_('Not a valid nickname.')); - return; - } else if (!is_null($homepage) && (strlen($homepage) > 0) && - !Validate::uri($homepage, - array('allowed_schemes' => - array('http', 'https')))) { + if (!is_null($homepage) && (strlen($homepage) > 0) && + !common_valid_http_url($homepage)) { // TRANS: Group create form validation error. - $this->showForm(_('Homepage is not a valid URL.')); - return; + throw new ClientException(_('Homepage is not a valid URL.')); } else if (!is_null($fullname) && mb_strlen($fullname) > 255) { // TRANS: Group create form validation error. - $this->showForm(_('Full name is too long (maximum 255 characters).')); - return; + throw new ClientException(_('Full name is too long (maximum 255 characters).')); } else if (User_group::descriptionTooLong($description)) { // TRANS: Group create form validation error. // TRANS: %d is the maximum number of allowed characters. - $this->showForm(sprintf(_m('Description is too long (maximum %d character).', + throw new ClientException(sprintf(_m('Description is too long (maximum %d character).', 'Description is too long (maximum %d characters).', User_group::maxDescription()), User_group::maxDescription())); - return; } else if (!is_null($location) && mb_strlen($location) > 255) { // TRANS: Group create form validation error. - $this->showForm(_('Location is too long (maximum 255 characters).')); - return; + throw new ClientException(_('Location is too long (maximum 255 characters).')); } if (!empty($aliasstring)) { - $aliases = array_map('common_canonical_nickname', array_unique(preg_split('/[\s,]+/', $aliasstring))); + $aliases = array_map(array('Nickname', 'normalize'), array_unique(preg_split('/[\s,]+/', $aliasstring))); } else { $aliases = array(); } @@ -174,39 +113,22 @@ class NewgroupAction extends Action if (count($aliases) > common_config('group', 'maxaliases')) { // TRANS: Group create form validation error. // TRANS: %d is the maximum number of allowed aliases. - $this->showForm(sprintf(_m('Too many aliases! Maximum %d allowed.', + throw new ClientException(sprintf(_m('Too many aliases! Maximum %d allowed.', 'Too many aliases! Maximum %d allowed.', common_config('group', 'maxaliases')), common_config('group', 'maxaliases'))); - return; } - foreach ($aliases as $alias) { - if (!Nickname::isValid($alias)) { - // TRANS: Group create form validation error. - // TRANS: %s is the invalid alias. - $this->showForm(sprintf(_('Invalid alias: "%s"'), $alias)); - return; - } - if ($this->nicknameExists($alias)) { - // TRANS: Group create form validation error. %s is the already used alias. - $this->showForm(sprintf(_('Alias "%s" already in use. Try another one.'), - $alias)); - return; - } - // XXX assumes alphanum nicknames - if (strcmp($alias, $nickname) == 0) { - // TRANS: Group create form validation error. - $this->showForm(_('Alias cannot be the same as nickname.')); - return; - } + if ($private) { + $force_scope = 1; + $join_policy = User_group::JOIN_POLICY_MODERATE; + } else { + $force_scope = 0; + $join_policy = User_group::JOIN_POLICY_OPEN; } - $cur = common_current_user(); - - // Checked in prepare() above - - assert(!is_null($cur)); + // This is set up in parent->prepare and checked in self->prepare + assert(!is_null($this->scoped)); $group = User_group::register(array('nickname' => $nickname, 'fullname' => $fullname, @@ -214,7 +136,9 @@ class NewgroupAction extends Action 'description' => $description, 'location' => $location, 'aliases' => $aliases, - 'userid' => $cur->id, + 'userid' => $this->scoped->id, + 'join_policy' => $join_policy, + 'force_scope' => $force_scope, 'local' => true)); $this->group = $group; @@ -224,21 +148,4 @@ class NewgroupAction extends Action common_redirect($group->homeUrl(), 303); } } - - function nicknameExists($nickname) - { - $local = Local_group::staticGet('nickname', $nickname); - - if (!empty($local)) { - return true; - } - - $alias = Group_alias::staticGet('alias', $nickname); - - if (!empty($alias)) { - return true; - } - - return false; - } }