Commit 355a1a68 authored by catch's avatar catch
Browse files

Issue #3285131 by andregp, daffie, catch, longwave: Fix constructor... 

Issue #3285131 by andregp, daffie, catch, longwave: Fix constructor deprecation in Drupal\Core\Theme\Registry::__construct()
parent 70d480ab

core/core.services.yml

+1 −1
Original line number Diff line number Diff line
@@ -1416,7 +1416,7 @@ services: 
    arguments: ['%app.root%', '@theme_handler', '@cache.bootstrap', '@module_handler']

  theme.registry:

    class: Drupal\Core\Theme\Registry

    arguments: ['%app.root%', '@cache.default', '@lock', '@module_handler', '@theme_handler', '@theme.initialization', null, '@cache.bootstrap', '@extension.list.module']

    arguments: ['%app.root%', '@cache.default', '@lock', '@module_handler', '@theme_handler', '@theme.initialization', '@cache.bootstrap', '@extension.list.module']

    tags:

      - { name: needs_destruction }

    calls:

core/lib/Drupal/Core/Theme/Registry.php

+4 −4
Original line number Diff line number Diff line
@@ -174,19 +174,18 @@ class Registry implements DestructableInterface { 
   *   The theme handler.

   * @param \Drupal\Core\Theme\ThemeInitializationInterface $theme_initialization

   *   The theme initialization.

   * @param string $theme_name

   *   (optional) The name of the theme for which to construct the registry.

   * @param \Drupal\Core\Cache\CacheBackendInterface $runtime_cache

   *   The cache backend interface to use for the runtime theme registry data.

   * @param \Drupal\Core\Extension\ModuleExtensionList $module_list

   *   The module list.

   * @param string $theme_name

   *   (optional) The name of the theme for which to construct the registry.

   */

  public function __construct($root, CacheBackendInterface $cache, LockBackendInterface $lock, ModuleHandlerInterface $module_handler, ThemeHandlerInterface $theme_handler, ThemeInitializationInterface $theme_initialization, $theme_name = NULL, CacheBackendInterface $runtime_cache = NULL, ModuleExtensionList $module_list = NULL) {

  public function __construct($root, CacheBackendInterface $cache, LockBackendInterface $lock, ModuleHandlerInterface $module_handler, ThemeHandlerInterface $theme_handler, ThemeInitializationInterface $theme_initialization, CacheBackendInterface $runtime_cache, ModuleExtensionList $module_list, $theme_name = NULL) {

    $this->root = $root;

    $this->cache = $cache;

    $this->lock = $lock;

    $this->moduleHandler = $module_handler;

    $this->themeName = $theme_name;

    $this->themeHandler = $theme_handler;

    $this->themeInitialization = $theme_initialization;

    $this->runtimeCache = $runtime_cache;
@@ -195,6 +194,7 @@ public function __construct($root, CacheBackendInterface $cache, LockBackendInte 
      $module_list = \Drupal::service('extension.list.module');

    }

    $this->moduleList = $module_list;

    $this->themeName = $theme_name;

  }



  /**

core/tests/Drupal/KernelTests/Core/Theme/RegistryTest.php

+6 −6
Original line number Diff line number Diff line
@@ -74,11 +74,11 @@ public function testMultipleSubThemes() { 
    $module_list = $this->container->get('extension.list.module');

    assert($module_list instanceof ModuleExtensionList);



    $registry_subsub_theme = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $theme_handler, \Drupal::service('theme.initialization'), 'test_subsubtheme', NULL, $module_list);

    $registry_subsub_theme = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $theme_handler, \Drupal::service('theme.initialization'), \Drupal::service('cache.bootstrap'), $module_list, 'test_subsubtheme');

    $registry_subsub_theme->setThemeManager(\Drupal::theme());

    $registry_sub_theme = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $theme_handler, \Drupal::service('theme.initialization'), 'test_subtheme', NULL, $module_list);

    $registry_sub_theme = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $theme_handler, \Drupal::service('theme.initialization'), \Drupal::service('cache.bootstrap'), $module_list, 'test_subtheme');

    $registry_sub_theme->setThemeManager(\Drupal::theme());

    $registry_base_theme = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $theme_handler, \Drupal::service('theme.initialization'), 'test_basetheme', NULL, $module_list);

    $registry_base_theme = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $theme_handler, \Drupal::service('theme.initialization'), \Drupal::service('cache.bootstrap'), $module_list, 'test_basetheme');

    $registry_base_theme->setThemeManager(\Drupal::theme());



    $preprocess_functions = $registry_subsub_theme->get()['theme_test_template_test']['preprocess functions'];
@@ -112,7 +112,7 @@ public function testSuggestionPreprocessFunctions() { 


    $extension_list = $this->container->get('extension.list.module');

    assert($extension_list instanceof ModuleExtensionList);

    $registry_theme = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $theme_handler, \Drupal::service('theme.initialization'), 'test_theme', NULL, $extension_list);

    $registry_theme = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $theme_handler, \Drupal::service('theme.initialization'), \Drupal::service('cache.bootstrap'), $extension_list, 'test_theme');

    $registry_theme->setThemeManager(\Drupal::theme());



    $suggestions = ['__kitten', '__flamingo'];
@@ -154,7 +154,7 @@ public function testThemeRegistryAlterByTheme() { 


    $extension_list = $this->container->get('extension.list.module');

    assert($extension_list instanceof ModuleExtensionList);

    $registry = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $theme_handler, \Drupal::service('theme.initialization'), 'test_theme', NULL, $extension_list);

    $registry = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $theme_handler, \Drupal::service('theme.initialization'), \Drupal::service('cache.bootstrap'), $extension_list, 'test_theme');

    $registry->setThemeManager(\Drupal::theme());

    $this->assertEquals('value', $registry->get()['theme_test_template_test']['variables']['additional']);

  }
@@ -244,7 +244,7 @@ public function testThemeTemplatesRegisteredByModules() { 


    $extension_list = \Drupal::service('extension.list.module');

    assert($extension_list instanceof ModuleExtensionList);

    $registry_theme = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $theme_handler, \Drupal::service('theme.initialization'), 'test_theme', NULL, $extension_list);

    $registry_theme = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $theme_handler, \Drupal::service('theme.initialization'), \Drupal::service('cache.bootstrap'), $extension_list, 'test_theme');

    $registry_theme->setThemeManager(\Drupal::theme());



    $expected = [

core/tests/Drupal/KernelTests/Core/Theme/Stable9TemplateOverrideTest.php

+1 −1
Original line number Diff line number Diff line
@@ -87,7 +87,7 @@ protected function installAllModules() { 
   * Ensures that Stable 9 overrides all relevant core templates.

   */

  public function testStable9TemplateOverrides() {

    $registry = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $this->themeHandler, \Drupal::service('theme.initialization'), 'stable9', NULL, \Drupal::service('extension.list.module'));

    $registry = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $this->themeHandler, \Drupal::service('theme.initialization'), \Drupal::service('cache.bootstrap'), \Drupal::service('extension.list.module'), 'stable9');

    $registry->setThemeManager(\Drupal::theme());



    $registry_full = $registry->get();

core/tests/Drupal/KernelTests/Core/Theme/StableTemplateOverrideTest.php

+1 −1
Original line number Diff line number Diff line
@@ -85,7 +85,7 @@ protected function installAllModules() { 
   * Ensures that Stable overrides all relevant core templates.

   */

  public function testStableTemplateOverrides() {

    $registry = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $this->themeHandler, \Drupal::service('theme.initialization'), 'stable', NULL, \Drupal::service('extension.list.module'));

    $registry = new Registry($this->root, \Drupal::cache(), \Drupal::lock(), \Drupal::moduleHandler(), $this->themeHandler, \Drupal::service('theme.initialization'), \Drupal::service('cache.bootstrap'), \Drupal::service('extension.list.module'), 'stable');

    $registry->setThemeManager(\Drupal::theme());



    $registry_full = $registry->get();