From 066ffc5fb70ef1c78dfec6240fec4736827638b4 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(