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

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
6 years ago · 0cf57d1ed4
parent b5cf0467cd
commit 0cf57d1ed4
1 changed files with 1 additions and 2 deletions
--- 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;