From: Jan-Christoph Borchardt <hey@jancborchardt.net>
Date: Tue, 16 Jul 2019 08:23:34 +0000 (+0200)
Subject: Add icon to twofactor_backupcodes notification
X-Git-Tag: v17.0.0beta1~143^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=refs%2Fpull%2F16417%2Fhead;p=nextcloud-server.git

Add icon to twofactor_backupcodes notification

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
---

diff --git a/apps/twofactor_backupcodes/lib/Notifications/Notifier.php b/apps/twofactor_backupcodes/lib/Notifications/Notifier.php
index 658f23a7a93..e4e2bcebb68 100644
--- a/apps/twofactor_backupcodes/lib/Notifications/Notifier.php
+++ b/apps/twofactor_backupcodes/lib/Notifications/Notifier.php
@@ -60,6 +60,9 @@ class Notifier implements INotifier {
 				);
 
 				$notification->setLink($this->url->linkToRouteAbsolute('settings.PersonalSettings.index', ['section' => 'security']));
+
+				$notification->setIcon($this->url->getAbsoluteURL($this->url->imagePath('core', 'actions/password.svg')));
+
 				return $notification;
 
 			default:
diff --git a/apps/twofactor_backupcodes/tests/Unit/Notification/NotifierTest.php b/apps/twofactor_backupcodes/tests/Unit/Notification/NotifierTest.php
index 9a641eb9f85..0f03fba7de8 100644
--- a/apps/twofactor_backupcodes/tests/Unit/Notification/NotifierTest.php
+++ b/apps/twofactor_backupcodes/tests/Unit/Notification/NotifierTest.php
@@ -117,7 +117,7 @@ class NotifierTest extends TestCase {
 			->willReturnSelf();
 		$notification->expects($this->once())
 			->method('setParsedMessage')
-			->with('You have enabled two-factor authentication but have not yet generated backup codes. Be sure to do this in case you lose access to your second factor.')
+			->with('You enabled two-factor authentication but did not generate backup codes yet. They are needed to restore access to your account in case you lose your second factor.')
 			->willReturnSelf();
 
 		$this->url->expects($this->once())