Phpcs coding standard

Created on 6 April 2020, over 4 years ago
Updated 24 July 2024, about 2 months ago

FILE: ...s/drupal/modules/field_layout/src/Entity/FieldLayoutEntityDisplayTrait.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------------
73 | WARNING | Line exceeds 80 characters; contains 91 characters
81 | WARNING | Line exceeds 80 characters; contains 91 characters
--------------------------------------------------------------------------------

FILE: ...al/modules/field_layout/tests/src/FunctionalJavascript/FieldLayoutTest.php
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
18 | ERROR | If the line declaring an array spans longer than 80 characters,
| | each element should be broken into its own line
--------------------------------------------------------------------------------

FILE: ...tdocs/drupal/modules/field_layout/tests/src/Functional/FieldLayoutTest.php
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
17 | ERROR | If the line declaring an array spans longer than 80 characters,
| | each element should be broken into its own line
--------------------------------------------------------------------------------

FILE: ...docs/drupal/modules/field_layout/tests/src/Unit/FieldLayoutBuilderTest.php
--------------------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 5 LINES
--------------------------------------------------------------------------------
21 | ERROR | Missing short description in doc comment
26 | ERROR | Missing short description in doc comment
31 | ERROR | Missing short description in doc comment
36 | ERROR | Missing short description in doc comment
41 | ERROR | Missing short description in doc comment
--------------------------------------------------------------------------------

FILE: .../drupal/modules/field_layout/tests/src/Kernel/FieldLayoutUninstallTest.php
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
7 | ERROR | Missing short description in doc comment
--------------------------------------------------------------------------------

FILE: ...pal/modules/field_layout/tests/src/Kernel/FieldLayoutEntityDisplayTest.php
--------------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
--------------------------------------------------------------------------------
17 | ERROR | If the line declaring an array spans longer than 80 characters,
| | each element should be broken into its own line
172 | ERROR | Missing parameter comment
173 | ERROR | Missing parameter comment
174 | ERROR | Missing parameter comment
--------------------------------------------------------------------------------

FILE: ...es/field_layout/tests/modules/field_layout_test/field_layout_test.info.yml
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
1 | WARNING | Remove "version" from the info file, it will be added by
| | drupal.org packaging automatically
--------------------------------------------------------------------------------

FILE: /opt/lampp/htdocs/drupal/modules/field_layout/field_layout.info.yml
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
1 | WARNING | Remove "version" from the info file, it will be added by
| | drupal.org packaging automatically
--------------------------------------------------------------------------------

🐛 Bug report
Status

Closed: works as designed

Version

1.0

Component

Code

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • 🇮🇳India riddhi.addweb

    I have applied the patch cleanly but still, there are some errors left.
    Please check the screenshot for the same.

    FILE: /var/www/html/web/Drupal_9.5/web/modules/contrib/field_layout-field_layout/tests/src/Unit/FieldLayoutBuilderTest.php
    -----------------------------------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    -----------------------------------------------------------------------------------------------------------------------------------------
     9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Layout\LayoutPluginManagerInterface.
    -----------------------------------------------------------------------------------------------------------------------------------------
    PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
    -----------------------------------------------------------------------------------------------------------------------------------------
    
    
    FILE: /var/www/html/web/Drupal_9.5/web/modules/contrib/field_layout-field_layout/src/FieldLayoutBuilder.php
    -----------------------------------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    -----------------------------------------------------------------------------------------------------------------------------------------
     9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Layout\LayoutPluginManagerInterface.
    -----------------------------------------------------------------------------------------------------------------------------------------
    PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
    -----------------------------------------------------------------------------------------------------------------------------------------
    
    
    FILE: /var/www/html/web/Drupal_9.5/web/modules/contrib/field_layout-field_layout/field_layout.module
    ----------------------------------------------------------------------------------------------------
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ----------------------------------------------------------------------------------------------------
     37 | WARNING | [x] The variable name should be defined after the type
    ----------------------------------------------------------------------------------------------------
    PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
    ----------------------------------------------------------------------------------------------------
    
  • Status changed to Closed: works as designed about 2 months ago
  • 🇺🇸United States bnjmnm Ann Arbor, MI

    Your passion for coding standards in contrib modules is commendable, but we're fine with it as is. Thanks.

Production build 0.71.5 2024