Browse Source

fix(shares): Remove noisy text from emails

Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
pull/49847/head
Git'Fellow 11 months ago
parent
commit
20215fbd09
  1. 4
      apps/sharebymail/lib/ShareByMailProvider.php
  2. 24
      apps/sharebymail/tests/ShareByMailProviderTest.php
  3. 4
      lib/private/Share20/DefaultShareProvider.php

4
apps/sharebymail/lib/ShareByMailProvider.php

@ -352,10 +352,6 @@ class ShareByMailProvider extends DefaultShareProvider implements IShareProvider
);
}
$emailTemplate->addBodyText(
$this->l->t('Click the button below to open it.')
);
$emailTemplate->addBodyButton(
$this->l->t('Open %s', [$filename]),
$link

24
apps/sharebymail/tests/ShareByMailProviderTest.php

@ -1289,10 +1289,6 @@ class ShareByMailProviderTest extends TestCase {
->expects($this->once())
->method('addHeading')
->with('Mrs. Owner User shared file.txt with you');
$template
->expects($this->once())
->method('addBodyText')
->with('Click the button below to open it.');
$template
->expects($this->once())
->method('addBodyButton')
@ -1399,10 +1395,6 @@ class ShareByMailProviderTest extends TestCase {
->expects($this->once())
->method('addHeading')
->with('Mrs. Owner User shared file.txt with you');
$template
->expects($this->once())
->method('addBodyText')
->with('Click the button below to open it.');
$this->urlGenerator->expects($this->once())->method('imagePath')
->with('core', 'caldav/description.png')
@ -1525,10 +1517,6 @@ class ShareByMailProviderTest extends TestCase {
->expects($this->once())
->method('addHeading')
->with('Mrs. Owner User shared file.txt with you');
$template
->expects($this->once())
->method('addBodyText')
->with('Click the button below to open it.');
$expiration = new DateTime('2001-01-01');
$this->l->expects($this->once())
@ -1657,10 +1645,6 @@ class ShareByMailProviderTest extends TestCase {
->expects($this->once())
->method('addHeading')
->with('Mr. Initiator User shared file.txt with you');
$template
->expects($this->once())
->method('addBodyText')
->with('Click the button below to open it.');
$template
->expects($this->once())
->method('addBodyButton')
@ -1758,10 +1742,6 @@ class ShareByMailProviderTest extends TestCase {
->expects($this->once())
->method('addHeading')
->with('Mrs. Owner User shared file.txt with you');
$template
->expects($this->once())
->method('addBodyText')
->with('Click the button below to open it.');
$template
->expects($this->once())
->method('addBodyButton')
@ -1863,10 +1843,6 @@ class ShareByMailProviderTest extends TestCase {
->expects($this->once())
->method('addHeading')
->with('Mr. Initiator User shared file.txt with you');
$template
->expects($this->once())
->method('addBodyText')
->with('Click the button below to open it.');
$template
->expects($this->once())
->method('addBodyButton')

4
lib/private/Share20/DefaultShareProvider.php

@ -1520,10 +1520,6 @@ class DefaultShareProvider implements IShareProviderWithNotification, IShareProv
$emailTemplate->addBodyText(htmlspecialchars($note), $note);
}
$emailTemplate->addBodyText(
$l->t('Click the button below to open it.')
);
$emailTemplate->addBodyButton(
$l->t('Open %s', [$filename]),
$link

Loading…
Cancel
Save