From 72b6508d910e5286eccfd3bf55ff506bc4114695 Mon Sep 17 00:00:00 2001 From: nfebe Date: Tue, 28 Jan 2025 23:17:59 +0100 Subject: [PATCH] fix: Show recipient email addresses in share owner notification email Previously, the share owner notification email did not display the recipient email addresses, making it difficult for the owner to know who the share was sent to. This fix ensures that the recipient email addresses are included in the notification email. Signed-off-by: nfebe --- apps/sharebymail/lib/ShareByMailProvider.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/sharebymail/lib/ShareByMailProvider.php b/apps/sharebymail/lib/ShareByMailProvider.php index 5e0637c1a3ac0..d6a12c8312353 100644 --- a/apps/sharebymail/lib/ShareByMailProvider.php +++ b/apps/sharebymail/lib/ShareByMailProvider.php @@ -567,7 +567,7 @@ protected function sendPasswordToOwner(IShare $share, string $password): bool { $initiator = $this->userManager->get($share->getSharedBy()); $initiatorEMailAddress = ($initiator instanceof IUser) ? $initiator->getEMailAddress() : null; $initiatorDisplayName = ($initiator instanceof IUser) ? $initiator->getDisplayName() : $share->getSharedBy(); - $shareWith = $share->getSharedWith(); + $shareWith = implode(', ', $this->getSharedWithEmails($share)); if ($initiatorEMailAddress === null) { throw new \Exception(