Treat required recipes as installed, every time
2 unresolved threads
2 unresolved threads
Closes #3477329
Merge request reports
Activity
Filter activity
383 383 384 384 $configurator = new RecipeConfigurator($recipe_being_validated['recipes'] ?? [], $include_path); 385 385 386 /** @var \Drupal\Core\Extension\ModuleExtensionList $module_list */ 387 $module_list = \Drupal::service('extension.list.module'); 386 388 // The config provider must either be an already-installed module or theme, 387 389 // or an extension being installed by this recipe or a recipe it depends on. 388 390 $all_extensions = [ 389 ...array_keys(\Drupal::service('extension.list.module')->getAllInstalledInfo()), 391 ...array_keys($module_list->getAllInstalledInfo()), 390 392 ...array_keys(\Drupal::service('extension.list.theme')->getAllInstalledInfo()), 391 393 ...$recipe_being_validated['install'] ?? [], 392 394 ...$configurator->listAllExtensions(), 393 395 ]; 396 // Required modules should always be considered considered installed, which changed this line in version 2 of the diff
385 385 386 /** @var \Drupal\Core\Extension\ModuleExtensionList $module_list */ 387 $module_list = \Drupal::service('extension.list.module'); 386 388 // The config provider must either be an already-installed module or theme, 387 389 // or an extension being installed by this recipe or a recipe it depends on. 388 390 $all_extensions = [ 389 ...array_keys(\Drupal::service('extension.list.module')->getAllInstalledInfo()), 391 ...array_keys($module_list->getAllInstalledInfo()), 390 392 ...array_keys(\Drupal::service('extension.list.theme')->getAllInstalledInfo()), 391 393 ...$recipe_being_validated['install'] ?? [], 392 394 ...$configurator->listAllExtensions(), 393 395 ]; 396 // Required modules should always be considered considered installed, which 397 // allows the recipe to be valid even during the early installer. 398 foreach ($module_list->getAllAvailableInfo() as $name => $info) { 399 if (!empty($info['required'])) {
Please register or sign in to reply