Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[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
Morty Proxy This is a proxified and sanitized view of the page, visit original site.