Merge pull request #16 from strebl/fix-15

Fix #15 - ErrorException if you try to login an account that don't exist and primarygroup is set to true
strebl · Jul 17, 2015 · 3cc8d5a · 3cc8d5a
2 parents b74dec1 + a6d023b
commit 3cc8d5a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/LdapAuthUserProvider.php b/src/LdapAuthUserProvider.php
@@ -246,7 +246,7 @@ protected function getPrimaryGroup($groupList)
     {
         $groups = explode(',', $groupList);
 
-        return substr($groups[1], '3');
+        return isset($groups[1]) ? substr($groups[1], '3') : null;
     }
 
     /**

diff --git a/tests/LdapAuthUserProviderTest.php b/tests/LdapAuthUserProviderTest.php
@@ -22,7 +22,8 @@ public function setUp()
 
         $this->config = [
             'fields'   => [
-                'groups' => 'groups',
+                'groups'       => 'groups',
+                'primarygroup' => 'primarygroup',
             ],
             'userlist' => false,
             'group'    => [],