Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
Add a section on front controllers and the AppKernel#2465
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
wouterj commentedApr 4, 2013
Could you please add thepull request format to your PR message? And I think this issue is related:#997 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
you are missing an.. index:: block at the first line of your page
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
@wouterj Is that correct with a triple ` plus the underscore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
@mpdude no, you can't mix inline formats in Sphinx. So you just need to use 1 backtick to create a link.
mpdude commentedApr 4, 2013
@wouterj Thanks for the quick feedback and sorry for getting so many basics wrong. I hope I did not forget one of your comments. |
mpdude commentedApr 5, 2013
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
typoreques missing at
lsmith77 commentedApr 5, 2013
there are several places that talk about Apache rewrite rules, i think it should be stated somewhere that equivalent behavior can be achieved with pretty much every other web server. |
mpdude commentedApr 5, 2013
All feedback picked up - thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
The way it is wrapped is weird here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Doesthis method refer togetEnvironment orregisterbundles here ?
mpdude commentedApr 6, 2013
@wouterj thanks for keeping an eye on it. I removed the reference to the Debug Component. |
mpdude commentedApr 7, 2013
symfony/symfony#7441 has been merged. |
wouterj commentedApr 7, 2013
mpdude commentedApr 8, 2013
weaverryan commentedApr 15, 2013
Hi Matthias! Very nice work! I've patched this into the 2.0 branch so that all branches can enjoy it, and removed the Debug reference (but re-added it to the master branch only). Thanks! |
mpdude commentedApr 16, 2013
@weaverryan just saw that the 'decorate' link in the Front Controller paragraph is broken. Could you please fix that? Thanks! |
mpdude commentedApr 16, 2013
The link to the RewriteRule is broken as well :-( |
Here are some additions on how front controllers and custom AppKernels can be used to further tweak the runtime environment.
This is my first piece of .rst, so probably a lot of things and markup needs to be fixed. Additionally, we should discuss which is the best place for this document.