Phpcs Drupal coding standard issue

Created on 13 February 2023, over 1 year ago
Updated 29 March 2023, over 1 year ago

web/modules/contrib/entities_info/src/EntitiesInfoManager.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------------
100 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
100 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------

web/modules/contrib/entities_info/src/Controller/EntitiesInfoExportController.php
------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
------------------------------------------------------------------------------------------------------------------
72 | ERROR | Missing short description in doc comment
73 | ERROR | Missing parameter comment
74 | ERROR | Missing parameter comment
75 | ERROR | Description for the @return value is missing
------------------------------------------------------------------------------------------------------------------

web/modules/contrib/entities_info/src/EntitiesInfoGenerateTables.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 8 WARNINGS AFFECTING 8 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
14 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
20 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
66 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
67 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
68 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
69 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
70 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
71 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-----------------------------------------------------------------------------------------------------------------------------------------

web/modules/contrib/entities_info/src/EntitiesInfoManagerInterface.php
-------------------------------------------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 5 LINES
-------------------------------------------------------------------------------------------------------
12 | ERROR | Missing short description in doc comment
13 | ERROR | Description for the @return value is missing
17 | ERROR | Missing short description in doc comment
18 | ERROR | Missing parameter comment
19 | ERROR | Description for the @return value is missing
-------------------------------------------------------------------------------------------------------

web/modules/contrib/entities_info/src/Form/EntitiesInfoExportForm.php
-----------------------------------------------------------------------------------------------------------------------------------
FOUND 6 ERRORS AND 7 WARNINGS AFFECTING 8 LINES
-----------------------------------------------------------------------------------------------------------------------------------
73 | WARNING | [x] A comma should follow the last multiline array item. Found: $entity_id
74 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
84 | WARNING | [x] A comma should follow the last multiline array item. Found: TRUE
85 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
86 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
87 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
97 | ERROR | [x] No space found before comment text; expected "// select" but found "//select"
97 | ERROR | [x] Inline comments must start with a capital letter
97 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
97 | ERROR | [x] Comments may not appear after statements
134 | WARNING | [ ] There must be no blank line following an inline comment
134 | ERROR | [x] No space found before comment text; expected "// $results['table_type']" but found "//$results['table_type']"
134 | ERROR | [x] Comments may not appear after statements
-----------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 12 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------------

web/modules/contrib/entities_info/entities_info.routing.yml
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------
7 | WARNING | Open page callback found, please add a comment before the line why there is no access restriction
8 | WARNING | The administration page callback should probably use "administer site configuration" - which implies the user can change something - rather than "access
| | administration pages" which is about viewing but not changing configurations.
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------

🐛 Bug report
Status

Fixed

Version

1.0

Component

Code

Created by

🇮🇳India himanshu_jhaloya Mandsaur

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

Comments & Activities

Production build 0.71.5 2024