Heya,

I just hit an issue with the RepeatedField, if you try to use |render on
one, it goes into the FormExtension::render() method and then that check
isn't passing:

if ($field instanceof Form || get_class($field) ===
'Symfony\Component\Form\FieldGroup') {

Because RepeatedField extends from FieldGroup, but not Form. The result
is that it tries to render the field with the "field" block, and that
ends up in an endless recursion.

Why is this not just checking for "$field instanceof FieldGroup"?

I know it's been changed a few times already I saw commits fly by but
didn't really look into why it ended up the way it currently is, so
unless someone has a very good reason, I think it should be fixed.

Cheers

-- 
Jordi Boggiano
@seldaek :: http://seld.be/

-- 
If you want to report a vulnerability issue on symfony, please send it to 
security at symfony-project.com

You received this message because you are subscribed to the Google
Groups "symfony developers" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/symfony-devs?hl=en

Reply via email to