Cleaned up some comments about group acl hooks.

git-svn-id: http://code.elgg.org/elgg/trunk@5842 36083f99-b078-4883-b0ff-0f9b5a30f544

diff --git a/mod/groups/start.php b/mod/groups/start.php
index 9dfe74bf7a745660bc975377a54a9e03afdecbd8..1ca0af9aa170594b10ef9fb1c68272e0a507cce3 100644 (file)
--- a/mod/groups/start.php
+++ b/mod/groups/start.php
@@ -426,11 +426,11 @@
                        if ($page_owner->isMember($loggedin)) {
                                $returnvalue[$page_owner->group_acl] = elgg_echo('groups:group') . ': ' . $page_owner->name;
 
-                               // remove friends access
                                unset($returnvalue[ACCESS_FRIENDS]);
                        }
                } else {
-                       // remove all group access ids the user has
+                       // if the user owns the group, remove all access collections manually
+                       // this won't be a problem once the group itself owns the acl.
                        $groups = elgg_get_entities_from_relationship(array(
                                'relationship' => 'member',
                                'relationship_guid' => $loggedin->getGUID(),
@@ -580,13 +580,13 @@
 
                return $invitations;
        }
-       
+
        /**
         * Function to use on groups for access. It will house private, loggedin, public,
         * and the group itself. This is when you don't want other groups or channels in the access options available
         * Returns an array
         **/
-        
+
        function group_access_options($group){
                $access_array = array(0 => 'private',1 => 'logged in users',2 => 'public',$group->group_acl => 'Group: ' . $group->name );
                return $access_array;