Skip to content

Update associations.rst #18828

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

Merged
merged 1 commit into from
Dec 28, 2023
Merged

Update associations.rst #18828

merged 1 commit into from
Dec 28, 2023

Conversation

fishbone1
Copy link
Contributor

Added caution block that informs about the risks of generated convenience "add" and "remove" methods on the inverse side.

We wasted hours for debugging an "Out of memory" error. In our case we have emails that are connected to users who can access those emails (many-to-many). There is a very large amount of emails. The memory was exceeded when we called $email->addUser($user) because the generated code checked if the user already has that email.

@carsonbot carsonbot added this to the 6.4 milestone Sep 5, 2023
Added caution block that informs about the risks of generated convenience "add" and "remove" methods on the inverse side.
@OskarStark
Copy link
Contributor

OskarStark commented Dec 28, 2023

Thank you @fishbone1 and congratulations on your first contribution to the Symfony documentation 🎉 This is much appreciated 👏

@OskarStark OskarStark merged commit 042709a into symfony:5.4 Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants