Problem/Motivation

FILE: ...web/modules/contrib/gin_toolbar_custom_menu/gin_toolbar_custom_menu.module
--------------------------------------------------------------------------------
FOUND 8 ERRORS AFFECTING 6 LINES
--------------------------------------------------------------------------------
3 | ERROR | [ ] Missing short description in doc comment
27 | ERROR | [x] Spaces must be used to indent lines; tabs are not allowed
27 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 3
28 | ERROR | [x] Spaces must be used to indent lines; tabs are not allowed
28 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 3
68 | ERROR | [ ] The array declaration extends to column 91 (the limit is 80).
| | The array content should be split up over multiple lines
76 | ERROR | [x] Expected 1 space after IF keyword; 0 found
79 | ERROR | [x] Whitespace found at end of line
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: ...ontrib/gin_toolbar_custom_menu/src/Render/Element/GinToolbarCustomMenu.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
12 | WARNING | The class short comment should describe what the class does and
| | not simply repeat the class name
--------------------------------------------------------------------------------

FILE: .../contrib/gin_toolbar_custom_menu/src/Form/GinToolbarCustomMenuSettings.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 5 LINES
--------------------------------------------------------------------------------
45 | WARNING | Menu::loadMultiple calls should be avoided in classes, use
| | dependency injection instead
65 | WARNING | Do not concatenate strings to translatable strings, they should
| | be part of the t() argument and you should use placeholders
84 | WARNING | \Drupal calls should be avoided in classes, use dependency
| | injection instead
89 | WARNING | Do not concatenate strings to translatable strings, they should
| | be part of the t() argument and you should use placeholders
93 | WARNING | \Drupal calls should be avoided in classes, use dependency
| | injection instead
--------------------------------------------------------------------------------

FILE: ...odules/contrib/gin_toolbar_custom_menu/gin_toolbar_custom_menu.routing.yml
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
7 | ERROR | [x] Expected 1 newline at end of file; 0 found
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: ...es/contrib/gin_toolbar_custom_menu/gin_toolbar_custom_menu.permissions.yml
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
2 | ERROR | [x] Expected 1 newline at end of file; 0 found
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

Time: 543ms; Memory: 12MB

Steps to reproduce

Execute the command: phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,js,info,txt,md,yml,twig gin_toolbar_custom_menu/

Proposed resolution

Resolve all issues for Drupal and DrupalPractice coding standards.

Remaining tasks

Patch review and fix remaining issues

FILE: .../contrib/gin_toolbar_custom_menu/src/Form/GinToolbarCustomMenuSettings.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
45 | WARNING | Menu::loadMultiple calls should be avoided in classes, use
| | dependency injection instead
84 | WARNING | \Drupal calls should be avoided in classes, use dependency
| | injection instead
93 | WARNING | \Drupal calls should be avoided in classes, use dependency
| | injection instead
--------------------------------------------------------------------------------

Time: 552ms; Memory: 12MB

CommentFileSizeAuthor
coding-standard-fixes.patch4.74 KBurvashi_vora

Comments

urvashi_vora created an issue. See original summary.

jmauricio’s picture

Changes commited to 1.0.x
Please review

falsovsky’s picture

Status: Needs review » Reviewed & tested by the community

Looks OK

jmauricio’s picture

Assigned: Unassigned » jmauricio
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

jmauricio’s picture

Assigned: jmauricio » Unassigned