Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[TwigBundle] update the default Twig escape strategy #7089

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 10, 2016

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Oct 26, 2016

reflect the changes made in symfony/symfony#20224

contents according to the filename extension (e.g. it uses ``html`` for
``*.html.twig`` templates and ``js`` for ``*.js.html`` templates).
and ``name``. The default value is ``name``. This strategy escapes contents
according to the filename extension (e.g. it uses ``html`` for ``*.html.twig``
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

according to the filename extension -> according to the template name extension ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@xabbuh xabbuh force-pushed the default-twig-escape-strategy branch from 1d717ae to 402d556 Compare November 5, 2016 10:43
@wouterj
Copy link
Member

wouterj commented Nov 10, 2016

Thanks!

@wouterj wouterj merged commit 402d556 into symfony:2.7 Nov 10, 2016
wouterj added a commit that referenced this pull request Nov 10, 2016
…abbuh)

This PR was merged into the 2.7 branch.

Discussion
----------

[TwigBundle] update the default Twig escape strategy

reflect the changes made in symfony/symfony#20224

Commits
-------

402d556 update the default Twig escape strategy
@xabbuh xabbuh deleted the default-twig-escape-strategy branch November 11, 2016 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants