Active
Project:
Drupal core
Version:
main
Component:
entity system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
3 Sep 2026 at 15:58 UTC
Updated:
3 Sep 2026 at 15:58 UTC
Jump to comment: Most recent
Per #3610122: Field data for multiple cardinality fields are not populated on entity load
loadMultipleCardinalityFields() does too much and takes way too many arguments for my liking, maybe we can refactor this elsewhere.
The original method had some of the highest cyclomatic complexity of any method in core until we split it up, but the split up version is not really much better either.
Adding the needs followup tag since still need an issue for refactoring ::loadMultipleCardinalityFields() but I think the single cardinality method is as bad or worse, and we might want to tackle them together
This issue to tackle both for now, but can be split if it works out better that way.
Comments
Comment #2
godotislate