Fix issues reported by PHPCS

Created on 14 February 2024, 9 months ago
Updated 15 February 2024, 9 months ago

Problem/Motivation

With 📌 Integration with Gitlab CI Fixed in place, we can start checking for PHPCS. There is a report now for the issues we can fix.

FILE: .../web/modules/contrib/admin_toolbar_content/admin_toolbar_content.api.php
--------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 7 WARNINGS AFFECTING 7 LINES
--------------------------------------------------------------------------------------------------------------
 67 | WARNING | [x] A comma should follow the last multiline array item. Found: 'item b'
 73 | WARNING | [x] A comma should follow the last multiline array item. Found: 'item c'
 74 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
 75 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
 76 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
 77 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
 78 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
--------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/admin_toolbar_content.install
--------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 2 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------------------------------------
  1 | ERROR   | [x] Missing file doc comment
 42 | WARNING | [x] A comma should follow the last multiline array item. Found: 1
 53 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
--------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/admin_toolbar_content.module
-----------------------------------------------------------------------------------------------------------------------
FOUND 24 ERRORS AND 9 WARNINGS AFFECTING 21 LINES
-----------------------------------------------------------------------------------------------------------------------
   1 | ERROR   | [x] Missing file doc comment
   6 | ERROR   | [x] Use statements should be sorted alphabetically. The first wrong one is
     |         |     Drupal\Core\Routing\RouteMatchInterface.
  15 | ERROR   | [x] Line indented incorrectly; expected 4 spaces, found 6
  16 | ERROR   | [x] Line indented incorrectly; expected 4 spaces, found 6
  41 | WARNING | [ ] Format should be "* Implements hook_foo().", "* Implements hook_foo_BAR_ID_bar() for
     |         |     xyz_bar().",, "* Implements hook_foo_BAR_ID_bar() for xyz-bar.html.twig.", "* Implements
     |         |     hook_foo_BAR_ID_bar() for xyz-bar.tpl.php.", or "* Implements hook_foo_BAR_ID_bar() for block
     |         |     templates."
  41 | ERROR   | [ ] Doc comment short description must end with a full stop
  69 | WARNING | [x] There must be no blank line following an inline comment
  69 | WARNING | [ ] There must be no blank line following an inline comment
  76 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
  78 | ERROR   | [x] Expected 1 space after SWITCH keyword; 0 found
  83 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
  90 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
 132 | WARNING | [x] There must be no blank line following an inline comment
 132 | WARNING | [ ] There must be no blank line following an inline comment
 138 | ERROR   | [x] Expected 1 space after IF keyword; 0 found
 139 | WARNING | [x] There must be no blank line following an inline comment
 139 | WARNING | [ ] There must be no blank line following an inline comment
 139 | ERROR   | [x] No space found before comment text; expected "// check current path is user profile page" but
     |         |     found "//check current path is user profile page"
 139 | ERROR   | [x] Inline comments must start with a capital letter
 139 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing
     |         |     parentheses
 151 | ERROR   | [x] Concat operator must be surrounded by a single space
 153 | ERROR   | [x] No space found before comment text; expected "// else {" but found "//else {"
 153 | ERROR   | [x] Inline comments must start with a capital letter
 154 | ERROR   | [x] Line indented incorrectly; expected 10 spaces, found 12
 155 | ERROR   | [x] Line indented incorrectly; expected 10 spaces, found 12
 156 | ERROR   | [x] Line indented incorrectly; expected 10 spaces, found 12
 157 | ERROR   | [x] No space found before comment text; expected "// }" but found "//}"
 157 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing
     |         |     parentheses
 169 | WARNING | [ ] Hook implementations should not duplicate @param documentation
 169 | ERROR   | [ ] Parameter tags must be defined first in a doc comment
 169 | ERROR   | [ ] Missing parameter comment
 169 | ERROR   | [ ] Missing parameter type
 174 | WARNING | [x] A comma should follow the last multiline array item. Found: 'preRender'
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 24 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/AdminToolbarContentPluginManagerInterface.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 6 ERRORS AFFECTING 6 LINES
-----------------------------------------------------------------------------------------------------------------------
 18 | ERROR | Missing parameter comment
 20 | ERROR | Description for the @return value is missing
 27 | ERROR | Missing parameter comment
 29 | ERROR | Description for the @return value is missing
 36 | ERROR | Missing parameter comment
 38 | ERROR | Description for the @return value is missing
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/Form/AdminToolbarContentSettingsForm.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 9 ERRORS AND 1 WARNING AFFECTING 8 LINES
-----------------------------------------------------------------------------------------------------------------------
  12 | ERROR   | [x] Missing class doc comment
  21 | ERROR   | [ ] Missing short description in doc comment
  33 | ERROR   | [ ] Missing parameter comment
  80 | ERROR   | [x] Short array syntax must be used to define arrays
  86 | ERROR   | [x] Short array syntax must be used to define arrays
 119 | ERROR   | [x] Expected 1 space after comma in argument list; 2 found
 119 | ERROR   | [x] Expected one space after the comma, 2 found
 121 | ERROR   | [x] Expected newline after closing brace
 121 | ERROR   | [x] Namespaced classes/interfaces/traits should be referenced with use statements
 144 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/AlternativeContentView.php
---------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
---------------------------------------------------------------------------------------------------------------
 57 | ERROR | [x] Expected 1 blank line after function; 0 found
 58 | ERROR | [x] The closing brace for the class must have an empty line before it
---------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/AdminToolbarContentPluginInterface.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 20 ERRORS AFFECTING 20 LINES
-----------------------------------------------------------------------------------------------------------------------
  1 | ERROR | [x] The PHP open tag must be followed by exactly one blank line
  8 | ERROR | [x] Expected 1 space(s) before asterisk; 0 found
  9 | ERROR | [x] Expected 1 space(s) before asterisk; 0 found
 14 | ERROR | [ ] Doc comment short description must be on a single line, further text should be a separate paragraph
 16 | ERROR | [ ] Missing parameter comment
 17 | ERROR | [ ] Missing parameter comment
 19 | ERROR | [ ] Description for the @return value is missing
 26 | ERROR | [ ] Description for the @return value is missing
 32 | ERROR | [ ] Doc comment short description must be on a single line, further text should be a separate paragraph
 34 | ERROR | [ ] Missing parameter comment
 36 | ERROR | [ ] Description for the @return value is missing
 43 | ERROR | [ ] Missing parameter comment
 45 | ERROR | [ ] Description for the @return value is missing
 47 | ERROR | [x] Expected 1 blank line after function; 2 found
 53 | ERROR | [ ] Missing parameter comment
 55 | ERROR | [ ] Description for the @return value is missing
 61 | ERROR | [ ] Doc comment short description must be on a single line, further text should be a separate paragraph
 63 | ERROR | [ ] Missing parameter comment
 64 | ERROR | [ ] Missing parameter comment
 66 | ERROR | [ ] Description for the @return value is missing
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/Plugin/Derivative/AdminToolbarContentMenuLinks.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
-----------------------------------------------------------------------------------------------------------------------
 19 | ERROR | [ ] Missing short description in doc comment
 27 | ERROR | [ ] Missing parameter comment
 56 | ERROR | [x] Expected newline after closing brace
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/Plugin/AdminToolbarContent/AdminToolbarContentDrupalMenuPlugin.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 3 WARNINGS AFFECTING 4 LINES
-----------------------------------------------------------------------------------------------------------------------
  1 | ERROR   | [x] The PHP open tag must be followed by exactly one blank line
 36 | WARNING | [x] A comma should follow the last multiline array item. Found: ''
 65 | WARNING | [x] A comma should follow the last multiline array item. Found: )
 69 | WARNING | [x] A comma should follow the last multiline array item. Found: 'admin_toolbar_tools.help'
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/Plugin/AdminToolbarContent/AdminToolbarContentMediaPlugin.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AND 1 WARNING AFFECTING 5 LINES
-----------------------------------------------------------------------------------------------------------------------
  1 | ERROR   | [x] The PHP open tag must be followed by exactly one blank line
  9 | ERROR   | [ ] Doc comment short description must be on a single line, further text should be a separate
    |         |     paragraph
 33 | WARNING | [x] A comma should follow the last multiline array item. Found: 0
 49 | ERROR   | [x] Expected 1 space after FOREACH keyword; 0 found
 80 | ERROR   | [x] Missing function doc comment
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/Plugin/AdminToolbarContent/AdminToolbarContentCategoriesPlugin.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 6 ERRORS AND 1 WARNING AFFECTING 7 LINES
-----------------------------------------------------------------------------------------------------------------------
  1 | ERROR   | [x] The PHP open tag must be followed by exactly one blank line
  8 | ERROR   | [ ] Doc comment short description must be on a single line, further text should be a separate
    |         |     paragraph
 30 | WARNING | [ ] Line exceeds 80 characters; contains 103 characters
 32 | ERROR   | [x] Array indentation error, expected 10 spaces but found 12
 33 | ERROR   | [x] Array indentation error, expected 10 spaces but found 12
 34 | ERROR   | [x] Array indentation error, expected 10 spaces but found 12
 35 | ERROR   | [x] Array closing indentation error, expected 8 spaces but found 10
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/Plugin/AdminToolbarContent/AdminToolbarContentWebformsPlugin.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------------------------
 1 | ERROR   | [x] The PHP open tag must be followed by exactly one blank line
 7 | WARNING | [ ] Line exceeds 80 characters; contains 82 characters
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/Plugin/AdminToolbarContent/AdminToolbarContentMenusPlugin.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------
 10 | ERROR | Doc comment short description must be on a single line, further text should be a separate paragraph
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/Plugin/AdminToolbarContent/AdminToolbarContentContentPlugin.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 3 WARNINGS AFFECTING 4 LINES
-----------------------------------------------------------------------------------------------------------------------
  1 | ERROR   | [x] The PHP open tag must be followed by exactly one blank line
  6 | WARNING | [x] Unused use statement
 34 | WARNING | [x] A comma should follow the last multiline array item. Found: 10
 58 | WARNING | [x] A comma should follow the last multiline array item. Found: 0
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/Annotation/AdminToolbarContentPlugin.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
-----------------------------------------------------------------------------------------------------------------------
 10 | ERROR | [x] Doc comment long description must end with a full stop
 31 | ERROR | [ ] The @var tag must be the first tag in a member variable comment
 40 | ERROR | [ ] The @var tag must be the first tag in a member variable comment
 45 | ERROR | [x] The closing brace for the class must have an empty line before it
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/AdminToolbarContentPluginBase.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 97 ERRORS AND 19 WARNINGS AFFECTING 109 LINES
-----------------------------------------------------------------------------------------------------------------------
  10 | WARNING | [x] Unused use statement
  57 | ERROR   | [ ] Class property $base_menu_link_plugin_definition should use lowerCamel naming without underscores
  62 | ERROR   | [x] Do not append variable name "$languageManager" to the type declaration in a member variable
     |         |     comment
  94 | ERROR   | [ ] Missing short description in doc comment
 106 | ERROR   | [ ] Missing short description in doc comment
 114 | ERROR   | [ ] Missing parameter comment
 115 | ERROR   | [ ] Missing parameter comment
 116 | ERROR   | [ ] Missing parameter comment
 117 | ERROR   | [ ] Missing parameter comment
 118 | ERROR   | [ ] Missing parameter comment
 119 | ERROR   | [ ] Missing parameter comment
 120 | ERROR   | [ ] Missing parameter comment
 121 | ERROR   | [ ] Missing parameter comment
 122 | ERROR   | [ ] Missing parameter comment
 123 | ERROR   | [ ] Missing parameter comment
 124 | ERROR   | [ ] Missing parameter comment
 126 | ERROR   | [x] Expected 1 space between type hint and argument "$configuration"; 22 found
 127 | ERROR   | [x] Expected 1 space between type hint and argument "$plugin_id"; 21 found
 128 | ERROR   | [x] Expected 1 space between type hint and argument "$plugin_definition"; 22 found
 129 | ERROR   | [x] Expected 1 space between type hint and argument "$languageManager"; 3 found
 130 | ERROR   | [x] Expected 1 space between type hint and argument "$access_manager"; 5 found
 131 | ERROR   | [x] Expected 1 space between type hint and argument "$current_user"; 6 found
 133 | ERROR   | [x] Expected 1 space between type hint and argument "$route_provider"; 5 found
 134 | ERROR   | [x] Expected 1 space between type hint and argument "$module_handler"; 5 found
 135 | ERROR   | [x] Expected 1 space between type hint and argument "$config_factory"; 5 found
 136 | ERROR   | [x] Expected 1 space between type hint and argument "$entity_repository"; 2 found
 181 | WARNING | [x] A comma should follow the last multiline array item. Found: 0
 237 | ERROR   | [ ] Description for the @return value is missing
 245 | WARNING | [ ] Line exceeds 80 characters; contains 81 characters
 252 | ERROR   | [x] Expected newline after closing brace
 252 | ERROR   | [x] Expected 1 space before "|"; 0 found
 252 | ERROR   | [x] Expected 1 space after "|"; 0 found
 264 | ERROR   | [ ] Missing parameter comment
 265 | ERROR   | [ ] Missing parameter comment
 267 | ERROR   | [ ] Description for the @return value is missing
 281 | WARNING | [x] A comma should follow the last multiline array item. Found: )
 302 | ERROR   | [x] Doc comment short description must end with a full stop
 326 | ERROR   | [ ] Missing parameter comment
 327 | ERROR   | [ ] Missing parameter comment
 329 | ERROR   | [ ] Description for the @return value is missing
 371 | ERROR   | [ ] Description for the @return value is missing
 381 | ERROR   | [x] Expected newline after closing brace
 381 | ERROR   | [x] Expected 1 space before "|"; 0 found
 381 | ERROR   | [x] Expected 1 space after "|"; 0 found
 408 | ERROR   | [ ] Description for the @return value is missing
 431 | ERROR   | [ ] Missing parameter comment
 432 | ERROR   | [ ] Missing parameter comment
 433 | ERROR   | [ ] Missing parameter comment
 435 | ERROR   | [ ] Description for the @return value is missing
 456 | ERROR   | [ ] Parameter $route_name is not described in comment
 456 | ERROR   | [ ] Parameter $route_parameters is not described in comment
 463 | ERROR   | [x] Doc comment long description must end with a full stop
 466 | ERROR   | [x] Parameter comment indentation must be 3 spaces, found 2 spaces
 468 | ERROR   | [ ] Description for the @return value is missing
 473 | ERROR   | [x] Array indentation error, expected 6 spaces but found 8
 474 | ERROR   | [x] Array closing indentation error, expected 4 spaces but found 6
 501 | ERROR   | [ ] Missing parameter comment
 501 | ERROR   | [ ] Missing parameter type
 503 | ERROR   | [ ] Description for the @return value is missing
 512 | ERROR   | [ ] Missing parameter comment
 513 | ERROR   | [ ] Missing parameter comment
 515 | ERROR   | [ ] Description for the @return value is missing
 531 | ERROR   | [x] Parameter comment indentation must be 3 spaces, found 2 spaces
 532 | ERROR   | [ ] Missing parameter comment
 533 | ERROR   | [ ] Missing parameter comment
 535 | ERROR   | [ ] Description for the @return value is missing
 543 | ERROR   | [x] Space after closing parenthesis of function call prohibited
 543 | ERROR   | [x] Space found before semicolon; expected ");" but found ") ;"
 545 | ERROR   | [x] Expected 1 space after "="; 2 found
 557 | ERROR   | [x] No space found before comment text; expected "// 'title' => $this->getItemTitle($item)," but
     |         |     found "//'title' => $this->getItemTitle($item),"
 566 | WARNING | [x] A comma should follow the last multiline array item. Found: )
 567 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
 568 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
 581 | ERROR   | [ ] Missing parameter comment
 582 | ERROR   | [ ] Missing parameter comment
 584 | ERROR   | [ ] Description for the @return value is missing
 599 | ERROR   | [x] Doc comment long description must start with a capital letter
 602 | ERROR   | [x] Parameter comment indentation must be 3 spaces, found 2 spaces
 603 | ERROR   | [ ] Missing parameter comment
 604 | ERROR   | [ ] Missing parameter comment
 606 | ERROR   | [ ] Description for the @return value is missing
 621 | WARNING | [x] A comma should follow the last multiline array item. Found: )
 634 | WARNING | [x] A comma should follow the last multiline array item. Found: )
 635 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
 636 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
 647 | ERROR   | [ ] Parameter $route_name is not described in comment
 650 | ERROR   | [ ] Missing parameter comment
 651 | ERROR   | [ ] Missing parameter comment
 653 | ERROR   | [ ] Description for the @return value is missing
 671 | ERROR   | [ ] Missing short description in doc comment
 672 | ERROR   | [ ] Missing parameter comment
 673 | ERROR   | [ ] Missing parameter comment
 674 | ERROR   | [ ] Missing parameter comment
 675 | ERROR   | [ ] Missing parameter comment
 676 | ERROR   | [ ] Missing parameter comment
 678 | ERROR   | [ ] Description for the @return value is missing
 689 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing
     |         |     parentheses
 714 | WARNING | [x] A comma should follow the last multiline array item. Found: )
 715 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
 716 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
 733 | WARNING | [ ] There must be no blank line following an inline comment
 737 | ERROR   | [x] Array indentation error, expected 8 spaces but found 10
 738 | ERROR   | [x] Array closing indentation error, expected 6 spaces but found 8
 754 | WARNING | [x] A comma should follow the last multiline array item. Found: )
 764 | ERROR   | [ ] Missing short description in doc comment
 765 | ERROR   | [ ] Missing parameter comment
 766 | ERROR   | [ ] Missing parameter comment
 767 | ERROR   | [ ] Missing parameter comment
 768 | ERROR   | [ ] Missing parameter comment
 769 | ERROR   | [ ] Missing parameter comment
 770 | ERROR   | [ ] Missing parameter comment
 772 | ERROR   | [ ] Description for the @return value is missing
 776 | ERROR   | [x] Data types in @var tags need to be fully namespaced
 789 | WARNING | [x] A comma should follow the last multiline array item. Found: )
 790 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
 791 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 50 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------


FILE: .../web/modules/contrib/admin_toolbar_content/src/AdminToolbarContentPluginManager.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 9 ERRORS AFFECTING 9 LINES
-----------------------------------------------------------------------------------------------------------------------
  30 | ERROR | [ ] Class property $menu_link_plugin_manager should use lowerCamel naming without underscores
  39 | ERROR | [x] Parameter comment must end with a full stop
  41 | ERROR | [x] Parameter comment must end with a full stop
  66 | ERROR | [x] There should be no white space before a closing ")"
  82 | ERROR | [x] Expected newline after closing brace
 105 | ERROR | [x] Expected newline after closing brace
 124 | ERROR | [x] Expected newline after closing brace
 128 | ERROR | [x] Expected 1 blank line after function; 0 found
 129 | ERROR | [x] The closing brace for the class must have an empty line before it
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 8 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------

Steps to reproduce

Run phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

Fixed

Version

2.0

Component

Code

Created by

🇧🇪Belgium tim-diels Belgium 🇧🇪

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

Merge Requests

Comments & Activities

Production build 0.71.5 2024