fixes auto-detecting UUID attributes

the continue (and later the early return) avoided proper looping over the
attribute candidates.

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

diff --git a/apps/user_ldap/lib/Access.php b/apps/user_ldap/lib/Access.php
index 3ebd053402bba9e7cd525333d7d9659b244b0f16..6bef997a42f9efe5215232e78e7b2a807b2f5aa5 100644 (file)
--- a/apps/user_ldap/lib/Access.php
+++ b/apps/user_ldap/lib/Access.php
@@ -1698,7 +1698,6 @@ class Access extends LDAPUtility {
                                        $this->connection->$uuidAttr = $attribute;
                                        return true;
                                }
-                               continue;
                        }
 
                        $value = $this->readAttribute($dn, $attribute);
@@ -1714,9 +1713,6 @@ class Access extends LDAPUtility {
                                $this->connection->$uuidAttr = $attribute;
                                $this->connection->writeToCache($uuidAttr, $attribute);
                                return true;
-                       } elseif ($value === false) {
-                               // record not available
-                               return false;
                        }
                }
                \OC::$server->getLogger()->debug('Could not autodetect the UUID attribute', ['app' => 'user_ldap']);