sascha
/
nextcloud-server
mirror of https://github.com/nextcloud/server.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #4507 from owncloud/existingusersfromgroup 

return only existing users in group
remotes/origin/stable6
blizzz 2013-08-27 03:47:56 +07:00
parent ea446544ef 1f5a55ddff
commit 163269805a
2 changed files with 29 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
lib/group/group.php
Unescape Escape View File

@ -62,7 +62,6 @@ class Group {
return $this->users;
}
$users = array();
$userIds = array();
foreach ($this->backends as $backend) {
$diff = array_diff(
@ -74,11 +73,8 @@ class Group {
}
}
foreach ($userIds as $userId) {
$users[] = $this->userManager->get($userId);
}
$this->users = $users;
return $users;
$this->users = $this->getVerifiedUsers($userIds);
return $this->users;
}
/**
@ -113,7 +109,7 @@ class Group {
if ($backend->implementsActions(OC_GROUP_BACKEND_ADD_TO_GROUP)) {
$backend->addToGroup($user->getUID(), $this->gid);
if ($this->users) {
$this->users[] = $user;
$this->users[$user->getUID()] = $user;
}
if ($this->emitter) {
$this->emitter->emit('\OC\Group', 'postAddUser', array($this, $user));
@ -172,9 +168,7 @@ class Group {
if (!is_null($offset)) {
$offset -= count($userIds);
}
foreach ($userIds as $userId) {
$users[$userId] = $this->userManager->get($userId);
}
$users += $this->getVerifiedUsers($userIds);
if (!is_null($limit) and $limit <= 0) {
return array_values($users);
}
@ -191,7 +185,6 @@ class Group {
* @return \OC\User\User[]
*/
public function searchDisplayName($search, $limit = null, $offset = null) {
$users = array();
foreach ($this->backends as $backend) {
if ($backend->implementsActions(OC_GROUP_BACKEND_GET_DISPLAYNAME)) {
$userIds = array_keys($backend->displayNamesInGroup($this->gid, $search, $limit, $offset));
@ -204,9 +197,7 @@ class Group {
if (!is_null($offset)) {
$offset -= count($userIds);
}
foreach ($userIds as $userId) {
$users[$userId] = $this->userManager->get($userId);
}
$users = $this->getVerifiedUsers($userIds);
if (!is_null($limit) and $limit <= 0) {
return array_values($users);
}
@ -235,4 +226,23 @@ class Group {
}
return $result;
}
/**
* @brief returns all the Users from an array that really exists
* @param $userIds an array containing user IDs
* @return an Array with the userId as Key and \OC\User\User as value
*/
private function getVerifiedUsers($userIds) {
if(!is_array($userIds)) {
return array();
}
$users = array();
foreach ($userIds as $userId) {
$user = $this->userManager->get($userId);
if(!is_null($user)) {
$users[$userId] = $user;
}
}
return $users;
}
}

10
tests/lib/group/group.php
Unescape Escape View File

@ -43,8 +43,8 @@ class Group extends \PHPUnit_Framework_TestCase {
$users = $group->getUsers();
$this->assertEquals(2, count($users));
$user1 = $users[0];
$user2 = $users[1];
$user1 = $users['user1'];
$user2 = $users['user2'];
$this->assertEquals('user1', $user1->getUID());
$this->assertEquals('user2', $user2->getUID());
}
@ -68,9 +68,9 @@ class Group extends \PHPUnit_Framework_TestCase {
$users = $group->getUsers();
$this->assertEquals(3, count($users));
$user1 = $users[0];
$user2 = $users[1];
$user3 = $users[2];
$user1 = $users['user1'];
$user2 = $users['user2'];
$user3 = $users['user3'];
$this->assertEquals('user1', $user1->getUID());
$this->assertEquals('user2', $user2->getUID());
$this->assertEquals('user3', $user3->getUID());