summaryrefslogtreecommitdiffstats
path: root/lib
diff options
context:
space:
mode:
authorThomas Müller <thomas.mueller@tmit.eu>2014-02-14 21:53:03 +0100
committerThomas Müller <thomas.mueller@tmit.eu>2014-02-14 21:53:03 +0100
commit41f0ad2d1397af147fa9068c1236b53b5b8a05bd (patch)
tree95ba1199044032b05ab7295e459f3eafa9ec26a7 /lib
parente2c0c0e256be02b837eee2c8f30ad3318cbb5385 (diff)
parentb35f679483f2b8f1dab56b903ca2e942ac4606ff (diff)
downloadnextcloud-server-41f0ad2d1397af147fa9068c1236b53b5b8a05bd.tar.gz
nextcloud-server-41f0ad2d1397af147fa9068c1236b53b5b8a05bd.zip
Merge pull request #7204 from owncloud/groupexists-duplicate
Remove duplicate call to groupExists
Diffstat (limited to 'lib')
-rw-r--r--lib/private/group/manager.php14
1 files changed, 6 insertions, 8 deletions
diff --git a/lib/private/group/manager.php b/lib/private/group/manager.php
index bf469d51d12..9b433b64fd4 100644
--- a/lib/private/group/manager.php
+++ b/lib/private/group/manager.php
@@ -76,12 +76,7 @@ class Manager extends PublicEmitter {
if (isset($this->cachedGroups[$gid])) {
return $this->cachedGroups[$gid];
}
- foreach ($this->backends as $backend) {
- if ($backend->groupExists($gid)) {
- return $this->getGroupObject($gid);
- }
- }
- return null;
+ return $this->getGroupObject($gid);
}
protected function getGroupObject($gid) {
@@ -91,6 +86,9 @@ class Manager extends PublicEmitter {
$backends[] = $backend;
}
}
+ if (count($backends) === 0) {
+ return null;
+ }
$this->cachedGroups[$gid] = new Group($gid, $backends, $this->userManager, $this);
return $this->cachedGroups[$gid];
}
@@ -110,8 +108,8 @@ class Manager extends PublicEmitter {
public function createGroup($gid) {
if (!$gid) {
return false;
- } else if ($this->groupExists($gid)) {
- return $this->get($gid);
+ } else if ($group = $this->get($gid)) {
+ return $group;
} else {
$this->emit('\OC\Group', 'preCreate', array($gid));
foreach ($this->backends as $backend) {