Remove old calls to class GroupPortalManager

main/admin/index.php

@@ -107,8 +107,6 @@ if (api_is_platform_admin()) {
         array('url' => 'user_import.php', 'label' => get_lang('ImportUserListXMLCSV')),
         array('url' => 'user_update_import.php', 'label' => get_lang('EditUserListCSV')),
     );
-    //$items[] = array('url' => 'group_add.php', 'label' => get_lang('AddGroups'));
-    //$items[] = array('url' => 'group_list.php', 'label' => get_lang('GroupList'));
 
     if (isset($extAuthSource) && isset($extAuthSource['extldap']) && count($extAuthSource['extldap']) > 0) {
         $items[] = array('url' => 'ldap_users_list.php', 'label' => get_lang('ImportLDAPUsersIntoPlatform'));

main/inc/lib/group_portal_manager.lib.php

@@ -975,6 +975,7 @@ class GroupPortalManager
      * with dirname() or the file with basename(). This also works for the
      * functions dealing with the user's productions, as they are located in
      * the same directory.
+     * @internal Don't delete this function
      * @param	integer	$id
      * @param	string	$type Type of path to return (can be 'system', 'web')
      * @param	bool	$preview Whether we want to have the directory name returned 'as if' there was a file or not

main/inc/lib/usergroup.lib.php

@@ -2233,7 +2233,7 @@ class UserGroup extends Model
         }
 
         foreach ($groupsId as $groupId) {
-            $groupUsers = GroupPortalManager::get_users_by_group($groupId);
+            $groupUsers = self::get_users_by_group($groupId);
 
             if (empty($groupUsers)) {
                 continue;