NavigationHooks::modulesInstalled() does not invoke all the hook_navigation_defaults() implementations

Problem/Motivation

Drupal\navigation\Hook\NavigationHooks::modulesInstalled() uses the following code.

    // Do not modify config during sync. Config should be already consolidated.
    if ($is_syncing) {
      return;
    }
    foreach ($modules as $module) {
      $blocks = $this->moduleHandler->invoke($module, 'navigation_defaults');

      if (!is_array($blocks)) {
        return;
      }

      foreach ($blocks as $block) {
        $this->configActionManager->applyAction('addNavigationBlock', 'navigation.block_layout', $block);
      }
    }

It stops invoking those hook implementations after one of them returns a value that is different from the expected one.

navigation_install() uses instead the following code.

  // Do not modify config during sync. Config should be already consolidated.
  if ($is_syncing) {
    return;
  }

  $blocks = \Drupal::moduleHandler()->invokeAll('navigation_defaults');
  $manager = \Drupal::service('plugin.manager.config_action');
  foreach ($blocks as $block) {
    $manager->applyAction('addNavigationBlock', 'navigation.block_layout', $block);
  }

Proposed resolution

Change Drupal\navigation\Hook\NavigationHooks::modulesInstalled() to use the following code.

    // Do not modify config during sync. Config should be already consolidated.
    if ($is_syncing) {
      return;
    }
    foreach ($modules as $module) {
      $blocks = $this->moduleHandler->invoke($module, 'navigation_defaults');

      if (!is_array($blocks)) {
        continue;
      }

      foreach ($blocks as $block) {
        $this->configActionManager->applyAction('addNavigationBlock', 'navigation.block_layout', $block);
      }
    }

navigation_install() does not need to be changed, since ModuleHandler::invokeAll() returns an array.