Skip to content

Update the_controller.rst #5039

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

Closed
wants to merge 1 commit into from
Closed

Update the_controller.rst #5039

wants to merge 1 commit into from

Conversation

oldskool
Copy link
Contributor

Fixed some things:

  • Add missing closing parenthesis
  • Use the DefaultController helloAction as forward example instead

The latter is to match the rest of the documentation page. The blog controller is not even made yet at this point in the tutorial, so there is no sense in trying to forward there.

Fixed some things:

* Add missing closing parenthesis
* Use the DefaultController helloAction as forward example instead

The latter is to match the rest of the documentation page. The blog controller is not even made yet at this point in the tutorial, so there is no sense in trying to forward there.
@wouterj
Copy link
Member

wouterj commented Feb 22, 2015

Thank you for fixing this code, @oldskool. I'm going to close this PR however, as this section will be removed soon from the quick tour (see #5037). Forwarding seem to be something that's not often used and we therefor decided it shouldn't be documented in the Quick Tour (if even experienced symfony devs don't use it, it makes no sense to put it in the first introduction of symfony).

Thanks again!

@wouterj wouterj closed this Feb 22, 2015
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