Skip to content

Note was added to preferred_choices option of entity field type #3316

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 19, 2013
Merged

Note was added to preferred_choices option of entity field type #3316

merged 2 commits into from
Dec 19, 2013

Conversation

mgrinko
Copy link
Contributor

@mgrinko mgrinko commented Dec 12, 2013

Note was added to clarify that 'preferred_choices' option expects for array of objects but not for array of keys as default choice field type

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets

Note was added to clarify expectations of this option
@xabbuh
Copy link
Member

xabbuh commented Dec 12, 2013

Can you include the pull request template?

@@ -182,6 +182,10 @@ These options inherit from the :doc:`choice </reference/forms/types/choice>` typ

.. include:: /reference/forms/types/options/preferred_choices.rst.inc

.. note::

This option expects for array of objects unlike choice field that requires an array of keys
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This option expects an array of objects unlike the choice field that requires an array of keys.

@mgrinko
Copy link
Contributor Author

mgrinko commented Dec 12, 2013

@xabbuh I am not sure I understand what is 'pull request template'?

@xabbuh
Copy link
Member

xabbuh commented Dec 12, 2013

Have a look at the Pull Request Format. Hope this helps you. :)

@mgrinko
Copy link
Contributor Author

mgrinko commented Dec 12, 2013

@xabbuh added, thank you.

weaverryan added a commit that referenced this pull request Dec 19, 2013
Note was added to preferred_choices option of entity field type
@weaverryan weaverryan merged commit a5e3aae into symfony:2.3 Dec 19, 2013
weaverryan added a commit that referenced this pull request Dec 19, 2013
@weaverryan
Copy link
Member

Hi Michael!

Ah, this is an excellent note - thanks so much for adding it!

Cheers!

@mgrinko mgrinko deleted the patch-1 branch December 19, 2013 18:40
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