Follow-up to
📌
Split File hooks into separate classes
Active
where I noticed we have the following code in \Drupal\file\Hook\CronHook::__invoke():
$url_options = ['absolute' => TRUE];
if (isset($options['langcode'])) {
$url_options['language'] = $this->languageManager->getLanguage($options['langcode']);
$langcode = $options['langcode'];
}
else {
$langcode = NULL;
}
However, $url_options is never read anywhere, which means \Drupal\Core\Language\LanguageManagerInterface::getLanguage() is being called unneccessarily and the LanguageManagerInterface dependency can be removed.
Remove the $url_options variable and LanguageManagerInterface dependency.
Active
11.0 🔥
file.module
Removed $url_options as suggested, LanguageManagerInterface was not referenced in code.
Sorry I should have postponed this on 📌 Split File hooks into separate classes Active