X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Fnewgroup.php;h=07a302ef45befda654a12e088ec8839fe6e2a044;hb=261ccfac8699534ff584a2f93d5dcd384529d855;hp=9682b875cb0c666a058c0b241b4310d939cd2225;hpb=8eb66467165e107d0c80c64f6c429fed1d1101d5;p=quix0rs-gnu-social.git diff --git a/actions/newgroup.php b/actions/newgroup.php index 9682b875cb..07a302ef45 100644 --- a/actions/newgroup.php +++ b/actions/newgroup.php @@ -24,11 +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')) { +if (!defined('STATUSNET')) { exit(1); } @@ -43,9 +44,13 @@ 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; + + function getGroup() { + return $this->group; + } function title() { @@ -56,117 +61,67 @@ class NewgroupAction extends Action /** * Prepare to run */ - function prepare($args) + protected function prepare(array $args=array()) { 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); + $this->clientError(_('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(); - } - } - - function showForm($msg=null) - { - $this->msg = $msg; - $this->showPage(); - } - - function showContent() + public function showContent() { $form = new GroupEditForm($this); $form->show(); } - function showPageNotice() + public function showInstructions() { - 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.')); - } + $this->element('p', 'instructions', + // TRANS: Form instructions for group create form. + _('Use this form to create a new group.')); } - function trySave() + protected function handlePost() { + parent::handlePost(); + 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)) { + if (!is_null($homepage) && (strlen($homepage) > 0) && + !common_valid_http_url($homepage)) { // 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')))) { - // 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 +129,23 @@ 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 +153,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 +165,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; - } }