I like breaking it onto 2 lines, but I don't usually add extra spaces on the second line of a chained call. Any reason why you're doing this - is it in the core code somewhere?

If the method returns itself each time and you have method chaining, I won't indent the second line, but if the method returns a new instance it is nice to indent the next line. We have done that in thecomponent/config article too.

the only place where we use extra indentation levels when chaining in Symfony is when using theSymfony\Component\Config\Definition\TreeBuilder, to make the indentation reflect the tree structure (making it far more readable than 100 chained calls indented the same)

@wouterj, I see your point, and I don't think we have a super-clear standard here one way or another, but I'm personally -1 on the format. It makes more sense in theTreeBuilder because you actually have->end() statements at each node - so you build up the tree (and indent), and then have to work your way back down the tree (with->end()). In this case, we're not returning the same instance, but it's not that same tree structure.

@weaverryan ok, I will change it today.