Verified Commit 5e3b8c6d authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3324995 by claudiu.cristea, GaëlG, dajjen: hook_requirements_alter() is... 

Issue #3324995 by claudiu.cristea, GaëlG, dajjen: hook_requirements_alter() is not invoked on update

(cherry picked from commit 64e5e002)
parent 4c461146

core/includes/update.inc

+1 −0
Original line number Diff line number Diff line
@@ -52,6 +52,7 @@ function update_check_requirements() { 


  // Check requirements of all loaded modules.

  $requirements = \Drupal::moduleHandler()->invokeAll('requirements', ['update']);

  \Drupal::moduleHandler()->alter('requirements', $requirements);

  $requirements += update_system_schema_requirements();

  return $requirements;

}

core/modules/system/tests/modules/update_script_test/update_script_test.install

+10 −1
Original line number Diff line number Diff line
@@ -28,7 +28,7 @@ function update_script_test_requirements($phase) { 
        $requirements['update_script_test'] = [

          'title' => 'Update script test',

          'value' => 'Error',

          'description' => 'This is a requirements error provided by the update_script_test module.',

          'description' => 'This is a (buggy description fixed in update_script_test_requirements_alter()) requirements error provided by the update_script_test module.',

          'severity' => REQUIREMENT_ERROR,

        ];

        break;
@@ -38,6 +38,15 @@ function update_script_test_requirements($phase) { 
  return $requirements;

}



/**

 * Implements hook_requirements_alter().

 */

function update_script_test_requirements_alter(array &$requirements): void {

  if (isset($requirements['update_script_test']) && $requirements['update_script_test']['severity'] === REQUIREMENT_ERROR) {

    $requirements['update_script_test']['description'] = 'This is a requirements error provided by the update_script_test module.';

  }

}



/**

 * Implements hook_update_last_removed().

 */