Contrib.social

Improve coding standards

Open on Drupal.org →
Created on 15 August 2024, 4 months ago
Updated 18 September 2024, 3 months ago

This patch improves the organization of the use statements in the pathauto module files, adjusting them to comply with the coding standards recommended by Drupal. The goal is to ensure that dependencies are arranged logically and consistently, making the code easier to read and maintain.

🐛 Bug report
Status

Needs work

Version

1.0

Component

Code

Created by

🇨🇴Colombia andresalvarez Bogota

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @andresalvarez
  • Issue was unassigned.
  • Status changed to Needs review 3 months ago
  • Comment 3 months ago →
    🇳🇴Norway steinmb
  • Comment 3 months ago →
    🇵🇪Peru alyaj2a

    Hi! I applied the patch but I still get Coding Standard errors.

    phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml web/modules/contrib/pathauto/

    FILE: .../web/modules/contrib/pathauto/src/Commands/PathautoCommands.php
    -------------------------------------------------------------------------------------------------
    FOUND 10 ERRORS AFFECTING 10 LINES
    -------------------------------------------------------------------------------------------------
    65 | ERROR | Doc comment short description must start with a capital letter
    69 | ERROR | Parameter tags must be defined first in a doc comment
    107 | ERROR | Parameter $options is not described in comment
    112 | ERROR | Parameter tags must be defined first in a doc comment
    170 | ERROR | Missing short description in doc comment
    183 | ERROR | Missing short description in doc comment
    197 | ERROR | Missing short description in doc comment
    201 | ERROR | @throws tag comment must end with a full stop
    216 | ERROR | Missing short description in doc comment
    220 | ERROR | @throws tag comment must end with a full stop
    -------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/src/Entity/PathautoPattern.php
    ----------------------------------------------------------------------------------------------
    FOUND 3 ERRORS AFFECTING 3 LINES
    ----------------------------------------------------------------------------------------------
    101 | ERROR | Missing short description in doc comment
    127 | ERROR | Missing short description in doc comment
    132 | ERROR | Missing short description in doc comment
    ----------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/src/Form/PatternEditForm.php
    --------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    --------------------------------------------------------------------------------------------
    25 | ERROR | Missing short description in doc comment
    --------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/src/PathautoState.php
    -------------------------------------------------------------------------------------
    FOUND 2 ERRORS AFFECTING 2 LINES
    -------------------------------------------------------------------------------------
    30 | ERROR | Missing short description in doc comment
    123 | ERROR | Description for the @return value is missing
    -------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/src/PathautoGeneratorInterface.php
    --------------------------------------------------------------------------------------------------
    FOUND 8 ERRORS AFFECTING 5 LINES
    --------------------------------------------------------------------------------------------------
    13 | ERROR | Doc comment short description must start with a capital letter
    13 | ERROR | Doc comment short description must end with a full stop
    18 | ERROR | Doc comment short description must start with a capital letter
    18 | ERROR | Doc comment short description must end with a full stop
    23 | ERROR | Doc comment short description must start with a capital letter
    23 | ERROR | Doc comment short description must end with a full stop
    53 | ERROR | Description for the @return value is missing
    81 | ERROR | Parameter comment must start with a capital letter
    --------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/src/AliasUniquifier.php
    ---------------------------------------------------------------------------------------
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ---------------------------------------------------------------------------------------
    105 | WARNING | Line exceeds 80 characters; contains 81 characters
    ---------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/src/AliasUniquifierInterface.php
    ------------------------------------------------------------------------------------------------
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ------------------------------------------------------------------------------------------------
    13 | WARNING | Line exceeds 80 characters; contains 81 characters
    ------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/src/PathautoPatternInterface.php
    ------------------------------------------------------------------------------------------------
    FOUND 11 ERRORS AFFECTING 11 LINES
    ------------------------------------------------------------------------------------------------
    15 | ERROR | Description for the @return value is missing
    22 | ERROR | Missing parameter comment
    31 | ERROR | Description for the @return value is missing
    35 | ERROR | Missing short description in doc comment
    36 | ERROR | Description for the @return value is missing
    43 | ERROR | Description for the @return value is missing
    60 | ERROR | Description for the @return value is missing
    128 | ERROR | Description for the @return value is missing
    149 | ERROR | Description for the @return value is missing
    174 | ERROR | Missing parameter type
    177 | ERROR | Description for the @return value is missing
    ------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/src/PathautoWidget.php
    --------------------------------------------------------------------------------------
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    --------------------------------------------------------------------------------------
    35 | WARNING | Line exceeds 80 characters; contains 144 characters
    --------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/src/Plugin/pathauto/AliasType/EntityAliasTypeBase.php
    ----------------------------------------------------------------------------------------------------------------
    FOUND 2 ERRORS AFFECTING 1 LINE
    ----------------------------------------------------------------------------------------------------------------
    300 | ERROR | The text '@deprecated Use \Drupal\pathauto\PathautoState::bulkDelete() instead.' does not match
    | | the standard format: @deprecated in %deprecation-version% and is removed from %removal-version%.
    | | %extra-info%.
    300 | ERROR | Each @deprecated tag must have a @see tag immediately following it
    ----------------------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/src/Plugin/Deriver/EntityAliasTypeDeriver.php
    -------------------------------------------------------------------------------------------------------------
    FOUND 2 ERRORS AFFECTING 2 LINES
    -------------------------------------------------------------------------------------------------------------
    23 | ERROR | Missing short description in doc comment
    35 | ERROR | Missing short description in doc comment
    -------------------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/src/EventSubscriber/PathautoSettingsCacheTag.php
    ----------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    ----------------------------------------------------------------------------------------------------------------
    16 | ERROR | Missing short description in doc comment
    ----------------------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/PATCHES.txt
    ---------------------------------------------------------------------------
    FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
    ---------------------------------------------------------------------------
    1 | WARNING | [ ] Line exceeds 80 characters; contains 104 characters
    5 | ERROR | [x] Expected 1 newline at end of file; 3 found
    ---------------------------------------------------------------------------
    PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
    ---------------------------------------------------------------------------

    FILE: ...es/contrib/pathauto/tests/modules/pathauto_custom_punctuation_test/pathauto_custom_punctuation_test.module
    ----------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    ----------------------------------------------------------------------------------------------------------------
    3 | ERROR | Missing short description in doc comment
    ----------------------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/tests/src/Functional/PathautoEnablingEntityTypesTest.php
    ----------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    ----------------------------------------------------------------------------------------------------------------
    63 | ERROR | Doc comment short description must be on a single line, further text should be a separate
    | | paragraph
    ----------------------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/tests/src/Functional/PathautoNodeWebTest.php
    ------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AND 2 WARNINGS AFFECTING 3 LINES
    ------------------------------------------------------------------------------------------------------------
    184 | ERROR | Missing short description in doc comment
    210 | WARNING | Line exceeds 80 characters; contains 92 characters
    227 | WARNING | Line exceeds 80 characters; contains 81 characters
    ------------------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/tests/src/Functional/PathautoBulkUpdateTest.php
    ---------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    ---------------------------------------------------------------------------------------------------------------
    75 | ERROR | Doc comment is empty
    ---------------------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/tests/src/Kernel/PathautoTokenTest.php
    ------------------------------------------------------------------------------------------------------
    FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
    ------------------------------------------------------------------------------------------------------
    23 | ERROR | Doc comment is empty
    41 | WARNING | Line exceeds 80 characters; contains 81 characters
    127 | ERROR | Doc comment is empty
    ------------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/tests/src/Kernel/PathautoEntityWithStringIdTest.php
    ----------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    ----------------------------------------------------------------------------------------------------------------
    78 | ERROR | Parameter tags must be defined first in a doc comment
    ----------------------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/tests/src/FunctionalJavascript/PathautoUiTest.php
    ----------------------------------------------------------------------------------------------------------------
    FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
    ----------------------------------------------------------------------------------------------------------------
    62 | ERROR | Doc comment is empty
    75 | ERROR | Doc comment is empty
    115 | WARNING | Line exceeds 80 characters; contains 97 characters
    ----------------------------------------------------------------------------------------------------------------

    FILE: .../web/modules/contrib/pathauto/tests/src/FunctionalJavascript/PathautoLocaleTest.php
    ----------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    ----------------------------------------------------------------------------------------------------------------
    45 | ERROR | Doc comment short description must be on a single line, further text should be a separate
    | | paragraph
    ----------------------------------------------------------------------------------------------------------------

    Time: 1.29 secs; Memory: 16MB

  • Comment 3 months ago →
    🇵🇪Peru alyaj2a
  • Status changed to Needs work 3 months ago
  • Comment 3 months ago →
    🇳🇴Norway steinmb

    Based on feedback in #4

  • Comment 3 months ago →
    🇨🇴Colombia andresalvarez Bogota

    hello sorry I uploaded a test patch that I had and I will upload the updated one.

  • Comment 3 months ago →
    🇨🇴Colombia andresalvarez Bogota

    i upload real patch adjusted

  • First commit to issue fork.
  • Merge request !90MR created from patch#7. → (Open) created by chaitanyadessai
  • Comment 3 months ago →
    🇮🇳India chaitanyadessai Goa
  • Comment 3 months ago →
    🇮🇳India chaitanyadessai Goa

    Raised MR from #7 which fixes all code standard issue.

  • Pipeline finished with Failed
    3 months ago
    Total: 616s
    #290130
contrib.social Blog FAQ Discussions
Production build 0.71.5 2024