Chaining for EM #620

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
6 participants

pomaxa commented Mar 18, 2013

Chaining for EM methods persist()/flush(), to write more elegant code $em->persist($account)->flush();

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-2357

Member

Majkl578 commented Mar 18, 2013

Why only these two and not also other methods (beginTransaction, rollback, commit, clear, close, remove, detach, copy, lock and initializeObject)?

Member

stof commented Mar 18, 2013

I think this has already been rejected at least 10 times since the release of Doctrine 2.0

Owner

Ocramius commented Mar 18, 2013

This was already refused before. Doctrine generally discourages usage of fluid interfaces where they are not used to build domain specific languages, like the query builder. The idea of saving few keystrokes is not sufficient to introduce a fluid interface.

@beberlei beberlei closed this Mar 18, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment