From: Vinicius Cubas Brand <viniciuscb@gmail.com>
Date: Thu, 14 Mar 2019 14:59:27 +0000 (-0300)
Subject: LDAP plugin: force createUser to return new user's DN
X-Git-Tag: v16.0.0beta1~6^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=61572a5b2e92182c6fea17855abd9b2b4f942334;p=nextcloud-server.git

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 <viniciuscb@gmail.com>
---

diff --git a/apps/user_ldap/lib/User_LDAP.php b/apps/user_ldap/lib/User_LDAP.php
index 855c13e13fb..e69eafecc86 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;