Skip to content

Commit

Permalink
minor twigphp#4193 Remove escape (ruudk)
Browse files Browse the repository at this point in the history
This PR was merged into the 3.x branch.

Discussion
----------

Remove escape

Interestingly this didn’t fail on the branch that introduced the change. Are we sure the integration tests work at all?

See [failing job](https://github.com/twigphp/Twig/actions/runs/10341129793) on merge to main.

Commits
-------

8ef1258 Remove escape
  • Loading branch information
fabpot committed Aug 11, 2024
2 parents 110c1ab + 8ef1258 commit 5541e6a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion extra/html-extra/Tests/Fixtures/data_uri.test
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
--TEST--
"data_uri" filter
--TEMPLATE--
{{ 'foobar#'|data_uri(parameters={charset: "utf-8", foo: "\$bar"}) }}
{{ 'foobar#'|data_uri(parameters={charset: "utf-8", foo: "$bar"}) }}
{{ '<b>foobar</b>'|data_uri(mime="text/html", parameters={charset: "ascii"}) }}
<img src="{{ sf_logo|data_uri }}" />
--DATA--
Expand Down

0 comments on commit 5541e6a

Please sign in to comment.