Injection on PathautoGenerator

1. +++ b/pathauto.services.yml
   @@ -1,7 +1,7 @@
    services:
      pathauto.generator:
        class: Drupal\pathauto\PathautoGenerator
   -    arguments: ['@config.factory', '@module_handler', '@token', '@pathauto.alias_cleaner', '@pathauto.alias_storage_helper', '@pathauto.alias_uniquifier', '@pathauto.verbose_messenger', '@string_translation', '@token.entity_mapper', '@entity_type.manager']
   +    arguments: ['@config.factory', '@module_handler', '@token', '@pathauto.alias_cleaner', '@pathauto.alias_storage_helper', '@pathauto.alias_uniquifier', '@pathauto.verbose_messenger', '@string_translation', '@token.entity_mapper', '@entity_type.manager', '@plugin.manager.alias_type']
   

   Let's add an empty post_update hook so the services container is rebuilt, see \taxonomy_post_update_clear_entity_bundle_field_definitions_cache() for an example. Existing sites will trigger an error: ArgumentCountError: Too few arguments to function Drupal\pathauto\PathautoGenerator::__construct()

2. +++ b/src/PathautoGenerator.php
   @@ -134,6 +143,7 @@ class PathautoGenerator implements PathautoGeneratorInterface {
   +    $this->aliasTypeManager = $plugin_manager_alias_type;
   

   Let's use $alias_type_manager for consistency with the other variables.

1. I'm not that fond of the empty post update pattern, we could for BC just make the arguments optional and fall back to \Drupal::service() in the constructor, then it will work fine too.

Thanks, committed.