Skip to content

[Form] Deprecate calling FormErrorIterator::children() if the current element is not iterable #42387

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

Conversation

W0rma
Copy link
Contributor

@W0rma W0rma commented Aug 5, 2021

Q A
Branch? 5.4 for features
Bug fix? yes
New feature? no
Deprecations? yes
Tickets #42301 (comment)
License MIT
Doc PR

@W0rma W0rma requested review from xabbuh and yceruto as code owners August 5, 2021 07:47
@W0rma W0rma changed the title Deprecate calling FormErrorIterator::children() if the current element is not iterable [Form] Deprecate calling FormErrorIterator::children() if the current element is not iterable Aug 5, 2021
@carsonbot carsonbot changed the title [Form] Deprecate calling FormErrorIterator::children() if the current element is not iterable Deprecate calling FormErrorIterator::children() if the current element is not iterable Aug 5, 2021
@derrabus derrabus added the Form label Aug 5, 2021
@carsonbot carsonbot changed the title Deprecate calling FormErrorIterator::children() if the current element is not iterable [Form] Deprecate calling FormErrorIterator::children() if the current element is not iterable Aug 5, 2021
@xabbuh xabbuh added this to the 5.4 milestone Aug 5, 2021
Copy link
Member

@xabbuh xabbuh left a comment

Choose a reason for hiding this comment

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

we need to update the UPGRADE-5.4.md and UPGRADE-6.0.md files

@W0rma W0rma force-pushed the deprecate-form-error-iterator-get-children-not-iterable branch 3 times, most recently from f8910cd to 03a2a13 Compare August 5, 2021 11:16
@derrabus derrabus requested a review from xabbuh August 5, 2021 11:19
@W0rma W0rma force-pushed the deprecate-form-error-iterator-get-children-not-iterable branch 2 times, most recently from 6de432e to 9cf7d8a Compare August 5, 2021 12:21
@W0rma W0rma force-pushed the deprecate-form-error-iterator-get-children-not-iterable branch from 9cf7d8a to d8e74fc Compare August 5, 2021 12:51
@W0rma W0rma requested a review from xabbuh August 5, 2021 12:53
@fabpot
Copy link
Member

fabpot commented Aug 15, 2021

Thank you @W0rma.

@fabpot fabpot merged commit f0b1262 into symfony:5.4 Aug 15, 2021
nicolas-grekas added a commit that referenced this pull request Aug 19, 2021
…rma)

This PR was merged into the 6.0 branch.

Discussion
----------

[Form] Remove deprecated code from FormErrorIterator

| Q             | A
| ------------- | ---
| Branch?       | 6.0
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | Follow-up of #42387
| License       | MIT
| Doc PR        |

Remove deprecated code from `FormErrorIterator::children()`

Commits
-------

0798623 Throw exception if child element is not iterable
symfony-splitter pushed a commit to symfony/form that referenced this pull request Aug 19, 2021
…rma)

This PR was merged into the 6.0 branch.

Discussion
----------

[Form] Remove deprecated code from FormErrorIterator

| Q             | A
| ------------- | ---
| Branch?       | 6.0
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | Follow-up of symfony/symfony#42387
| License       | MIT
| Doc PR        |

Remove deprecated code from `FormErrorIterator::children()`

Commits
-------

07986234cc Throw exception if child element is not iterable
This was referenced Nov 5, 2021
@W0rma W0rma deleted the deprecate-form-error-iterator-get-children-not-iterable branch November 7, 2021 10: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.

5 participants