Hi,

I am working with the multidomain site and each domain have a different theme. I have noticed on my running site that one method "getFilePathByKey" in CriticalCssService.php is not giving the correct active theme name. It is always giving default set theme name which is wrong in my site. I'm on the latest version of D8.

I thought the issue is related to $themeName = $this->configFactory->get('system.theme')->get('default'); which should be replaced with $themeName = \Drupal::service('theme.manager')->getActiveTheme()->getName();

Comments

rakesh.drupal created an issue. See original summary.

rakesh.drupal’s picture

Attached is the patch for above mentioned issue.

  • asilgag committed 691ef9e on 8.x-1.x authored by rakesh.drupal
    Issue #3068081 by rakesh.drupal: Current theme name is not correct in...
albertosilva’s picture

Assigned: rakesh.drupal » albertosilva
Status: Active » Fixed

Thank you very much, @rakesh.drupal

I have applied your patch and will release a new module version when I finish some other pending issues.

albertosilva’s picture

Status: Fixed » Closed (fixed)