\Drupal calls should be avoided in classes, use dependency injection instead

I review your patch.

I have reviewed this patch but patch is not applied cleanly.
Thanks

I reviewed the MR the current changes crashed the modules. Providing the updated MR.
ArgumentCountError: Too few arguments to function Drupal\eu_cookie_compliance\EventSubscriber\EuCookieComplianceConfigEventsSubscriber::__construct(), 0 passed in /app/web/core/lib/Drupal/Component/DependencyInjection/Container.php on line 259 and exactly 3 expected in Drupal\eu_cookie_compliance\EventSubscriber\EuCookieComplianceConfigEventsSubscriber->__construct() (line 65 of /app/web/modules/contrib/eu_cookie_compliance/src/EventSubscriber/EuCookieComplianceConfigEventsSubscriber.php)

Thanks!

The issue summary should always describe what should be fixed/changed. Neither the title nor screenshots are sufficient to describe what the issue is, even in the case a patch is provided.

We fixed this over in 🐛 \Drupal calls should be avoided in classes, use dependency injection instead Fixed

Thanks all for the effort.

svenryen → closed merge request !121