Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[FrameworkBundle] Ignore backslashes in service ids when using debug:container and debug:autowiring #28487

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

respinoza
Copy link
Contributor

@respinoza respinoza commented Sep 17, 2018

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #28143
License MIT
Doc PR symfony/symfony-docs#...

Copy link
Member

@nicolas-grekas nicolas-grekas left a comment

Choose a reason for hiding this comment

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

Thanks for submitting!

@nicolas-grekas nicolas-grekas changed the title [FrameworkBundle] Add the ability to ignore backslashes in service id… [FrameworkBundle] Add the ability to ignore backslashes in service ids when using debug:container and debug:autowiring Sep 17, 2018
@nicolas-grekas nicolas-grekas added this to the next milestone Sep 17, 2018
@respinoza
Copy link
Contributor Author

@nicolas-grekas do we need a doc PR for this?
Maybe https://symfony.com/doc/current/service_container/debug.html ?

@nicolas-grekas
Copy link
Member

I'm not sure actually.

@@ -13,6 +13,7 @@ CHANGELOG
* Added new "auto" mode for `framework.session.cookie_secure` to turn it on when HTTPS is used
* Removed the `framework.messenger.encoder` and `framework.messenger.decoder` options. Use the `framework.messenger.serializer.id` option to replace the Messenger serializer.
* Deprecated the `ContainerAwareCommand` class in favor of `Symfony\Component\Console\Command\Command`
* Add the ability to ignore backslashes in service ids when using `debug:container` and `debug:autowiring`
Copy link
Member

Choose a reason for hiding this comment

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

Added :)

@nicolas-grekas nicolas-grekas changed the title [FrameworkBundle] Add the ability to ignore backslashes in service ids when using debug:container and debug:autowiring [FrameworkBundle] Ignore backslashes in service ids when using debug:container and debug:autowiring Sep 19, 2018
@nicolas-grekas nicolas-grekas force-pushed the ignore-backslash-when-searching-service branch from 67763b5 to 0aba355 Compare September 19, 2018 14:43
@nicolas-grekas
Copy link
Member

Thank you @respinoza.

@nicolas-grekas nicolas-grekas merged commit 0aba355 into symfony:master Sep 19, 2018
nicolas-grekas added a commit that referenced this pull request Sep 19, 2018
…en using debug:container and debug:autowiring (respinoza)

This PR was merged into the 4.2-dev branch.

Discussion
----------

[FrameworkBundle] Ignore backslashes in service ids when using debug:container and debug:autowiring

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #28143
| License       | MIT
| Doc PR        | symfony/symfony-docs#... <!-- required for new features -->

Commits
-------

0aba355 [FrameworkBundle] Ignore backslashes in service ids when using `debug:container` and `debug:autowiring`
@nicolas-grekas nicolas-grekas modified the milestones: next, 4.2 Nov 1, 2018
This was referenced Nov 3, 2018
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.

4 participants