-
-
Notifications
You must be signed in to change notification settings - Fork 502
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
fix(core): rework Collection
initialization to use em.populate()
#4571
Conversation
This unifies the population mechanism for to-many relations by using the `em.populate()` internally. Previously, the `Collection.init` (and methods using it, e.g. `loadItems`) were using custom implementation, which in some cases resulting in different results as opposed to `em.populate()`. With this PR, the `init` method is also using `em.populate`, yielding the same results as if you would populate the relation. Closes #4464
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## v6 #4571 +/- ##
==========================================
- Coverage 99.38% 99.38% -0.01%
==========================================
Files 219 219
Lines 14859 14845 -14
Branches 3440 3443 +3
==========================================
- Hits 14768 14754 -14
Misses 88 88
Partials 3 3
☔ View full report in Codecov by Sentry. |
@B4nan by chance, does this refactor fix a potential issue where calling |
Yes, this PR is part of 6.0.0-dev.89. Keep in mind that |
…4571) This unifies the population mechanism for to-many relations by using the `em.populate()` internally. Previously, the `Collection.init` (and methods using it, e.g. `loadItems`) were using custom implementation, which in some cases resulting in different results as opposed to `em.populate()`. With this PR, the `init` method is also using `em.populate`, yielding the same results as if you would populate the relation. Closes #4464
…4571) This unifies the population mechanism for to-many relations by using the `em.populate()` internally. Previously, the `Collection.init` (and methods using it, e.g. `loadItems`) were using custom implementation, which in some cases resulting in different results as opposed to `em.populate()`. With this PR, the `init` method is also using `em.populate`, yielding the same results as if you would populate the relation. Closes #4464
…4571) This unifies the population mechanism for to-many relations by using the `em.populate()` internally. Previously, the `Collection.init` (and methods using it, e.g. `loadItems`) were using custom implementation, which in some cases resulting in different results as opposed to `em.populate()`. With this PR, the `init` method is also using `em.populate`, yielding the same results as if you would populate the relation. Closes #4464
…4571) This unifies the population mechanism for to-many relations by using the `em.populate()` internally. Previously, the `Collection.init` (and methods using it, e.g. `loadItems`) were using custom implementation, which in some cases resulting in different results as opposed to `em.populate()`. With this PR, the `init` method is also using `em.populate`, yielding the same results as if you would populate the relation. Closes #4464
…4571) This unifies the population mechanism for to-many relations by using the `em.populate()` internally. Previously, the `Collection.init` (and methods using it, e.g. `loadItems`) were using custom implementation, which in some cases resulting in different results as opposed to `em.populate()`. With this PR, the `init` method is also using `em.populate`, yielding the same results as if you would populate the relation. Closes #4464
…4571) This unifies the population mechanism for to-many relations by using the `em.populate()` internally. Previously, the `Collection.init` (and methods using it, e.g. `loadItems`) were using custom implementation, which in some cases resulting in different results as opposed to `em.populate()`. With this PR, the `init` method is also using `em.populate`, yielding the same results as if you would populate the relation. Closes #4464
…4571) This unifies the population mechanism for to-many relations by using the `em.populate()` internally. Previously, the `Collection.init` (and methods using it, e.g. `loadItems`) were using custom implementation, which in some cases resulting in different results as opposed to `em.populate()`. With this PR, the `init` method is also using `em.populate`, yielding the same results as if you would populate the relation. Closes #4464
…4571) This unifies the population mechanism for to-many relations by using the `em.populate()` internally. Previously, the `Collection.init` (and methods using it, e.g. `loadItems`) were using custom implementation, which in some cases resulting in different results as opposed to `em.populate()`. With this PR, the `init` method is also using `em.populate`, yielding the same results as if you would populate the relation. Closes #4464
…4571) This unifies the population mechanism for to-many relations by using the `em.populate()` internally. Previously, the `Collection.init` (and methods using it, e.g. `loadItems`) were using custom implementation, which in some cases resulting in different results as opposed to `em.populate()`. With this PR, the `init` method is also using `em.populate`, yielding the same results as if you would populate the relation. Closes #4464
…4571) This unifies the population mechanism for to-many relations by using the `em.populate()` internally. Previously, the `Collection.init` (and methods using it, e.g. `loadItems`) were using custom implementation, which in some cases resulting in different results as opposed to `em.populate()`. With this PR, the `init` method is also using `em.populate`, yielding the same results as if you would populate the relation. Closes #4464
This unifies the population mechanism for to-many relations by using the
em.populate()
internally.Previously, the
Collection.init
(and methods using it, e.g.loadItems
) were using custom implementation, which in some cases resulting in different results as opposed toem.populate()
. With this PR, theinit
method is also usingem.populate
, yielding the same results as if you would populate the relation.Closes #4464