Skip to content
This repository has been archived by the owner on Feb 6, 2022. It is now read-only.

Adapt collector template name to webprofiler conventions #46

Merged
merged 1 commit into from
Aug 5, 2013
Merged

Adapt collector template name to webprofiler conventions #46

merged 1 commit into from
Aug 5, 2013

Conversation

hacfi
Copy link
Contributor

@hacfi hacfi commented Aug 5, 2013

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets fabpot/Twig#1156
License MIT

Debugging the issue I noticed that the @WebProfiler templates use a different convention.

This is a quick fix.

To allow the conventions most bundles use the Twig_Loader_Filesystem needs to be updated instead.

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | fabpot/Twig#1156
| License       | MIT

Debugging the issue I noticed that the @WebProfiler templates use a different convention than the @SecurityBundle collector template.

This is a quick fix.

To allow the conventions most bundles use the Twig_Loader_Filesystem needs to be updated instead.
fabpot added a commit that referenced this pull request Aug 5, 2013
This PR was merged into the master branch.

Discussion
----------

Adapt collector template name to webprofiler conventions

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | fabpot/Twig#1156
| License       | MIT

Debugging the issue I noticed that the @WebProfiler templates use a different convention.

This is a quick fix.

To allow the conventions most bundles use the Twig_Loader_Filesystem needs to be updated instead.

Commits
-------

1a05027 Adapt collector template name to webprofiler conventions
@fabpot fabpot merged commit 1a05027 into symfony:master Aug 5, 2013
@hacfi hacfi deleted the patch-1 branch August 5, 2013 08:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants