summaryrefslogtreecommitdiff
path: root/app/Core/Ldap
diff options
context:
space:
mode:
authorFrederic Guillot <fred@kanboard.net>2016-07-02 18:23:47 -0400
committerFrederic Guillot <fred@kanboard.net>2016-07-02 18:23:47 -0400
commita8a8bfb0afdd105980181a5fa11701028ff5adfe (patch)
treeb2f7d22a03e660aa76b039703158811c3589fbe3 /app/Core/Ldap
parent43337d58c0be097ca510f2abd1497f13f25bda5b (diff)
Added more unit tests for LDAP user class
Diffstat (limited to 'app/Core/Ldap')
-rw-r--r--app/Core/Ldap/User.php8
1 files changed, 4 insertions, 4 deletions
diff --git a/app/Core/Ldap/User.php b/app/Core/Ldap/User.php
index 91b48530..4bc1f5f9 100644
--- a/app/Core/Ldap/User.php
+++ b/app/Core/Ldap/User.php
@@ -116,7 +116,7 @@ class User
*/
protected function getRole(array $groupIds)
{
- if ($this->hasGroupsNotConfigured()) {
+ if (! $this->hasGroupsConfigured()) {
return null;
}
@@ -278,14 +278,14 @@ class User
}
/**
- * Return true if LDAP Group mapping is not configured
+ * Return true if LDAP Group mapping are configured
*
* @access public
* @return boolean
*/
- public function hasGroupsNotConfigured()
+ public function hasGroupsConfigured()
{
- return !$this->getGroupAdminDn() && !$this->getGroupManagerDn();
+ return $this->getGroupAdminDn() || $this->getGroupManagerDn();
}
/**