Skip to content

Conversation

@xabbuh
Copy link
Member

@xabbuh xabbuh commented Nov 15, 2023

Q A
Branch? 6.4
Bug fix? yes
New feature? no
Deprecations? no
Issues Fix #52280 (comment)
License MIT

The debug.php file is never loaded when the Stopwatch component is not installed. However, the virtual request stack is always valuable as soon as the profiling feature is enabled.

Comment on lines 51 to 52
->args([service('request_stack')])
->public()
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
->args([service('request_stack')])
->public()
->args([service('request_stack')])
->public()

@carsonbot carsonbot changed the title register the virtual request stack together with common profiling services [FrameworkBundle] register the virtual request stack together with common profiling services Nov 17, 2023
Copy link
Contributor

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

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

Thanks!

…vices

The debug.php file is never loaded when the Stopwatch component is not installed.
However, the virtual request stack is always valuable as soon as the profiling
feature is enabled.
@fabpot
Copy link
Member

fabpot commented Nov 18, 2023

Thank you @xabbuh.

@fabpot fabpot merged commit 80db691 into symfony:6.4 Nov 18, 2023
@xabbuh xabbuh deleted the issue-52280 branch November 19, 2023 08:40
@scheb
Copy link
Contributor

scheb commented Nov 22, 2023

FYI: I can confirm, that fix works (for me) 👍

I now have working integration tests (which run an actual Symfony application) on Symfony 6.4 and 7.0, using the RC1-packages and symfony/framework-bundle in versions 6.4.x-dev / 7.0.x-dev to include the patch.

@xabbuh
Copy link
Member Author

xabbuh commented Nov 22, 2023

@scheb Thank you very much for testing development versions and providing valuable feedback. This is much appreciated.

This was referenced Nov 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants