Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d38b6d2c authored by Akhil's avatar Akhil 🙂
Browse files

Check is username taken at registerUser

parent 6f5b1fe4

lib/Controller/AccountController.php

+1 −0
Original line number Diff line number Diff line
@@ -216,6 +216,7 @@ class AccountController extends Controller { 
	 * @return \OCP\AppFramework\Http\DataResponse

	 */

	public function checkUsernameAvailable(string $username) : DataResponse {

		$this->session->remove(self::SESSION_USERNAME_CHECK);

		$response = new DataResponse();

		$response->setStatus(400);

lib/Service/UserService.php

+1 −1
Original line number Diff line number Diff line
@@ -249,7 +249,7 @@ class UserService { 
	 */

	public function registerUser(string $displayname, string $recoveryEmail, string $username, string $userEmail, string $password): void {

		

		if ($this->userExists($username)) {

		if ($this->userExists($username) || $this->isUsernameTaken($username)) {

			throw new Exception("Username '$username' is already taken.");

		}

		if (!empty($recoveryEmail)) {