Contribution Date
Technology
Contribution Project
Contribution Details
@longwave , thankyou for review . Removing state injected service from ProviderRepository class from last patch.
There more instances that can be injected:
grep -nri '\Drupal::state()' core | grep -v .module | grep -vi test | grep -v .api.php | grep -v authorize.php | grep -v .inc
core/lib/Drupal/Core/Render/HtmlResponseAttachmentsProcessor.php:
321:$optimize_js = !defined('MAINTENANCE_MODE') && !\Drupal::state()->get('system.maintenance_mode') && $this->config->get('js.preprocess');
core/lib/Drupal/Core/Updater/Theme.php:52: $themes = \Drupal::state()->get('system.theme.files', []);
Injecting state service in HtmlResponseAttachmentsProcessor file , but for Theme.php i am not sure because it is not using create menthod neither it is service.
Kindly review a new patch.
Contribution Issue Link
Contribution Link
Files count
0
Patches count
0