Modified guessDefaultEscapingStrategy to not escape txt templates #9639

Closed
wants to merge 1 commit into
from

Projects

None yet

4 participants

@fabpot
Symfony member

This PR was submitted on the symfony/TwigBundle read-only repository and moved automatically to the main Symfony repository (closes symfony/twig-bundle#2).

Useful for plaintext emails.

@cordoval

@fabpot moved automatically? 👶

@fabpot fabpot added a commit that referenced this pull request Nov 28, 2013
@fabpot fabpot bug #9639 Modified guessDefaultEscapingStrategy to not escape txt tem…
…plates (fabpot)

This PR was submitted for the master branch but it was merged into the 2.2 branch instead (closes #9639).

Discussion
----------

Modified guessDefaultEscapingStrategy to not escape txt templates

This PR was submitted on the symfony/TwigBundle read-only repository and moved automatically to the main Symfony repository (closes symfony/twig-bundle#2).

Useful for plaintext emails.

Commits
-------

6affe94 Modified guessDefaultEscapingStrategy to not escape txt templates
c3b4b68
@fabpot fabpot closed this Nov 28, 2013
@maryo

Not merged? Do you think it was a bad idea at all?

@stloyd

@maryo It's merged in branch 2.2, and later will be merged to newer branches as well.

@fabpot
Symfony member

It has been merged, but in the 2.2 branch as this is definitely a bug. 2.3, 2.4, and master will get the bug fix as soon 2.2 is merged into those branches.

@maryo

I see. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment