public/static/js/mapguesser.js

@@ -89,9 +89,6 @@ var MapGuesser = {
 
                     formError.style.display = 'block';
                     formError.innerHTML = this.response.error.errorText;
-                    if (typeof grecaptcha !== 'undefined') {
-                        grecaptcha.reset();
-                    }
 
                     return;
                 }

src/Controller/LoginController.php

@@ -364,24 +364,11 @@ class LoginController
             $newUser->setPlainPassword(\Container::$request->post('password'));
         }
 
-        if (strlen(\Container::$request->post('username')) > 0) {
+        if (strlen(\Container::$request->post('username')) > 0 && preg_match('/^[a-zA-Z0-9_\-\.]+$/', \Container::$request->post('username')) !== 1) {
-            $username = \Container::$request->post('username');
+            return new JsonContent(['error' => ['errorText' => 'Username can contain only english letters, digits, - (hyphen), . (dot), _ (underscore).']]);
-
-            if (preg_match('/^[a-zA-Z0-9_\-\.]+$/', $username) !== 1) {
-                return new JsonContent(['error' => ['errorText' => 'Username can contain only english letters, digits, - (hyphen), . (dot), _ (underscore).']]);
-            }
-
-            if ($this->userRepository->getByUsername($username) !== null) {
-                return new JsonContent(['error' => ['errorText' => 'The given username is already taken.']]);
-            }
-        } else {
-            $usernameGenerator = new UsernameGenerator();
-            do {
-                $username = $usernameGenerator->generate();
-            } while ($this->userRepository->getByUsername($username));
         }
 
-        $newUser->setUsername($username);
+        $newUser->setUsername(strlen(\Container::$request->post('username')) > 0 ? \Container::$request->post('username') : (new UsernameGenerator())->generate());
         $newUser->setCreatedDate(new DateTime());
 
         \Container::$persistentDataManager->saveToDb($newUser);