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.

Sign up for GitHub

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

Jump to bottom

Provide more details to the ElementCollectionEvent and lower its priority #25

Closed
vladmihalcea opened this issue Apr 25, 2019 · 3 comments
Closed

Provide more details to the ElementCollectionEvent and lower its priority #25

vladmihalcea opened this issue Apr 25, 2019 · 3 comments
Labels
enhancement New feature or request
Milestone
1.1.1

Comments

@vladmihalcea
Copy link
Owner

The ElementCollectionEvent should have a MAJOR priority, not a CRITICAL one.

Also, as illustrated by the discussion on this GitHub issue, it's worth adding more details about why using an element collection is not as efficient as a bidirectional one-to-many association.
@vladmihalcea vladmihalcea added the enhancement New feature or request label Apr 25, 2019
@vladmihalcea vladmihalcea added this to the 1.1.1 milestone Apr 25, 2019
@vladmihalcea vladmihalcea mentioned this issue Apr 25, 2019
Closed
@vladmihalcea
Copy link
Owner Author

Fixed.

@eximius313
Copy link

If using Set or Map with ElementCollection is not efficient - I'd love to read an article why.
Changing enums used with ElementCollection to Entities just to have bidirectional relation is major change in business model

@vladmihalcea
Copy link
Owner Author

I have updated the User Guide with more details. I'll surely write an article as well in the near future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
1.1.1
Development

No branches or pull requests
2 participants
@vladmihalcea @eximius313