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 0cf3e4a91e3713468379d4eb604905c4f23df136..e7d34b16fa9c7ae5b51757b1580f1a50fd9c4e99 100644 (file)
--- a/apps/user_ldap/lib/Access.php
+++ b/apps/user_ldap/lib/Access.php
@@ -1730,7 +1730,6 @@ class Access extends LDAPUtility {
                                        $this->connection->$uuidAttr = $attribute;
                                        return true;
                                }
-                               continue;
                        }
 
                        $value = $this->readAttribute($dn, $attribute);
@@ -1746,9 +1745,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']);