Fix the issues reported by phpcs

Created on 11 July 2023, over 1 year ago

Please open merge requests and not patches for code you want to include: https://www.gregboggs.com/drupal-merge-requests/

phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig /var/www/html/VB/drupal-10.1.0/modules/contrib/easy_breadcrumb/

FILE: /var/www/html/VB/drupal-10.1.0/modules/contrib/easy_breadcrumb/easy_breadcrumb.module
-------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
-------------------------------------------------------------------------------------------
8 | WARNING | [x] Unused use statement
48 | ERROR | [x] Expected 1 space after IF keyword; 0 found
-------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------

FILE: /var/www/html/VB/drupal-10.1.0/modules/contrib/easy_breadcrumb/src/EasyBreadcrumbConstants.php
----------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------
145 | WARNING | Line exceeds 80 characters; contains 96 characters
----------------------------------------------------------------------------------------------------

FILE: /var/www/html/VB/drupal-10.1.0/modules/contrib/easy_breadcrumb/src/TitleResolver.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 2 WARNINGS AFFECTING 3 LINES
------------------------------------------------------------------------------------------------------------------------------------------
9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Entity\EntityTypeManagerInterface.
11 | WARNING | [x] Unused use statement
22 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/VB/drupal-10.1.0/modules/contrib/easy_breadcrumb/src/Form/EasyBreadcrumbGeneralSettingsForm.php
----------------------------------------------------------------------------------------------------------------------------------------------
FOUND 5 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
----------------------------------------------------------------------------------------------------------------------------------------------
128 | ERROR | [x] Opening brace must be the last content on the line
128 | ERROR | [x] There should be no white space after an opening "{"
128 | ERROR | [x] There should be no white space before a closing "}"
128 | ERROR | [x] Closing brace must be on a line by itself
128 | WARNING | [ ] Menu::loadMultiple calls should be avoided in classes, use dependency injection instead
128 | ERROR | [x] Each PHP statement must be on a line by itself
134 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
377 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
----------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/VB/drupal-10.1.0/modules/contrib/easy_breadcrumb/src/EasyBreadcrumbStructuredDataJsonLd.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Extension\ModuleHandlerInterface.
15 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
56 | ERROR | [x] Parameter comment must end with a full stop
-----------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/VB/drupal-10.1.0/modules/contrib/easy_breadcrumb/src/EasyBreadcrumbBuilder.php
--------------------------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AND 4 WARNINGS AFFECTING 7 LINES
--------------------------------------------------------------------------------------------------------------------------------------
33 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Messenger\MessengerInterface.
275 | ERROR | [x] Whitespace found at end of line
295 | ERROR | [x] Expected 1 space after IF keyword; 0 found
361 | WARNING | [ ] Line exceeds 80 characters; contains 83 characters
892 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
900 | WARNING | [x] There must be no blank line following an inline comment
900 | WARNING | [ ] There must be no blank line following an inline comment
901 | ERROR | [x] Whitespace found at end of line
--------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/VB/drupal-10.1.0/modules/contrib/easy_breadcrumb/tests/src/Kernel/EasyBreadcrumbBuilderTest.php
-------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------
23 | ERROR | The array declaration extends to column 83 (the limit is 80). The array content should be split up over multiple lines
-------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/VB/drupal-10.1.0/modules/contrib/easy_breadcrumb/README.md
------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------
8 | WARNING | Line exceeds 80 characters; contains 84 characters
------------------------------------------------------------------------------

Time: 983ms; Memory: 16MB

๐Ÿ“Œ Task
Status

Fixed

Version

2.0

Component

Code

Created by

๐Ÿ‡ฎ๐Ÿ‡ณIndia indrapatil Bangalore

Live updates comments and jobs are added and updated live.
  • Coding standards

    It involves compliance with, or the content of coding standards. Requires broad community agreement.

Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024