-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[Form] unintended validation BC break by #14050 #14165
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
Comments
Could you please post some code to reproduce this issue? Preferably a clone of symfony-standard? |
@webmozart: Done in https://github.com/craue/symfony-standard/tree/2.7-symfony-issue-14165. Pre-PR (using Post-PR (using |
Thanks, I can confirm this. The reason is that We need to find some other solution to allow empty strings if (i) the placeholder is enabled and (ii) the choice list does not already contain a choice with an empty string as value. |
I'm also getting this. For an entity form type that is not required by default it will cause a SQL error due to an empty string If the This stops any form from having optional entity form types. |
Any idea for a workaround ? |
Thanks |
It's fixed by #14465. Thank you, @webmozart. |
fix bugs TwigTrait - add Test TwigTrait - add Silex\Provider\HttpFragmentServiceProvider() fix bugs autoload.php precision -> scale scale の引数はint only http://symfony.com/blog/new-in-symfony-2-7-form-and-validator-updates entity maps => false http://symfony.com/doc/current/reference/forms/types/entity.html#mapped 多分これが原因 skip bugs symfony/symfony#14165
#14050 seems to inadvertently affect validation. When submitting an empty required choice field, there's now a message "This value is not valid." which wasn't there before. See #14050 (comment).
The text was updated successfully, but these errors were encountered: