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

Bug fix escape quotes #138

Merged
merged 2 commits into from Mar 2, 2014
Merged

Bug fix escape quotes #138

merged 2 commits into from Mar 2, 2014

Conversation

kaos
Copy link
Member

@kaos kaos commented Mar 1, 2014

No description provided.

Running the command:

erlydtl:compile(<<"{{ text|add:\"\\\"\" }}">>, test).

will result in a crash. string:strip will remove all \" from the
beginning and end of the string \"\\\"\", leaving only \\. It should
only remove one \" from the beginning of the string and one from the end.
kaos added a commit that referenced this pull request Mar 2, 2014
```erlang
erlydtl_tests: all_defs_test_ (filters: |truncatewords:0)...*failed*
in function erlydtl_eunit_testrunner:'-run_render/1-fun-0-'/2 (test/erlydtl_eunit_testrunner.erl, line 73)
**error:{assertEqual_failed,[{module,erlydtl_eunit_testrunner},
                     {line,73},
                     {expression,"B"},
                     {expected,<<" ...">>},
                     {value,<<>>}]}
```
kaos added a commit that referenced this pull request Mar 2, 2014
@kaos kaos merged commit a3fa62f into erlydtl:master Mar 2, 2014
@kaos kaos added the filters label Mar 2, 2014
@kaos kaos added this to the 0.9.1 milestone Mar 2, 2014
@kaos kaos self-assigned this Mar 2, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants