From 0fe4b717dcc5ad50853e0a887bfa60416657988d Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Mon, 9 Sep 2019 14:40:14 +0200
Subject: [PATCH] Correctly use the displayname of the sharer in the
 notification

Signed-off-by: Joas Schilling <coding@schilljs.com>
---
 .../lib/Notification/Notifier.php             | 22 ++++++++++++++-----
 1 file changed, 16 insertions(+), 6 deletions(-)

diff --git a/apps/files_sharing/lib/Notification/Notifier.php b/apps/files_sharing/lib/Notification/Notifier.php
index 57b96be57b..52bb0eb823 100644
--- a/apps/files_sharing/lib/Notification/Notifier.php
+++ b/apps/files_sharing/lib/Notification/Notifier.php
@@ -165,7 +165,12 @@ class Notifier implements INotifier {
 					throw new AlreadyProcessedException();
 				}
 
-				$subject = $l->t('You received {share} as a share from {user}');
+				$sharer = $this->userManager->get($share->getSharedBy());
+				if (!$sharer instanceof IUser) {
+					throw new \InvalidArgumentException('Temporary failure');
+				}
+
+				$subject = $l->t('You received {share} as a share by {user}');
 				$subjectParameters = [
 					'share' => [
 						'type' => 'highlight',
@@ -174,8 +179,8 @@ class Notifier implements INotifier {
 					],
 					'user' =>  [
 						'type' => 'user',
-						'id' => $share->getShareOwner(),
-						'name' => $share->getShareOwner(),
+						'id' => $sharer->getUID(),
+						'name' => $sharer->getDisplayName(),
 					],
 				];
 				break;
@@ -196,7 +201,12 @@ class Notifier implements INotifier {
 					throw new AlreadyProcessedException();
 				}
 
-				$subject = $l->t('You received {share} to group {group} as a share from {user}');
+				$sharer = $this->userManager->get($share->getSharedBy());
+				if (!$sharer instanceof IUser) {
+					throw new \InvalidArgumentException('Temporary failure');
+				}
+
+				$subject = $l->t('You received {share} to group {group} as a share by {user}');
 				$subjectParameters = [
 					'share' => [
 						'type' => 'highlight',
@@ -210,8 +220,8 @@ class Notifier implements INotifier {
 					],
 					'user' =>  [
 						'type' => 'user',
-						'id' => $share->getShareOwner(),
-						'name' => $share->getShareOwner(),
+						'id' => $sharer->getUID(),
+						'name' => $sharer->getDisplayName(),
 					],
 				];
 				break;