Skip to content

Update dependency_injection.rst #12421

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

Merged
merged 1 commit into from
Oct 18, 2019
Merged

Update dependency_injection.rst #12421

merged 1 commit into from
Oct 18, 2019

Conversation

Foksler
Copy link
Contributor

@Foksler Foksler commented Oct 4, 2019

In previous chapter we changed our controller logic to return string, also we used ExceptionListener, if we won't register them after we move logic into Framework class we will get fatal

In previous chapter we changed our controller logic to return string, also we used ExceptionListener, if we won't register them after we move logic into Framework class we will get fatal
@javiereguiluz
Copy link
Member

javiereguiluz commented Oct 18, 2019

Aleksej we're sorry we didn't merge this earlier. Thanks a lot for your contribution!

javiereguiluz added a commit that referenced this pull request Oct 18, 2019
This PR was merged into the 4.3 branch.

Discussion
----------

Update dependency_injection.rst

In previous chapter we changed our controller logic to return string, also we used ExceptionListener, if we won't register them after we move logic into Framework class we will get fatal

<!--

If your pull request fixes a BUG, use the oldest maintained branch that contains
the bug (see https://symfony.com/roadmap for the list of maintained branches).

If your pull request documents a NEW FEATURE, use the same Symfony branch where
the feature was introduced (and `master` for features of unreleased versions).

-->

Commits
-------

4f72d1e Update dependency_injection.rst
@javiereguiluz javiereguiluz merged commit 4f72d1e into symfony:4.3 Oct 18, 2019
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.

3 participants