Use consistent method chaining in BlogBundle sample application#3585

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

Jump to bottom

Merged

weaverryan merged 1 commit intosymfony:2.3fromoradwell:blog_bundle_consistent_chaining

Mar 4, 2014

Merged

Use consistent method chaining in BlogBundle sample application#3585

weaverryan merged 1 commit intosymfony:2.3fromoradwell:blog_bundle_consistent_chaining

Mar 4, 2014

Conversation

Copy link

Contributor

oradwell commentedFeb 17, 2014

Moved semicolon to end of last chained method
Moved getManager() to next line to avoid having multiple chained
methods in the same line

Use consistent method chaining in BlogBundle sample application

3670cbb

- Moved semicolon to end of last chained method- Moved getManager() to next line to avoid having multiple chained  methods in the same line

Copy link

Member

weaverryan commentedMar 4, 2014

Agreed, much better indeed :). Thanks again!

weaverryan added a commit that referenced this pull request

Mar 4, 2014

minor#3585Use consistent method chaining in BlogBundle sample appli…

1714a31

…cation (ockcyp)This PR was merged into the 2.3 branch.Discussion----------Use consistent method chaining in BlogBundle sample application- Moved semicolon to end of last chained method- Moved getManager() to next line to avoid having multiple chained  methods in the same lineCommits-------3670cbb Use consistent method chaining in BlogBundle sample application - Moved semicolon to end of last chained method - Moved getManager() to next line to avoid having multiple chained   methods in the same line

weaverryan merged commit3670cbb intosymfony:2.3

Mar 4, 2014