Skip to content

Commit 9d899c2

Browse files
committed
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 <fenn25.fn@gmail.com>
1 parent 78ce667 commit 9d899c2

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

apps/sharebymail/lib/ShareByMailProvider.php

+4-1
Original file line numberDiff line numberDiff line change
@@ -567,7 +567,10 @@ protected function sendPasswordToOwner(IShare $share, string $password): bool {
567567
$initiator = $this->userManager->get($share->getSharedBy());
568568
$initiatorEMailAddress = ($initiator instanceof IUser) ? $initiator->getEMailAddress() : null;
569569
$initiatorDisplayName = ($initiator instanceof IUser) ? $initiator->getDisplayName() : $share->getSharedBy();
570-
$shareWith = $share->getSharedWith();
570+
$shareWith = $this->getSharedWithEmails($share);
571+
if (is_array($shareWith)) {
572+
$shareWith = implode(', ', $shareWith);
573+
}
571574

572575
if ($initiatorEMailAddress === null) {
573576
throw new \Exception(

0 commit comments

Comments
 (0)