Skip to content

[Propel1] add preferred_choices option to ModelType #6454

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 2 commits into from
Dec 27, 2012

Conversation

havvg
Copy link
Contributor

@havvg havvg commented Dec 21, 2012

This enables the ModelChoiceList to use 'preferred_choices' of the parent ChoiceType.

Ping @willdurand

This enables the ModelChoiceList to use 'preferred_choices' of the parent ChoiceType.
@willdurand
Copy link
Contributor

👍

fabpot added a commit that referenced this pull request Dec 27, 2012
This PR was merged into the 2.1 branch.

Commits
-------

05fca6d use preferred_choices in favor of preferred_query
6855cff add preferred_query option to ModelType

Discussion
----------

[Propel1] add preferred_choices option to ModelType

This enables the ModelChoiceList to use 'preferred_choices' of the parent ChoiceType.

Ping @willdurand

---------------------------------------------------------------------------

by willdurand at 2012-12-22T09:39:39Z

:+1:
@fabpot fabpot merged commit 05fca6d into symfony:2.1 Dec 27, 2012
fabpot added a commit that referenced this pull request Jan 5, 2013
This PR was merged into the master branch.

Commits
-------

310f691 [Propel1] Fixed preferred choices in the ModelChoiceList.

Discussion
----------

[Propel1] Fixed preferred choices in the ModelChoiceList

Related change was made in #6454 but for some reason part of that change was not merged from 2.1 branch.

Bug fix: yes
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: some Locale tests fail, fixed in #6568
Fixes the following tickets:
Todo: -
License of the code: MIT
Documentation PR: -
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.

3 participants