mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #25707 from nextcloud/bugfix/noid/send-emails-of-password-reset-to-displayname 

Send emails on password reset to the displayname
This commit is contained in:
Roeland Jago Douma 2021-02-18 14:04:09 +01:00 committed by GitHub
parent d79cc8ea6d 6ed4aaeeea
commit bbb39cf3de
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
core/Controller/LostController.php
View File

@ -378,7 +378,7 @@ class LostController extends Controller {
try {
$message = $this->mailer->createMessage();
$message->setTo([$email => $user->getUID()]);
$message->setTo([$email => $user->getDisplayName()]);
$message->setFrom([$this->from => $this->defaults->getName()]);
$message->useTemplate($emailTemplate);
$this->mailer->send($message);

9
tests/Core/Controller/LostControllerTest.php
View File

@ -93,6 +93,9 @@ class LostControllerTest extends \Test\TestCase {
$this->existingUser->expects($this->any())
->method('getUID')
->willReturn('ExistingUser');
$this->existingUser->expects($this->any())
->method('getDisplayName')
->willReturn('Existing User');
$this->existingUser->expects($this->any())
->method('isEnabled')
->willReturn(true);
@ -344,7 +347,7 @@ class LostControllerTest extends \Test\TestCase {
$message
->expects($this->at(0))
->method('setTo')
->with(['test@example.com' => 'ExistingUser']);
->with(['test@example.com' => 'Existing User']);
$message
->expects($this->at(1))
->method('setFrom')
@ -422,7 +425,7 @@ class LostControllerTest extends \Test\TestCase {
$message
->expects($this->at(0))
->method('setTo')
->with(['test@example.com' => 'ExistingUser']);
->with(['test@example.com' => 'Existing User']);
$message
->expects($this->at(1))
->method('setFrom')
@ -494,7 +497,7 @@ class LostControllerTest extends \Test\TestCase {
$message
->expects($this->at(0))
->method('setTo')
->with(['test@example.com' => 'ExistingUser']);
->with(['test@example.com' => 'Existing User']);
$message
->expects($this->at(1))
->method('setFrom')