This was noticed as part of 📌 Create class to replace hook_install_requirements Active
@alexpott made the following comment:
We should file an issue to remove this. It's wrong - because we don't do this in HEAD when we do install requirement checking when we're on the ModuleListForm and also we don't do it below for profiles. My guess is that how approach to autoloading for profiles is making it hard to convert some of the install hooks to OOP hooks...
The reason this doesn't affect anything really is it was only added for system module... see #2388255: (followup) Limit PDO MySQL to executing single statements if PHP supports it → - this was my mistake. And the whole reason it was added - to load Drupal\system\SystemRequirements was removed in #3054060: Deprecate \Drupal\system\SystemRequirements::phpVersionWithPdoDisallowMultipleStatements() as all supported versions of PHP return TRUE → and #3097453: Remove system.module BC layers →
Fun.
N/A
Explore just removing it.
N/A
N/A
N/A
N/A
N/A
Active
11.0 🔥
base system
Just pushed up a naive removal to see if anything fails since it was mentioned the need for it was removed.
Yeah the code that need it has been removed. It was a mistake adding it.
Automatically closed - issue fixed for 2 weeks with no activity.