Merge pull request #32247 from nextcloud/backport/31032/stable23

[stable23] Don't use plain URL on the email subject
nextcloud · May 16, 2022 · 9ff3b29 · 9ff3b29
2 parents adc0690 + c565f4a
commit 9ff3b29
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/apps/settings/lib/Hooks.php b/apps/settings/lib/Hooks.php
@@ -28,6 +28,7 @@
 
 use OCA\Settings\Activity\Provider;
 use OCP\Activity\IManager as IActivityManager;
+use OCP\Defaults;
 use OCP\IConfig;
 use OCP\IGroupManager;
 use OCP\IURLGenerator;
@@ -55,6 +56,8 @@ class Hooks {
 	protected $config;
 	/** @var IFactory */
 	protected $languageFactory;
+	/** @var Defaults */
+	protected $defaults;
 
 	public function __construct(IActivityManager $activityManager,
 								IGroupManager $groupManager,
@@ -63,7 +66,8 @@ public function __construct(IActivityManager $activityManager,
 								IURLGenerator $urlGenerator,
 								IMailer $mailer,
 								IConfig $config,
-								IFactory $languageFactory) {
+								IFactory $languageFactory,
+				   				Defaults $defaults) {
 		$this->activityManager = $activityManager;
 		$this->groupManager = $groupManager;
 		$this->userManager = $userManager;
@@ -72,6 +76,7 @@ public function __construct(IActivityManager $activityManager,
 		$this->mailer = $mailer;
 		$this->config = $config;
 		$this->languageFactory = $languageFactory;
+		$this->defaults = $defaults;
 	}
 
 	/**
@@ -93,6 +98,7 @@ public function onChangePassword($uid) {
 			->setType('personal_settings')
 			->setAffectedUser($user->getUID());
 
+		$instanceName = $this->defaults->getName();
 		$instanceUrl = $this->urlGenerator->getAbsoluteURL('/');
 		$language = $this->languageFactory->getUserLanguage($user);
 		$l = $this->languageFactory->get('settings', $language);
@@ -131,7 +137,7 @@ public function onChangePassword($uid) {
 				'instanceUrl' => $instanceUrl,
 			]);
 
-			$template->setSubject($l->t('Password for %1$s changed on %2$s', [$user->getDisplayName(), $instanceUrl]));
+			$template->setSubject($l->t('Password for %1$s changed on %2$s', [$user->getDisplayName(), $instanceName]));
 			$template->addHeader();
 			$template->addHeading($l->t('Password changed for %s', [$user->getDisplayName()]), false);
 			$template->addBodyText($text . ' ' . $l->t('If you did not request this, please contact an administrator.'));