Circular reference detected for service "theme_handler", path: "twig -> twi g.loader.filesystem -> theme_handler -> config.factory -> high_contrast.ove rrider -> theme.manager -> theme.negotiator -> access_check.theme".

I did not investigate a lot, just maked it work

Where do you see it and which Drupal are you using?

Same as tests, 9.5.2 stable (bootstrap 3 based theme if it comes from there)

What version of Bootstrap are you using? I can't see the error.

https://www.drupal.org/project/bootstrap/releases/8.x-3.25 →

I do not know where and why this error occurs, i did not investigate at all. Bootstrap was just an assumption, hope i can give time to this next weeks :(

Maybe sharing composer.json and composer.lock would shed some more light on this?

We encountered the same exception while doing a cache clear from the UI after upgrading to this version, running Core version 9.4.8. This was without the patch.

Symfony\Component\DependencyInjection\Exception\ServiceCircularReferenceException: Circular reference detected for service "extension.list.module", path: "extension.list.module -> config.factory -> high_contrast.overrider -> theme.manager -> theme.registry". in Drupal\Component\DependencyInjection\Container->get() (line 146 of /mnt/www/html/mccloskeymr/docroot/core/lib/Drupal/Component/DependencyInjection/Container.php).

The error is not encountered when running cache clear via Drush.

Thank you, @ansonwhan . I finally saw the error by clearing cache from UI.

The original patch seems ok, I've updated it a little bit and added a some not related things that confused me (and would probably confuse any other admin)

All in all it seems we can't use theme.manager dependency injection when overriding configs from module (with the config.factory.override way in high_contrast.services.yml).

There's was actually a comment there in our code from the past:
// Not using dependency injection to prevent circular references.

I think at some point we thought hey, why aren't we using dependency injection like proper modules should do and broke it again.

Sorry for the troubles. Could you check the merge request and patch from there (the "plain diff" link)?

While we have MR for this, i've also pinged about it on core issue queue https://www.drupal.org/project/drupal/issues/3342938 💬 Circular reference detected for service "theme_handler" on cache clear Closed: outdated

Commited the changes, going for another release.

I hope it's good now. I'm setting this to fixed, new release is out.

Automatically closed - issue fixed for 2 weeks with no activity.