Skip to content
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

fix: safe should call toString #849

Merged
merged 1 commit into from
Sep 14, 2016
Merged

fix: safe should call toString #849

merged 1 commit into from
Sep 14, 2016

Conversation

atian25
Copy link
Contributor

@atian25 atian25 commented Sep 14, 2016

close #847

relate: #843 (diff)

@vecmezoni
Copy link
Collaborator

Well, in jinja2:

import jinja2
jinja2.Template(u'{{ arr | safe }}').render({'arr': ['<html>']}) # == u"['<html>']"

But i think we can merge this and then decide what to do with the braces

@atian25
Copy link
Contributor Author

atian25 commented Sep 14, 2016

@vecmezoni thanks, plz merge it and help to publish a patch.

@vecmezoni vecmezoni merged commit acbce02 into mozilla:2.x Sep 14, 2016
@atian25 atian25 deleted the safe branch September 14, 2016 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants