From cba56d78dff7e140a0bd21c4ed30bdc8342a5e53 Mon Sep 17 00:00:00 2001
From: Greta Doci <gretadoci@gmail.com>
Date: Fri, 17 May 2019 13:00:08 +0200
Subject: [PATCH] Fix 2fa icons in dark theme, fix #13643

Signed-off-by: Greta Doci <gretadoci@gmail.com>
---
 lib/private/Settings/Personal/Security.php    |  8 ++++++++
 .../templates/settings/personal/security.php  | 20 ++++++++++++++++---
 2 files changed, 25 insertions(+), 3 deletions(-)

diff --git a/lib/private/Settings/Personal/Security.php b/lib/private/Settings/Personal/Security.php
index eee1d9ae084bb..a009c6a12f491 100644
--- a/lib/private/Settings/Personal/Security.php
+++ b/lib/private/Settings/Personal/Security.php
@@ -42,6 +42,7 @@
 use OCP\IUserSession;
 use OCP\Session\Exceptions\SessionNotAvailableException;
 use OCP\Settings\ISettings;
+use OCP\IConfig;
 
 class Security implements ISettings {
 
@@ -69,6 +70,10 @@ class Security implements ISettings {
 	 * @var string|null
 	 */
 	private $uid;
+	/**
+	 *@var IConfig
+	 */
+	private $config;
 
 	public function __construct(IUserManager $userManager,
 								TwoFactorManager $providerManager,
@@ -76,6 +81,7 @@ public function __construct(IUserManager $userManager,
 								ProviderLoader $providerLoader,
 								IUserSession $userSession,
 								ISession $session,
+								IConfig $config,
 								IInitialStateService $initialStateService,
 								?string $UserId) {
 		$this->userManager = $userManager;
@@ -86,6 +92,7 @@ public function __construct(IUserManager $userManager,
 		$this->session = $session;
 		$this->initialStateService = $initialStateService;
 		$this->uid = $UserId;
+		$this->config = $config;
 	}
 
 	/**
@@ -108,6 +115,7 @@ public function getForm() {
 		return new TemplateResponse('settings', 'settings/personal/security', [
 			'passwordChangeSupported' => $passwordChangeSupported,
 			'twoFactorProviderData' => $this->getTwoFactorProviderData(),
+			'themedark' => $this->config->getUserValue($this->uid, 'accessibility', 'theme', false)
 		]);
 	}
 
diff --git a/settings/templates/settings/personal/security.php b/settings/templates/settings/personal/security.php
index b7983b7c5d919..0ab6858bc87c6 100644
--- a/settings/templates/settings/personal/security.php
+++ b/settings/templates/settings/personal/security.php
@@ -66,15 +66,29 @@
 	   href="<?php p(link_to_docs('user-2fa')); ?>"></a>
 	<p class="settings-hint"><?php p($l->t('Use a second factor besides your password to increase security for your account.'));?></p>
 	<ul>
-	<?php foreach ($_['twoFactorProviderData']['providers'] as $data) { ?>
+	<?php foreach ($_['twoFactorProviderData']['providers']  as $data) { ?>
 		<li>
 			<?php
+
 			/** @var \OCP\Authentication\TwoFactorAuth\IProvidesPersonalSettings $provider */
 			$provider = $data['provider'];
+			//Handle 2FA provider icons and theme
 			if ($provider instanceof \OCP\Authentication\TwoFactorAuth\IProvidesIcons) {
-				$icon = $provider->getDarkIcon();
+				if ($_['themedark']) {
+					$icon = $provider->getLightIcon();
+				}
+				else {
+					$icon = $provider->getDarkIcon();
+				}
+				//fallback icon if the 2factor provider doesn't provide an icon.
 			} else {
-				$icon = image_path('core', 'actions/password.svg');
+				if ($_['themedark']) {
+					$icon = image_path('core', 'actions/password-white.svg');
+				}
+				else {
+					$icon = image_path('core', 'actions/password.svg');
+				}
+
 			}
 			/** @var \OCP\Authentication\TwoFactorAuth\IPersonalProviderSettings $settings */
 			$settings = $data['settings'];