Skip to content

Form collection has changed in 2.1 #1326

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

Closed
tristanbes opened this issue Apr 29, 2012 · 10 comments
Closed

Form collection has changed in 2.1 #1326

tristanbes opened this issue Apr 29, 2012 · 10 comments
Milestone

Comments

@tristanbes
Copy link
Contributor

Behaviour of form collection has changed in 2.1 and is not reflected in the doc.

@weaverryan
Copy link
Member

Hi Tristan!

Yes, we were just having this conversation at #1322 :). If we can verify that there are no other changes beyond the addXxx and removeXxx (the __name__ change is already there), then we can put some more specifics on this issue and make it actionable. So, if you know of any more specific changes, that would be great.!
Thanks!

@tristanbes
Copy link
Contributor Author

Hi, Yes i'm aware that there are other changes for form collection with BC break :

symfony/symfony#4046

@tristanbes
Copy link
Contributor Author

@weaverryan Do you have any updates on this ? It's quite important since the form collections are often used.

@weaverryan
Copy link
Member

This issue is still valid. Specifically, we need to update the docs to mention the addXxx and removeXxx collection changes. This would need to be done in the form_collections doc, and then probably also mention it in the reference section for the collection, choice and entity types.

Thanks!

@wouterj
Copy link
Member

wouterj commented Oct 31, 2012

@weaverryan do we need to update the documentation as said in upgrade-2.1.md? If so, I will take care of it.

EDIT: I read it to fast. Where can I find the differences?

@weaverryan
Copy link
Member

The addXxx and removeXxx stuff is mentioned in the CHANGELOG here - https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Form/CHANGELOG.md.

It's a small change, but will mean some somewhat large changes to the form_collections doc. If you'd like to make those changes, that would be awesome :). The form_collections doc is one spot where I usually actually put together some test code to make sure everything is actually working, since there are so many pieces that work together.

Thanks!

@wouterj
Copy link
Member

wouterj commented Nov 1, 2012

@weaverryan I don't understand what's missing (I'm not familiar with the Form Component), sorry. I will take a look on the Twig reference page and someone else can do this.

@weaverryan
Copy link
Member

no problem :)

@weaverryan
Copy link
Member

Basically we're still just missing specifics on the addXxx and removeXxx methods, but the collection document works on both 2.0 and 2.1.

So, once we have the addXxx and removeXxx stuff documented, then we can close this. I think this is needed in the choice, collection and entity form field references docs, possibly just one little example highlighting how this works that's included in all 3 of those docs.

@weaverryan
Copy link
Member

Bernhard has already done the work for this in #1057, which explains the addXxx and removeXxx functionality. I'll close this issue in favor of that one, which is specific.

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

No branches or pull requests

3 participants