Phpcs Drupal coding standard issue

Created on 10 February 2023, almost 2 years ago
Updated 12 April 2023, over 1 year ago

FILE: /var/www/html/contribution/web/modules/contrib/spreadsheets/src/SpreadSheets.php
-----------------------------------------------------------------------------------------------------------
FOUND 46 ERRORS AND 8 WARNINGS AFFECTING 36 LINES
-----------------------------------------------------------------------------------------------------------
5 | ERROR | [x] Non-namespaced classes/interfaces/traits should not be referenced with use statements
6 | ERROR | [x] Non-namespaced classes/interfaces/traits should not be referenced with use statements
8 | ERROR | [x] Non-namespaced classes/interfaces/traits should not be referenced with use statements
17 | ERROR | [x] Additional blank lines found at end of doc comment
23 | ERROR | [x] Data types in @var tags need to be fully namespaced
32 | ERROR | [ ] Class property $sheet_id should use lowerCamel naming without underscores
52 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
65 | ERROR | [x] Data types in @throws tags need to be fully namespaced
68 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
70 | WARNING | [ ] File::load calls should be avoided in classes, use dependency injection instead
71 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
76 | WARNING | [x] A comma should follow the last multiline array item. Found: SPREADSHEETS
95 | WARNING | [ ] Unused variable $range.
96 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
97 | ERROR | [x] Expected 1 space after IF keyword; 0 found
101 | ERROR | [x] Expected 1 space after IF keyword; 0 found
115 | WARNING | [x] A comma should follow the last multiline array item. Found: $format
118 | ERROR | [x] Expected one space after the comma, 0 found
119 | ERROR | [x] Expected 1 space after IF keyword; 0 found
124 | ERROR | [x] Expected one space after the comma, 0 found
124 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
163 | ERROR | [x] Doc comment short description must end with a full stop
166 | ERROR | [x] Parameter comment must end with a full stop
170 | ERROR | [x] Expected 1 space after FUNCTION keyword; 0 found
170 | ERROR | [x] Expected 1 space before opening brace; found 0
170 | ERROR | [x] Expected 1 space before opening brace; found 0
170 | ERROR | [x] Opening brace must be the last content on the line
170 | ERROR | [x] Closing brace must be on a line by itself
170 | ERROR | [x] Each PHP statement must be on a line by itself
172 | ERROR | [x] Expected 1 space before "="; 0 found
172 | ERROR | [x] Expected 1 space after "="; 0 found
172 | ERROR | [x] Whitespace found at end of line
173 | ERROR | [x] Whitespace found at end of line
179 | ERROR | [ ] Parameter $setheader is not described in comment
183 | ERROR | [x] Parameter comment must end with a full stop
184 | ERROR | [ ] Missing parameter name
184 | ERROR | [x] Whitespace found at end of line
187 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
190 | ERROR | [x] Expected 1 space after FUNCTION keyword; 0 found
190 | ERROR | [x] Expected 1 space before opening brace; found 0
190 | ERROR | [x] Expected 1 space before opening brace; found 0
190 | ERROR | [x] Opening brace must be the last content on the line
190 | ERROR | [x] Closing brace must be on a line by itself
190 | ERROR | [x] Each PHP statement must be on a line by itself
191 | ERROR | [x] Expected 1 space after IF keyword; 0 found
192 | ERROR | [x] Expected 1 space before "="; 0 found
192 | ERROR | [x] Expected 1 space after "="; 0 found
192 | ERROR | [x] Whitespace found at end of line
193 | ERROR | [x] Whitespace found at end of line
198 | ERROR | [x] Expected 1 space before "="; 0 found
198 | ERROR | [x] Expected 1 space after "="; 0 found
198 | ERROR | [x] Whitespace found at end of line
199 | ERROR | [x] Whitespace found at end of line
205 | ERROR | [x] Whitespace found at end of line
-----------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 45 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/spreadsheets/src/Form/SpreadsheetsCredentials.php
----------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AND 5 WARNINGS AFFECTING 5 LINES
----------------------------------------------------------------------------------------------------------
7 | WARNING | [x] Unused use statement
7 | ERROR | [x] Non-namespaced classes/interfaces/traits should not be referenced with use statements
8 | WARNING | [x] Unused use statement
8 | ERROR | [x] Non-namespaced classes/interfaces/traits should not be referenced with use statements
9 | WARNING | [x] Unused use statement
9 | ERROR | [x] Non-namespaced classes/interfaces/traits should not be referenced with use statements
37 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
74 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
----------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/spreadsheets/src/Form/GoogleSheetsConfigForm.php
-----------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
-----------------------------------------------------------------------------------------------------
7 | WARNING | [x] Unused use statement
71 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
76 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
-----------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/spreadsheets/README.md
---------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------
3 | WARNING | Line exceeds 80 characters; contains 92 characters
---------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/spreadsheets/spreadsheets.module
-------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------
3 | ERROR | Missing short description in doc comment
-------------------------------------------------------------------------------------

📌 Task
Status

Fixed

Version

2.0

Component

Code

Created by

🇮🇳India himanshu_jhaloya Indore

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

Comments & Activities

Production build 0.71.5 2024