Skip to content
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

Update KeyValueRowType for Symfony 3. #28

Merged
merged 1 commit into from
Jul 18, 2019
Merged

Update KeyValueRowType for Symfony 3. #28

merged 1 commit into from
Jul 18, 2019

Conversation

ubermichael
Copy link

At some point Symfony dropped SimpleChoiceList, and stopped referring to choice widgets by name. This patch updates those two things.

It may need back-porting to Symfony 2.8, I'm not sure.

@Burgov
Copy link
Owner

Burgov commented Jan 17, 2017

Can anyone verify that this is safe to merge without breaking older versions?

@pmdc
Copy link

pmdc commented Jan 19, 2017

It will be breaking projects with Symfony older than 2.7 as Symfony\Component\Form\ChoiceList\ArrayChoiceList was introduced in 2.7.

It will also break projects running on PHP 5.4 and lower as ::class constant was introduced in PHP 5.5.

Except for Symfony 2.3 (support ending in may 2017), the Symfony and PHP versions that would break are all unsuported.

@Burgov
Copy link
Owner

Burgov commented Feb 10, 2017

I guess we should wait until June before merging this?

@Burgov
Copy link
Owner

Burgov commented Jul 17, 2017

Is this good to go now? @pmdc

@enekochan
Copy link

May be this could be merged to a new v2.0.0 (where we would target Symfony 3.3, 3.4 and 4.0) and leave v1.4.x BC for 2.8 and/or 3.0. Either way there are more things that should be changed to be Symfony 3.4 compatible like:

https://github.com/Burgov/KeyValueFormBundle/blob/master/Form/Type/KeyValueType.php#L54
https://github.com/Burgov/KeyValueFormBundle/blob/master/Form/Type/KeyValueType.php#L83

@mvehar
Copy link

mvehar commented Jul 18, 2019

Will this be merged and published anytime soon?

@Burgov Burgov merged commit 7d2b6e6 into Burgov:master Jul 18, 2019
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.

None yet

5 participants