Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DDC-2196] Improved "extendability" of EntityManager #535

Closed
wants to merge 1 commit into from

Conversation

mnapoli
Copy link
Contributor

@mnapoli mnapoli commented Dec 14, 2012

DDC-2196 EntityManager can't be extended easily

EntityManager::create now uses "new static" instead of "new EntityManager"

@doctrinebot
Copy link

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

@mnapoli
Copy link
Contributor Author

mnapoli commented Dec 14, 2012

How can I do to avoid the bot to create a new Jira issue for every pull request?

@stof
Copy link
Member

stof commented Dec 14, 2012

you cannot. This is intended as @beberlei prefers to use JIRA to track pending work

@Baachi
Copy link

Baachi commented Dec 14, 2012

@stof Maybe mark the EntityManager as final?

@mnapoli
Copy link
Contributor Author

mnapoli commented Dec 14, 2012

@stof I'm not questioning Jira, it's just that there's already an issue open and I referenced it in the title, but the bot still creates a new (useless) ticket.

@Baachi I think he was refering my question (not the PR)

@Baachi
Copy link

Baachi commented Dec 14, 2012

@mnapoli You should not extend the EntityManager, because it has some side effects.
And BTW have you looked at #524?

@mnapoli
Copy link
Contributor Author

mnapoli commented Dec 14, 2012

@Baachi Thanks I didn't see #514 indeed. So to sum up there is no way to extend the EntityManager, either by inheritance or by decoration?

And I'm curious of what the side effects are?

@beberlei
Copy link
Member

@mnapoli we want to prevent fiddiling with the EntityManager API, because its used internally in some places and requires to work exactly that way. We want to prevent weird bugs in user applications this way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants