Returns $this where it's possible in Doctrine\ORM\Configuration #539

Closed
wants to merge 1 commit into
from

Projects

None yet

4 participants

@alex-ception

This commit adds as it can, comfort to some developers.

@doctrinebot

Hello,

thank you for positing this Pull Request. I have automatically opened an issue on our Jira Bug Tracker for you with the details of this Pull-Request. See the Link:

http://doctrine-project.org/jira/browse/DDC-2207

@Ocramius
Member

I personally don't see that much of use on the fluent interface... Maybe acceptable on a configuration object though.
Also, this one would require to change the other config objects across doctrine project for consistency.

@beberlei
Member

A fluent interface has no benefit here.

Edit: We don't add fluent interfaces everywhere were multiple calls to one object happen, only when it makes sense in termes of creating a DSL of method calls.

@beberlei beberlei closed this Dec 22, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment