8000 [Bridge][Doctrine][Form] Add entity manager instance support for em option by egeloen · Pull Request #12251 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[Bridge][Doctrine][Form] Add entity manager instance support for em option #12251

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

Merged
merged 1 commit into from
Oct 20, 2014

Conversation

egeloen
Copy link
@egeloen egeloen commented Oct 18, 2014
Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #10157
License MIT
Doc PR symfony/symfony-docs#4336

This PR allows to pass an explicit entity manager instance as em option.

@fabpot
Copy link
Member
fabpot commented Oct 19, 2014

👍

@Tobion
Copy link
Contributor
Tobion commented Oct 19, 2014

Please make use of $resolver->setAllowedTypes now that the option can be multi-typed.

@egeloen egeloen force-pushed the form-entity-em-option branch from 9e4f16e to d84e3a8 Compare October 19, 2014 17:05
@egeloen
Copy link
Author
egeloen commented Oct 19, 2014

@Tobion Done

@Tobion
Copy link
Contributor
Tobion commented Oct 19, 2014

👍

@fabpot
Copy link
Member
fabpot commented Oct 20, 2014

Thank you @egeloen.

@fabpot fabpot merged commit d84e3a8 into symfony:master Oct 20, 2014
fabpot added a commit that referenced this pull request Oct 20, 2014
…upport for em option (egeloen)

This PR was merged into the 2.6-dev branch.

Discussion
----------

[Bridge][Doctrine][Form] Add entity manager instance support for em option

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #10157
| License       | MIT
| Doc PR        | symfony/symfony-docs#4336

This PR allows to pass an explicit entity manager instance as em option.

Commits
-------

d84e3a8 [Bridge][Doctrine][Form] Add entity manager instance support for em option
@egeloen egeloen deleted the form-entity-em-option branch October 20, 2014 09:52
weaverryan added a commit to symfony/symfony-docs that referenced this pull request Nov 13, 2014
…n (egeloen)

This PR was merged into the master branch.

Discussion
----------

[Form] Add entity manager instance support for em option

This PR documents symfony/symfony#12251

Commits
-------

09a6fd7 [Form] Add entity manager instance support for em option
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.

4 participants
0