- Issue created by @jonathanshaw
- Merge request !11934Issue #3520997: DefaultLazyPluginCollection unnecessarily instantiates plugins when sorting collection → (Open) created by jonathanshaw
- 🇬🇧United Kingdom jonathanshaw Stroud, UK
jonathanshaw → changed the visibility of the branch 3520997-2 to hidden.
- 🇬🇧United Kingdom jonathanshaw Stroud, UK
jonathanshaw → changed the visibility of the branch 3520997-defaultlazyplugincollection-unnecessarily-instantiates to hidden.
- 🇬🇧United Kingdom jonathanshaw Stroud, UK
jonathanshaw → changed the visibility of the branch 11.x to hidden.
- 🇬🇧United Kingdom jonathanshaw Stroud, UK
jonathanshaw → changed the visibility of the branch 3520997- to hidden.
- 🇩🇪Germany mxh Offenburg
Took a brief look into this and left two minor side notes.
- 🇩🇪Germany mxh Offenburg
Added another small note regards the array lookup. Other than that the code changes look good. I have no time to actually test it though.
- Status changed to Needs work
2 months ago 2:54pm 28 July 2025 - 🇺🇸United States smustgrave
Appears to be 1 open threads around the return. If that changes though do wonder if it will need test coverage that the exception is correctly thrown.
- 🇬🇧United Kingdom jonathanshaw Stroud, UK
I'm not sure what's going on with the tests, I think the fails are unrelated.
The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.