From fee95084ae1fddfc616db1e35ee4508cb8e02d57 Mon Sep 17 00:00:00 2001 From: Lukas Reschke Date: Fri, 15 Apr 2016 19:02:19 +0200 Subject: [PATCH] Rename `username` to `loginName` UID and login name are two different things. --- core/Controller/LoginController.php | 8 ++++---- core/templates/login.php | 2 +- tests/core/controller/LoginControllerTest.php | 6 +++--- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/core/Controller/LoginController.php b/core/Controller/LoginController.php index ef6e1974842..ba9a1fee23a 100644 --- a/core/Controller/LoginController.php +++ b/core/Controller/LoginController.php @@ -95,10 +95,10 @@ class LoginController extends Controller { $parameters['messages'] = $messages; if ($user !== '') { - $parameters['username'] = $user; + $parameters['loginName'] = $user; $parameters['user_autofocus'] = false; } else { - $parameters['username'] = ''; + $parameters['loginName'] = ''; $parameters['user_autofocus'] = true; } if (!empty($redirect_url)) { @@ -120,10 +120,10 @@ class LoginController extends Controller { $parameters['rememberLoginState'] = !empty($remember_login) ? $remember_login : 0; if ($user !== '') { - $parameters['username'] = $user; + $parameters['loginName'] = $user; $parameters['user_autofocus'] = false; } else { - $parameters['username'] = ''; + $parameters['loginName'] = ''; $parameters['user_autofocus'] = true; } diff --git a/core/templates/login.php b/core/templates/login.php index f683323337d..8405bac6890 100644 --- a/core/templates/login.php +++ b/core/templates/login.php @@ -41,7 +41,7 @@ script('core', [

autocomplete="on" autocapitalize="off" autocorrect="off" required> diff --git a/tests/core/controller/LoginControllerTest.php b/tests/core/controller/LoginControllerTest.php index 186b8c4c5bb..f0ffe57d7b4 100644 --- a/tests/core/controller/LoginControllerTest.php +++ b/tests/core/controller/LoginControllerTest.php @@ -104,7 +104,7 @@ class LoginControllerTest extends TestCase { 'MessageArray1', 'MessageArray2', ], - 'username' => '', + 'loginName' => '', 'user_autofocus' => true, 'canResetPassword' => true, 'alt_login' => [], @@ -162,7 +162,7 @@ class LoginControllerTest extends TestCase { 'login', [ 'messages' => [], - 'username' => 'LdapUser', + 'loginName' => 'LdapUser', 'user_autofocus' => false, 'canResetPassword' => $expectedResult, 'alt_login' => [], @@ -200,7 +200,7 @@ class LoginControllerTest extends TestCase { 'login', [ 'messages' => [], - 'username' => '0', + 'loginName' => '0', 'user_autofocus' => false, 'canResetPassword' => false, 'alt_login' => [],