Merge pull request #7590 from nextcloud/flow-hide-stay-logged-in

Hide stay logged in checkbox when flow authentication is used
nextcloud · Jan 2, 2018 · 37ed5b5 · 37ed5b5
2 parents 9e49b6e + f5f6ed6
commit 37ed5b5
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 0 deletions.
diff --git a/core/Controller/LoginController.php b/core/Controller/LoginController.php
@@ -179,6 +179,7 @@ public function showLoginForm($user, $redirect_url, $remember_login) {
 
 		$parameters['alt_login'] = OC_App::getAlternativeLogIns();
 		$parameters['rememberLoginState'] = !empty($remember_login) ? $remember_login : 0;
+		$parameters['hideRemeberLoginState'] = !empty($redirect_url) && $this->session->exists('client.flow.state.token');
 
 		if (!is_null($user) && $user !== '') {
 			$parameters['loginName'] = $user;

diff --git a/core/templates/login.php b/core/templates/login.php
@@ -70,6 +70,7 @@
 		<?php } ?>
 
 		<div class="login-additional">
+			<?php if (!$_['hideRemeberLoginState']) { ?>
 			<div class="remember-login-container">
 				<?php if ($_['rememberLoginState'] === 0) { ?>
 				<input type="checkbox" name="remember_login" value="1" id="remember_login" class="checkbox checkbox--white">
@@ -78,6 +79,7 @@
 				<?php } ?>
 				<label for="remember_login"><?php p($l->t('Stay logged in')); ?></label>
 			</div>
+			<?php } ?>
 			<?php if (!empty($_['canResetPassword'])) { ?>
 			<div class="lost-password-container">
 				<a id="lost-password" href="<?php p($_['resetPasswordLink']); ?>">

diff --git a/tests/Core/Controller/LoginControllerTest.php b/tests/Core/Controller/LoginControllerTest.php
@@ -182,12 +182,43 @@ public function testShowLoginFormWithErrorsInSession() {
 				'alt_login' => [],
 				'rememberLoginState' => 0,
 				'resetPasswordLink' => null,
+				'hideRemeberLoginState' => false,
 			],
 			'guest'
 		);
 		$this->assertEquals($expectedResponse, $this->loginController->showLoginForm('', '', ''));
 	}
 
+	public function testShowLoginFormForFlowAuth() {
+		$this->userSession
+			->expects($this->once())
+			->method('isLoggedIn')
+			->willReturn(false);
+		$this->session
+			->expects($this->once())
+			->method('exists')
+			->with('client.flow.state.token')
+			->willReturn(true);
+
+		$expectedResponse = new TemplateResponse(
+			'core',
+			'login',
+			[
+				'messages' => [],
+				'redirect_url' => 'login/flow',
+				'loginName' => '',
+				'user_autofocus' => true,
+				'canResetPassword' => true,
+				'alt_login' => [],
+				'rememberLoginState' => 0,
+				'resetPasswordLink' => null,
+				'hideRemeberLoginState' => true,
+			],
+			'guest'
+		);
+		$this->assertEquals($expectedResponse, $this->loginController->showLoginForm('', 'login/flow', ''));
+	}
+
 	/**
 	 * @return array
 	 */
@@ -240,6 +271,7 @@ public function testShowLoginFormWithPasswordResetOption($canChangePassword,
 				'alt_login' => [],
 				'rememberLoginState' => 0,
 				'resetPasswordLink' => false,
+				'hideRemeberLoginState' => false,
 			],
 			'guest'
 		);
@@ -278,6 +310,7 @@ public function testShowLoginFormForUserNamedNull() {
 				'alt_login' => [],
 				'rememberLoginState' => 0,
 				'resetPasswordLink' => false,
+				'hideRemeberLoginState' => false,
 			],
 			'guest'
 		);