Commit2d37c79

committed

[Mime] use isRendered method to avoid rendering an email twice

1 parentcb5fdaf commit2d37c79Copy full SHA for 2d37c79

File tree

-1

lines changed

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

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ public function render(Message $message): void`
`45`	`45`	`return;`
`46`	`46`	`}`
`47`	`47`
`48`		`-if (null ===$message->getTextTemplate() &&null ===$message->getHtmlTemplate()) {`
	`48`	`+if ($message->isRendered()) {`
`49`	`49`	`// email has already been rendered`
`50`	`50`	`return;`
`51`	`51`	`}`

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -105,10 +105,14 @@ public function testRenderedOnce()`
`105`	`105`	`;`
`106`	`106`	`$email->textTemplate('text');`
`107`	`107`
	`108`	`+$this->assertFalse($email->isRendered());`
`108`	`109`	`$renderer->render($email);`
	`110`	`+$this->assertTrue($email->isRendered());`
	`111`	`+`
`109`	`112`	`$this->assertEquals('Text',$email->getTextBody());`
`110`	`113`
`111`	`114`	`$email->text('reset');`
	`115`	`+$this->assertTrue($email->isRendered());`
`112`	`116`
`113`	`117`	`$renderer->render($email);`
`114`	`118`	`$this->assertEquals('reset',$email->getTextBody());`

Comments

(0)