Skip to content

Conversation

@fbourigault
Copy link
Contributor

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets n/a
Documentation n/a
License MIT

This fix the profiler panel display when used in a symfony/flex application.

I added a test to ensure the panel is loading (the case is with no requests sent).

The bug was caused by the profiler template path which was using the old twig path syntax (no @ and ::).

I also found that the profiler is dependent on the symfony/asset component.

"symfony/web-profiler-bundle": "^2.8 || ^3.0",
"symfony/finder": "^2.7 || ^3.0",
"symfony/cache": "^3.1",
"symfony/browser-kit": "^2.8 || ^3.0",
Copy link
Member

Choose a reason for hiding this comment

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

Why do we need this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We need it for testShowProfiler.

@Nyholm Nyholm merged commit 81de492 into php-http:master Aug 4, 2017
@Nyholm
Copy link
Member

Nyholm commented Aug 4, 2017

Thank you

@fbourigault fbourigault deleted the symfony-flex branch August 4, 2017 08:15
@fbourigault fbourigault mentioned this pull request Aug 4, 2017
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