getXbyY can still return false, e.g. when using ldap write support

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

diff --git a/apps/user_ldap/lib/Mapping/AbstractMapping.php b/apps/user_ldap/lib/Mapping/AbstractMapping.php
index 6fd07f5f483342f4ebab5b68bd13e97095688e2f..a4a3705e2238b333feba209379903d628a1ca709 100644 (file)
--- a/apps/user_ldap/lib/Mapping/AbstractMapping.php
+++ b/apps/user_ldap/lib/Mapping/AbstractMapping.php
@@ -123,8 +123,7 @@ abstract class AbstractMapping {
         */
        public function getDNByName($name) {
                $dn = array_search($name, $this->cache);
-               if ($dn === false) {
-                       $dn = $this->getXbyY('ldap_dn', 'owncloud_name', $name);
+               if ($dn === false && $dn = $this->getXbyY('ldap_dn', 'owncloud_name', $name)) {
                        $this->cache[$dn] = $name;
                }
                return $dn;