Skip to content

Conversation

Tobion
Copy link
Contributor

@Tobion Tobion commented Oct 9, 2014

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

twig does not know about template references and only expects a string.
this commit also fixes that name parsing and locating was called twice for nonexistent templates.

twig does not know about template references and only expects a string.
this commit also fixes that name parsing and locating was called twice for nonexistent templates
*/
public function exists($template)
public function exists($name)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this change makes it consistent with newer branches, so it applies cleanly

@stof
Copy link
Member

stof commented Oct 10, 2014

👍

@fabpot
Copy link
Member

fabpot commented Oct 10, 2014

Thank you @Tobion.

@fabpot fabpot merged commit 7fe33e3 into symfony:2.3 Oct 10, 2014
fabpot added a commit that referenced this pull request Oct 10, 2014
…ion)

This PR was merged into the 2.3 branch.

Discussion
----------

[TwigBundle] do not pass a template reference to twig

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

twig does not know about template references and only expects a string.
this commit also fixes that name parsing and locating was called twice for nonexistent templates.

Commits
-------

7fe33e3 [TwigBundle] do not pass a template reference to twig
@Tobion Tobion deleted the exception-tpl-reference branch October 10, 2014 17:06
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.

3 participants