Skip to content

Conversation

@HeahDude
Copy link
Contributor

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? not yet
Fixed tickets n/a
License MIT
Doc PR -

@HeahDude
Copy link
Contributor Author

Note for the merger :
if I don't answer to the ping for the rebase, just discard first commit when squashing before merging #17891 in 3.0.

@HeahDude HeahDude changed the title [WIP] [2.8] [Form] fix tests added in #17798 with FQCN [2.8] [Form] fix tests added in #17798 with FQCN Feb 27, 2016
fabpot added a commit that referenced this pull request Feb 28, 2016
…n to be 'false' in ChoiceType (HeahDude)

This PR was merged into the 2.7 branch.

Discussion
----------

[2.7] [Form] Fix BC break by allowing 'choice_label' option to be 'false' in ChoiceType

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

- [x] Make tests pass
- [x] Adress a doc PR
- [x] Replace alias by FQCN in tests for 2.8, see #17890
- [x] Remove `choices_as_values` in tests for 3.0, see #17891

Commits
-------

017e1d9 bug #17798 [Form] allow `choice_label` option to be `false`
@fabpot
Copy link
Member

fabpot commented Feb 28, 2016

cherry-picked, thanks.

@fabpot fabpot closed this Feb 28, 2016
@HeahDude HeahDude deleted the fix_tests-17798-2_8 branch March 24, 2016 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants