Needs work
Project:
Drupal core
Version:
main
Component:
extension system
Priority:
Major
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
5 May 2026 at 14:54 UTC
Updated:
21 May 2026 at 02:45 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
catchComment #4
catchComment #5
needs-review-queue-bot commentedThe Needs Review Queue Bot tested this issue. It fails the Drupal core commit checks. 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.
Comment #6
andyposthow come this happened?! sounds major at least
Comment #7
andypostComment #8
berdirNice find, I commented on why this IMHO fails so hard right now. There's also #3336621: Running hook_requirements() on all admin pages is very expensive to help with the "runs on every/many admin pages" and improve that.
Comment #9
catch@andypost I think this goes back to Drupal 7 or earlier and has been ported around.
Comment #10
catchDrupal 7 system_requirements() calls system_rebuild_module_data() - which both built the system data but also wrote it, so that confirms 'ported from at least Drupal 7' for why it's like this.
https://api.drupal.org/api/drupal/modules%21system%21system.install/func...
https://api.drupal.org/api/drupal/modules%21system%21system.module/funct...
Comment #11
catchComment #12
catchAdded a change record.
Comment #13
andypostThank you
Comment #14
catchComment #15
alexpottNice find. Can add typehinting on the new params.
Comment #16
nicxvan commented