Skip to content

Commit

Permalink
fix: Show recipient email addresses in share owner notification email
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
nfebe authored and AndyScherzinger committed Jan 29, 2025
1 parent e8844ca commit 066ffc5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion apps/sharebymail/lib/ShareByMailProvider.php
Original file line number Diff line number Diff line change
Expand Up @@ -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(
Expand Down

0 comments on commit 066ffc5

Please sign in to comment.