From 61572a5b2e92182c6fea17855abd9b2b4f942334 Mon Sep 17 00:00:00 2001 From: Vinicius Cubas Brand Date: Thu, 14 Mar 2019 11:59:27 -0300 Subject: [PATCH] LDAP plugin: force createUser to return new user's DN LDAP plugins must change the createUser method to return the DN, as we need this to update the cache. Signed-off-by: Vinicius Cubas Brand --- apps/user_ldap/lib/User_LDAP.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/apps/user_ldap/lib/User_LDAP.php b/apps/user_ldap/lib/User_LDAP.php index 855c13e13f..e69eafecc8 100644 --- a/apps/user_ldap/lib/User_LDAP.php +++ b/apps/user_ldap/lib/User_LDAP.php @@ -615,6 +615,7 @@ class User_LDAP extends BackendUtility implements \OCP\IUserBackend, \OCP\UserIn * create new user * @param string $username username of the new user * @param string $password password of the new user + * @throws \UnexpectedValueException * @return bool */ public function createUser($username, $password) { @@ -624,7 +625,7 @@ class User_LDAP extends BackendUtility implements \OCP\IUserBackend, \OCP\UserIn //updates user mapping $this->access->dn2ocname($dn, $username, true); } else { - throw new \Exception("LDAP Plugin: Method createUser changed to return the user DN instead of boolean."); + throw new \UnexpectedValueException("LDAP Plugin: Method createUser changed to return the user DN instead of boolean."); } } return (bool) $dn;