X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Feditgroup.php;h=7b499d00ffdff60819788b6b0adbcae0f8ba0fe1;hb=0a20abf1d8a9a068df9310d6903cc303f39b25ed;hp=10ce4a211236e33e1bf14ac651e3584f79afdf56;hpb=6ed66d9c7617332e9ea253e1f48583b4d9dbf45c;p=quix0rs-gnu-social.git

diff --git a/actions/editgroup.php b/actions/editgroup.php
index 10ce4a2112..7b499d00ff 100644
--- a/actions/editgroup.php
+++ b/actions/editgroup.php
@@ -59,14 +59,13 @@ class EditgroupAction extends GroupAction
      * 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 edit a group while not logged in.
             $this->clientError(_('You must be logged in to create a group.'));
-            return false;
         }
 
         $nickname_arg = $this->trimmed('nickname');
@@ -77,13 +76,11 @@ class EditgroupAction extends GroupAction
         if ($nickname_arg != $nickname) {
             $args = array('nickname' => $nickname);
             common_redirect(common_local_url('editgroup', $args), 301);
-            return false;
         }
 
         if (!$nickname) {
             // TRANS: Client error displayed trying to edit a group while not proving a nickname for the group to edit.
             $this->clientError(_('No nickname.'), 404);
-            return false;
         }
 
         $groupid = $this->trimmed('groupid');
@@ -100,7 +97,6 @@ class EditgroupAction extends GroupAction
         if (!$this->group) {
             // TRANS: Client error displayed trying to edit a non-existing group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $cur = common_current_user();
@@ -108,7 +104,6 @@ class EditgroupAction extends GroupAction
         if (!$cur->isAdmin($this->group)) {
             // TRANS: Client error displayed trying to edit a group while not being a group admin.
             $this->clientError(_('You must be an admin to edit the group.'), 403);
-            return false;
         }
 
         return true;
@@ -119,13 +114,11 @@ class EditgroupAction extends GroupAction
      *
      * On GET, show the form. On POST, try to save the group.
      *
-     * @param array $args unused
-     *
      * @return void
      */
-    function handle($args)
+    protected function handle()
     {
-        parent::handle($args);
+        parent::handle();
         if ($_SERVER['REQUEST_METHOD'] == 'POST') {
             $this->trySave();
         } else {
@@ -177,7 +170,7 @@ class EditgroupAction extends GroupAction
                 $nickname = Nickname::normalize($nickname, true);
             } catch (NicknameTakenException $e) {
                 // Abort only if the nickname is occupied by _another_ group
-                if ($e->profile->id != $this->group->id) {
+                if ($e->profile->id != $this->group->profile_id) {
                     $this->showForm($e->getMessage());
                     return;
                 }
@@ -270,21 +263,13 @@ class EditgroupAction extends GroupAction
                 $this->serverError(_('Could not create aliases.'));
             }
 
-            if ($nickname != $orig->nickname) {
-                common_log(LOG_INFO, "Saving local group info.");
-                $local = Local_group::getKV('group_id', $this->group->id);
-                $local->setNickname($nickname);
-            }
-
             $this->group->query('COMMIT');
 
             Event::handle('EndGroupSaveForm', array($this));
         }
 
         if ($this->group->nickname != $orig->nickname) {
-            common_redirect(common_local_url('editgroup',
-                                             array('nickname' => $nickname)),
-                            303);
+            common_redirect(common_local_url('editgroup', array('nickname' => $nickname)), 303);
         } else {
             // TRANS: Group edit form success message.
             $this->showForm(_('Options saved.'));