Skip to content

Commit 2d37c79

Browse files
committed
[Mime] use isRendered method to avoid rendering an email twice
1 parent cb5fdaf commit 2d37c79

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

src/Symfony/Bridge/Twig/Mime/BodyRenderer.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ public function render(Message $message): void
4545
return;
4646
}
4747

48-
if (null === $message->getTextTemplate() && null === $message->getHtmlTemplate()) {
48+
if ($message->isRendered()) {
4949
// email has already been rendered
5050
return;
5151
}

src/Symfony/Bridge/Twig/Tests/Mime/BodyRendererTest.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,10 +105,14 @@ public function testRenderedOnce()
105105
;
106106
$email->textTemplate('text');
107107

108+
$this->assertFalse($email->isRendered());
108109
$renderer->render($email);
110+
$this->assertTrue($email->isRendered());
111+
109112
$this->assertEquals('Text', $email->getTextBody());
110113

111114
$email->text('reset');
115+
$this->assertTrue($email->isRendered());
112116

113117
$renderer->render($email);
114118
$this->assertEquals('reset', $email->getTextBody());

0 commit comments

Comments
 (0)