Verified Commit 99c2af65 authored by quietone's avatar quietone
Browse files

Issue #3537396 by jesseh, borisson_: Removed function block_themes_installed()... 

Issue #3537396 by jesseh, borisson_: Removed function block_themes_installed() is still referenced in comments

(cherry picked from commit b74f08a1)
parent 8e4b392d

core/modules/block/src/Hook/BlockHooks.php

+5 −4
Original line number Diff line number Diff line
@@ -157,10 +157,11 @@ public function modulesInstalled($modules, bool $is_syncing): void { 
      return;

    }



    // block_themes_installed() does not call block_theme_initialize() during

    // site installation because block configuration can be optional or provided

    // by the profile. Now, when the profile is installed, this configuration

    // exists, call block_theme_initialize() for all installed themes.

    // BlockHooks::themesInstalled() does not call block_theme_initialize()

    // during site installation because block configuration can be optional or

    // provided by the profile. Now, when the profile is installed, this

    // configuration exists, call block_theme_initialize() for all installed

    // themes.

    $profile = \Drupal::installProfile();

    if (in_array($profile, $modules, TRUE)) {

      foreach (\Drupal::service('theme_handler')->listInfo() as $theme => $data) {

core/modules/block/tests/src/Kernel/NewDefaultThemeBlocksTest.php

+4 −1
Original line number Diff line number Diff line
@@ -51,7 +51,10 @@ protected function setUp(): void { 
  }



  /**

   * Check the blocks are correctly copied by block_themes_installed().

   * Check the blocks are correctly copied.

   *

   * This tests that blocks are correctly copied by

   * \Drupal\block\Hook\BlockHooks::themesInstalled().

   */

  public function testNewDefaultThemeBlocks(): void {

    $default_theme = $this->defaultTheme;