From 443bc17f558c71b7e9c73b4249eec248521e45bb Mon Sep 17 00:00:00 2001 From: Git'Fellow <12234510+solracsf@users.noreply.github.com> Date: Tue, 31 Mar 2026 11:01:34 +0200 Subject: [PATCH] fix: update email button text to 'Open shared item' Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> --- apps/sharebymail/lib/ShareByMailProvider.php | 4 ++-- apps/sharebymail/tests/ShareByMailProviderTest.php | 12 ++++++------ 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/apps/sharebymail/lib/ShareByMailProvider.php b/apps/sharebymail/lib/ShareByMailProvider.php index d28f7c51327a9..55b69e2ec42df 100644 --- a/apps/sharebymail/lib/ShareByMailProvider.php +++ b/apps/sharebymail/lib/ShareByMailProvider.php @@ -354,7 +354,7 @@ protected function sendEmail(IShare $share, array $emails): void { } $emailTemplate->addBodyButton( - $this->l->t('Open %s', [$filename]), + $this->l->t('Open shared item'), $link ); @@ -528,7 +528,7 @@ protected function sendNote(IShare $share): void { $link = $this->urlGenerator->linkToRouteAbsolute('files_sharing.sharecontroller.showShare', ['token' => $share->getToken()]); $emailTemplate->addBodyButton( - $this->l->t('Open %s', [$filename]), + $this->l->t('Open shared item'), $link ); diff --git a/apps/sharebymail/tests/ShareByMailProviderTest.php b/apps/sharebymail/tests/ShareByMailProviderTest.php index 8f70516f12cc5..80b29a6812f08 100644 --- a/apps/sharebymail/tests/ShareByMailProviderTest.php +++ b/apps/sharebymail/tests/ShareByMailProviderTest.php @@ -1328,7 +1328,7 @@ public function testSendMailNotificationWithSameUserAndUserEmail(): void { ->expects($this->once()) ->method('addBodyButton') ->with( - 'Open file.txt', + 'Open shared item', 'https://example.com/file.txt' ); $message @@ -1450,7 +1450,7 @@ public function testSendMailNotificationWithSameUserAndUserEmailAndNote(): void ->expects($this->once()) ->method('addBodyButton') ->with( - 'Open file.txt', + 'Open shared item', 'https://example.com/file.txt' ); $message @@ -1577,7 +1577,7 @@ public function testSendMailNotificationWithSameUserAndUserEmailAndExpiration(): ->expects($this->once()) ->method('addBodyButton') ->with( - 'Open file.txt', + 'Open shared item', 'https://example.com/file.txt' ); $message @@ -1684,7 +1684,7 @@ public function testSendMailNotificationWithDifferentUserAndNoUserEmail(): void ->expects($this->once()) ->method('addBodyButton') ->with( - 'Open file.txt', + 'Open shared item', 'https://example.com/file.txt' ); $message @@ -1781,7 +1781,7 @@ public function testSendMailNotificationWithSameUserAndUserEmailAndReplyToDesact ->expects($this->once()) ->method('addBodyButton') ->with( - 'Open file.txt', + 'Open shared item', 'https://example.com/file.txt' ); $message @@ -1882,7 +1882,7 @@ public function testSendMailNotificationWithDifferentUserAndNoUserEmailAndReplyT ->expects($this->once()) ->method('addBodyButton') ->with( - 'Open file.txt', + 'Open shared item', 'https://example.com/file.txt' ); $message