Drupal dependency injection on file PathautoSettingsForm.php

@Thalles patch doesn't apply

working fine

1. +++ b/src/Form/PathautoSettingsForm.php
   @@ -34,14 +37,38 @@ class PathautoSettingsForm extends ConfigFormBase {
   +
   +  /**
   +   * Object moduleHandle.
   +   *
   +   * @var Drupal\Core\Extension\ModuleHandler
   +   */
   +  protected $moduleHandler;
   +
   

   Should say something like "The module handler" and so on.

2. +++ b/src/Form/PathautoSettingsForm.php
   @@ -34,14 +37,38 @@ class PathautoSettingsForm extends ConfigFormBase {
      /**
       * {@inheritdoc}
       */
   -  public function __construct(ConfigFactoryInterface $config_factory, EntityTypeManagerInterface $entity_type_manager, EntityFieldManagerInterface $entity_field_manager, AliasTypeManager $alias_type_manager) {
   

   docblock needs to cover all arguments.

3. +++ b/src/Form/PathautoSettingsForm.php
   @@ -52,7 +79,10 @@ class PathautoSettingsForm extends ConfigFormBase {
   +      $container->get('entity.query'),
   

   the entity query service is deprecated, this should use entityTypeManager->getStorage('x')->getQuery()

4. +++ b/src/Form/PathautoSettingsForm.php
   @@ -127,10 +157,10 @@ class PathautoSettingsForm extends ConfigFormBase {
    
   -    $max_length = \Drupal::service('pathauto.alias_storage_helper')->getAliasSchemaMaxlength();
   +    $max_length = $this->serviceContainer->get('pathauto.alias_storage_helper')->getAliasSchemaMaxlength();
    
   

   this doesn't make sense, it should inject the actual service instead.

Attached patch implements the following changes:

1. #8.1 Standardised the property descriptions based on similar occurrences in Drupal Core.
2. #8.2 Updated the__construct() docblock.
3. #8.3 / #8.4 have been fixed in #14
4. Removed additional newlines that were introduced in this issue.
5. Reordered use statements alphabetically for better readability.

Thanks.