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 CollectionField.rst file for better documentation (adding extra considerations for its implementation) #6311

Open
wants to merge 2 commits into
base: 4.x
Choose a base branch
from

Conversation

FerJoTello
Copy link

@FerJoTello FerJoTello commented May 28, 2024

Adding some extra considerations in the docs to use CollectionField class, since this class uses "add" & "remove" methods related to the Collection property from the entity that owns the collection in order to add and delete binded values; there is no documentation that explains that Collections must implement these methods and it is not intuitive so it would be a great help for anyone who is implementing this class for the first time (as it was in my case, after finding the solution 4 hours later).

References: https://stackoverflow.com/questions/48304399/q-symfony4-easyadmin-onetomany-not-saving-in-db

Adding some extra considerations to use CollectionField class, since this class uses "add" & "remove" methods related to the Collection property from the entity that owns the collection in order to add and delete binded values; there is no documentation that explains that Collections must implement these methods and it is not intuitive so it would be a great help for anyone who is implementing this class for the first time (as it was in my case, after finding the solution 4 hours later).

References: https://stackoverflow.com/questions/48304399/q-symfony4-easyadmin-onetomany-not-saving-in-db
@FerJoTello FerJoTello changed the title Update CollectionField.rst adding extra considerations for its implementation Update CollectionField.rst file for better documentation (adding extra considerations for its implementation) Jun 11, 2024
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

1 participant