-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[Form] Update choice_loader.rst.inc #16500
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
Thanks for your contribution. Sadly I don't know if it's correct or not, so I'd need the help from someone who understands Symfony Forms well. Thanks! |
We should change the method instead. So instead of having a public function getParent(): string
{
return ChoiceType::class;
} |
Thank you very much @xabbuh for your reply. Yes, i totally agree with this suggestion too. |
@issamkhadiri1989 do you want to update this PR with the changes proposed by @xabbuh? Thanks! |
Hello @javiereguiluz I appreciate your feedback. Of course, i've just updated the PR. Can you check please ? Thanks ! |
Issam, thanks a lot for this contribution and all the updates that you did during the review! Thanks a lot to reviewers too. |
Hello,
I think it is better to extend the class
ConstantsType
from theForm\AbstractTypeExtension
instead. As we're using the functiongetExtendedTypes
i think it might be better. This method is available in theAbstractTypeExtension
class and not in theAbstractType
class. I might be wrong but here we're about to create a type extension from the ChoiceType.