Drupal Coding Standards Issues | phpcs

Created on 7 February 2023, over 1 year ago
Updated 23 April 2024, 2 months ago

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/css/editoria11y-admin.css
------------------------------------------------------------------------------------------
FOUND 25 ERRORS AFFECTING 22 LINES
------------------------------------------------------------------------------------------
1 | ERROR | [x] Additional whitespace found at start of file
3 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
4 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
5 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
8 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
8 | ERROR | [x] Multiple selectors should each be on a single line
9 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 8
10 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 8
11 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 8
12 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 6
13 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 6
13 | ERROR | [x] Multiple selectors should each be on a single line
14 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 8
15 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 8
16 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 6
21 | ERROR | [x] Expected 1 space before opening brace of class definition; 2 found
22 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
23 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
24 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
26 | ERROR | [x] Multiple selectors should each be on a single line
27 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
30 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
33 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
33 | ERROR | [x] Expected 1 space after colon in style definition; 0 found
34 | ERROR | [x] Expected 1 newline at end of file; 0 found
------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 25 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/editoria11y.links.menu.yml
-------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------
32 | ERROR | [x] Expected 1 newline at end of file; 2 found
-------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/editoria11y.services.yml
-----------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------------
12 | ERROR | [x] Expected 1 newline at end of file; 2 found
-----------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/editoria11y.info.yml
-------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------
1 | WARNING | Remove "project" from the info file, it will be added by drupal.org packaging automatically
1 | WARNING | Remove "datestamp" from the info file, it will be added by drupal.org packaging automatically
1 | WARNING | Remove "version" from the info file, it will be added by drupal.org packaging automatically
-------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/editoria11y.install
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 19 ERRORS AND 15 WARNINGS AFFECTING 34 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
10 | WARNING | [x] Unused use statement
29 | ERROR | [x] Doc comment short description must end with a full stop
32 | ERROR | [x] Opening brace should be on the same line as the declaration
47 | WARNING | [x] A comma should follow the last multiline array item. Found: 'The entity type; "route" if no type found.'
54 | WARNING | [x] A comma should follow the last multiline array item. Found: 'Route name for page..'
61 | WARNING | [x] A comma should follow the last multiline array item. Found: 'Internal, relative page path.'
68 | WARNING | [x] A comma should follow the last multiline array item. Found: 'Active translation.'
75 | WARNING | [x] A comma should follow the last multiline array item. Found: 'Raw URL.'
82 | WARNING | [x] A comma should follow the last multiline array item. Found: 'The name of the route where this was last seen.'
137 | ERROR | [x] Doc comment short description must end with a full stop
140 | ERROR | [x] Opening brace should be on the same line as the declaration
162 | WARNING | [x] A comma should follow the last multiline array item. Found: 'Code sample to identify the flagged element.'
169 | WARNING | [x] A comma should follow the last multiline array item. Found: 'The entity type of the route where this was dismissed'
176 | WARNING | [x] A comma should follow the last multiline array item. Found: 'The name of the route where this was last seen.'
183 | WARNING | [x] A comma should follow the last multiline array item. Found: 'Active translation.'
190 | WARNING | [x] A comma should follow the last multiline array item. Found: 'The name of the route where this was last seen.'
197 | WARNING | [x] A comma should follow the last multiline array item. Found: 'The entity type of the route where this was dismissed'
238 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
267 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
276 | ERROR | [x] Opening brace should be on the same line as the declaration
280 | ERROR | [x] Expected 1 blank line after function; 2 found
285 | ERROR | [x] Whitespace found at end of line
286 | ERROR | [x] Whitespace found at end of line
287 | ERROR | [x] Whitespace found at end of line
291 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
300 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
304 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
307 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
310 | ERROR | [x] Line indented incorrectly; expected 3 spaces, found 2
318 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 2
325 | ERROR | [ ] Doc comment short description must be on a single line, further text should be a separate paragraph
327 | ERROR | [x] Expected 1 blank line before function; 2 found
334 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
345 | ERROR | [x] Whitespace found at end of line
-----------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 33 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/src/Dashboard.php
---------------------------------------------------------------------------------------------------------------------------------------------
FOUND 43 ERRORS AND 18 WARNINGS AFFECTING 49 LINES
---------------------------------------------------------------------------------------------------------------------------------------------
16 | ERROR | Missing member variable doc comment
18 | ERROR | Doc comment is empty
28 | ERROR | Description for the @return value is missing
42 | ERROR | Description for the @return value is missing
58 | ERROR | Description for the @return value is missing
74 | ERROR | Description for the @return value is missing
86 | ERROR | Parameter $range is not described in comment
86 | ERROR | Parameter $form_title is not described in comment
86 | ERROR | Parameter $form_url is not described in comment
86 | ERROR | Parameter $form_filters is not described in comment
89 | ERROR | Missing parameter comment
90 | ERROR | Missing parameter comment
92 | ERROR | Description for the @return value is missing
95 | WARNING | Unused variable $type.
95 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
96 | WARNING | Unused variable $lang.
96 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
100 | ERROR | The array declaration extends to column 104 (the limit is 80). The array content should be split up over multiple lines
102 | WARNING | Line exceeds 80 characters; contains 85 characters
137 | ERROR | Doc comment is empty
141 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
143 | ERROR | The array declaration extends to column 107 (the limit is 80). The array content should be split up over multiple lines
159 | ERROR | Parameter $range is not described in comment
159 | ERROR | Parameter $form_title is not described in comment
159 | ERROR | Parameter $form_url is not described in comment
159 | ERROR | Parameter $form_filters is not described in comment
162 | ERROR | Missing parameter comment
163 | ERROR | Missing parameter comment
165 | ERROR | Description for the @return value is missing
168 | WARNING | Unused variable $type.
168 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
169 | WARNING | Unused variable $lang.
169 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
173 | ERROR | The array declaration extends to column 130 (the limit is 80). The array content should be split up over multiple lines
175 | WARNING | Line exceeds 80 characters; contains 85 characters
215 | ERROR | Description for the @return value is missing
224 | ERROR | The array declaration extends to column 86 (the limit is 80). The array content should be split up over multiple lines
241 | ERROR | Description for the @return value is missing
246 | ERROR | The array declaration extends to column 101 (the limit is 80). The array content should be split up over multiple lines
256 | ERROR | Description for the @return value is missing
279 | ERROR | Description for the @return value is missing
283 | ERROR | The array declaration extends to column 121 (the limit is 80). The array content should be split up over multiple lines
295 | ERROR | Description for the @return value is missing
304 | ERROR | The array declaration extends to column 144 (the limit is 80). The array content should be split up over multiple lines
317 | WARNING | Line exceeds 80 characters; contains 86 characters
336 | ERROR | Doc comment is empty
339 | ERROR | Public method name "Dashboard::get_page_lang" is not in lowerCamel format
340 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
345 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
353 | ERROR | Description for the @return value is missing
355 | ERROR | Public method name "Dashboard::get_language_filters" is not in lowerCamel format
356 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
360 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
360 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
360 | WARNING | Do not concatenate strings to translatable strings, they should be part of the t() argument and you should use placeholders
402 | ERROR | Description for the @return value is missing
407 | ERROR | The array declaration extends to column 104 (the limit is 80). The array content should be split up over multiple lines
426 | ERROR | Description for the @return value is missing
431 | ERROR | The array declaration extends to column 98 (the limit is 80). The array content should be split up over multiple lines
443 | ERROR | Description for the @return value is missing
449 | ERROR | The array declaration extends to column 144 (the limit is 80). The array content should be split up over multiple lines
---------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/src/Controller/ExportCSVController.php
---------------------------------------------------------------------------------------------------------------------------------------------
FOUND 7 ERRORS AND 15 WARNINGS AFFECTING 19 LINES
---------------------------------------------------------------------------------------------------------------------------------------------
14 | WARNING | Line exceeds 80 characters; contains 81 characters
14 | ERROR | Doc comment short description must end with a full stop
14 | ERROR | Doc comment short description must be on a single line, further text should be a separate paragraph
20 | ERROR | Missing short description in doc comment
25 | ERROR | Missing short description in doc comment
26 | ERROR | Missing parameter comment
26 | ERROR | Missing parameter type
45 | WARNING | Line exceeds 80 characters; contains 84 characters
48 | WARNING | Line exceeds 80 characters; contains 82 characters
86 | WARNING | Line exceeds 80 characters; contains 81 characters
97 | WARNING | Do not concatenate strings to translatable strings, they should be part of the t() argument and you should use placeholders
109 | WARNING | Line exceeds 80 characters; contains 84 characters
112 | WARNING | Line exceeds 80 characters; contains 82 characters
120 | ERROR | No key specified for array entry; first entry specifies key
150 | WARNING | Line exceeds 80 characters; contains 81 characters
161 | WARNING | Do not concatenate strings to translatable strings, they should be part of the t() argument and you should use placeholders
173 | WARNING | Line exceeds 80 characters; contains 84 characters
176 | WARNING | Line exceeds 80 characters; contains 82 characters
196 | WARNING | User::load calls should be avoided in classes, use dependency injection instead
199 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
220 | WARNING | Line exceeds 80 characters; contains 81 characters
231 | WARNING | Do not concatenate strings to translatable strings, they should be part of the t() argument and you should use placeholders
---------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/src/Controller/DashboardController.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 38 ERRORS AND 45 WARNINGS AFFECTING 78 LINES
------------------------------------------------------------------------------------------------------------------------------------------
31 | ERROR | Missing short description in doc comment
36 | ERROR | Missing short description in doc comment
37 | ERROR | Missing parameter comment
37 | ERROR | Missing parameter type
55 | ERROR | Private method name "DashboardController::sync_status" is not in lowerCamel format
56 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
60 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
61 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
74 | ERROR | Description for the @return value is missing
76 | ERROR | Public method name "DashboardController::get_test_results" is not in lowerCamel format
77 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
84 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
95 | ERROR | The array declaration extends to column 94 (the limit is 80). The array content should be split up over multiple lines
113 | ERROR | The array declaration extends to column 92 (the limit is 80). The array content should be split up over multiple lines
146 | ERROR | Description for the @return value is missing
148 | ERROR | Public method name "DashboardController::get_recent_issues" is not in lowerCamel format
149 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
156 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
166 | ERROR | The array declaration extends to column 82 (the limit is 80). The array content should be split up over multiple lines
186 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
191 | ERROR | The array declaration extends to column 116 (the limit is 80). The array content should be split up over multiple lines
224 | ERROR | Description for the @return value is missing
226 | ERROR | Public method name "DashboardController::get_issues_by_page" is not in lowerCamel format
228 | ERROR | The array declaration extends to column 89 (the limit is 80). The array content should be split up over multiple lines
234 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
251 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
252 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
258 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
258 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
295 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
314 | ERROR | Description for the @return value is missing
316 | ERROR | Public method name "DashboardController::get_pages_by_issue" is not in lowerCamel format
318 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
323 | ERROR | The array declaration extends to column 89 (the limit is 80). The array content should be split up over multiple lines
329 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
351 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
351 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
375 | ERROR | Doc comment is empty
378 | ERROR | Private method name "DashboardController::get_top_pages" is not in lowerCamel format
380 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
381 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
412 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
447 | ERROR | Description for the @return value is missing
449 | ERROR | Private method name "DashboardController::get_recent_dismissals" is not in lowerCamel format
451 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
455 | WARNING | Line exceeds 80 characters; contains 136 characters
457 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
458 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
459 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
460 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
461 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
467 | WARNING | User::load calls should be avoided in classes, use dependency injection instead
477 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
481 | ERROR | The array declaration extends to column 86 (the limit is 80). The array content should be split up over multiple lines
492 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
522 | ERROR | Description for the @return value is missing
524 | ERROR | Private method name "DashboardController::get_top_issues" is not in lowerCamel format
525 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
530 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
541 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
546 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
547 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
553 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
553 | WARNING | Only string literals should be passed to t() where possible
605 | ERROR | Description for the @return value is missing
607 | ERROR | Public method name "DashboardController::get_dismissals" is not in lowerCamel format
609 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
616 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
626 | WARNING | Unused variable $debugger.
632 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
640 | WARNING | Unused variable $counter.
642 | WARNING | User::load calls should be avoided in classes, use dependency injection instead
657 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
660 | ERROR | The array declaration extends to column 134 (the limit is 80). The array content should be split up over multiple lines
660 | ERROR | The array declaration extends to column 118 (the limit is 80). The array content should be split up over multiple lines
667 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
703 | ERROR | Public method name "DashboardController::get_export_links" is not in lowerCamel format
797 | ERROR | Public method name "DashboardController::all_issues" is not in lowerCamel format
809 | ERROR | Missing short description in doc comment
812 | ERROR | Public method name "DashboardController::pages_by_issue" is not in lowerCamel format
824 | ERROR | Missing short description in doc comment
827 | ERROR | Public method name "DashboardController::issues_by_page" is not in lowerCamel format
839 | ERROR | Missing short description in doc comment
------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/src/Controller/ApiController.php
-------------------------------------------------------------------------------------------------
FOUND 12 ERRORS AFFECTING 12 LINES
-------------------------------------------------------------------------------------------------
11 | ERROR | Doc comment is empty
16 | ERROR | Missing short description in doc comment
24 | ERROR | Missing parameter type
40 | ERROR | Doc comment is empty
54 | ERROR | Doc comment is empty
69 | ERROR | Doc comment is empty
83 | ERROR | Doc comment is empty
97 | ERROR | Doc comment is empty
100 | ERROR | Public method name "ApiController::purge_page" is not in lowerCamel format
111 | ERROR | Doc comment is empty
114 | ERROR | Public method name "ApiController::purge_dismissals" is not in lowerCamel format
125 | ERROR | Doc comment is empty
-------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/src/Api.php
---------------------------------------------------------------------------------------------------------------------------------------
FOUND 20 ERRORS AFFECTING 20 LINES
---------------------------------------------------------------------------------------------------------------------------------------
30 | ERROR | Missing member variable doc comment
32 | ERROR | Parameter $manager is not described in comment
50 | ERROR | Doc comment is empty
53 | ERROR | Public method name "Api::test_results" is not in lowerCamel format
147 | ERROR | Doc comment is empty
150 | ERROR | Public method name "Api::purge_page" is not in lowerCamel format
160 | ERROR | The array declaration extends to column 135 (the limit is 80). The array content should be split up over multiple lines
163 | ERROR | Doc comment is empty
166 | ERROR | Public method name "Api::purge_dismissal" is not in lowerCamel format
177 | ERROR | The array declaration extends to column 135 (the limit is 80). The array content should be split up over multiple lines
180 | ERROR | Doc comment is empty
249 | ERROR | The array declaration extends to column 140 (the limit is 80). The array content should be split up over multiple lines
252 | ERROR | Doc comment is empty
255 | ERROR | Private method name "Api::validate_not_null" is not in lowerCamel format
261 | ERROR | Doc comment is empty
264 | ERROR | Private method name "Api::validate_path" is not in lowerCamel format
270 | ERROR | Doc comment is empty
273 | ERROR | Private method name "Api::validate_dismissal_status" is not in lowerCamel format
279 | ERROR | Doc comment is empty
282 | ERROR | Private method name "Api::validate_number" is not in lowerCamel format
---------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/src/DismissalsOnPage.php
--------------------------------------------------------------------------------------------------------------------------------------
FOUND 7 ERRORS AFFECTING 6 LINES
--------------------------------------------------------------------------------------------------------------------------------------
12 | ERROR | Missing short description in doc comment
20 | ERROR | Missing parameter comment
26 | ERROR | Missing short description in doc comment
27 | ERROR | Missing parameter comment
27 | ERROR | Missing parameter type
29 | ERROR | Description for the @return value is missing
34 | ERROR | The array declaration extends to column 109 (the limit is 80). The array content should be split up over multiple lines
--------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/src/Form/DashboardFilters.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 6 WARNINGS AFFECTING 6 LINES
------------------------------------------------------------------------------------------------------------------------------------------
10 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
24 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
78 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
96 | WARNING | Line exceeds 80 characters; contains 88 characters
98 | WARNING | Line exceeds 80 characters; contains 87 characters
107 | WARNING | Line exceeds 80 characters; contains 141 characters
------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/src/Form/Editoria11ySettings.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 7 WARNINGS AFFECTING 7 LINES
------------------------------------------------------------------------------------------------------------------------------------------
12 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
37 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
39 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
47 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
81 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
114 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
149 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/src/Form/DismissalFilters.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 7 WARNINGS AFFECTING 7 LINES
------------------------------------------------------------------------------------------------------------------------------------------
25 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
45 | WARNING | User::load calls should be avoided in classes, use dependency injection instead
54 | WARNING | Line exceeds 80 characters; contains 84 characters
122 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
140 | WARNING | Line exceeds 80 characters; contains 88 characters
142 | WARNING | Line exceeds 80 characters; contains 87 characters
151 | WARNING | Line exceeds 80 characters; contains 141 characters
------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/README.md
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------
139 | WARNING | Line exceeds 80 characters; contains 110 characters
141 | WARNING | Line exceeds 80 characters; contains 121 characters
--------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/CONTRIBUTORS.md
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 7 WARNINGS AFFECTING 7 LINES
--------------------------------------------------------------------------------
2 | WARNING | Line exceeds 80 characters; contains 106 characters
5 | WARNING | Line exceeds 80 characters; contains 111 characters
6 | WARNING | Line exceeds 80 characters; contains 93 characters
9 | WARNING | Line exceeds 80 characters; contains 82 characters
18 | WARNING | Line exceeds 80 characters; contains 104 characters
27 | WARNING | Line exceeds 80 characters; contains 89 characters
29 | WARNING | Line exceeds 80 characters; contains 168 characters
--------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/editoria11y.module
-----------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 42 ERRORS AND 15 WARNINGS AFFECTING 44 LINES
-----------------------------------------------------------------------------------------------------------------------------------------------------------------
9 | WARNING | [x] Unused use statement
29 | ERROR | [x] There should be no white space after an opening "("
45 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
46 | WARNING | [x] 'todo convert to standard Bool call once schema is updated' should match the format '@todo Fix problem X here.'
46 | ERROR | [x] Inline comments must start with a capital letter
46 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
52 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
57 | ERROR | [x] Expected newline after closing brace
62 | ERROR | [x] No space found before comment text; expected "// $page_path = $request->getPathInfo();" but found "//$page_path = $request->getPathInfo();"
68 | ERROR | [x] Short array syntax must be used to define arrays
69 | ERROR | [x] Short array syntax must be used to define arrays
70 | ERROR | [x] Expected one space after the comma, 0 found
74 | WARNING | [ ] Line exceeds 80 characters; contains 140 characters
80 | ERROR | [x] Expected one space after the comma, 0 found
81 | WARNING | [ ] Line exceeds 80 characters; contains 156 characters
82 | WARNING | [ ] Line exceeds 80 characters; contains 162 characters
107 | WARNING | [x] 'todo: media?' should match the format '@todo Fix problem X here.'
108 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
113 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
115 | ERROR | [x] There should be no white space after an opening "("
118 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
119 | WARNING | [x] 'todo figure out how to get view and media nice names' should match the format '@todo Fix problem X here.'
119 | ERROR | [x] Inline comments must start with a capital letter
119 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
120 | ERROR | [x] Expected one space after the comma, 0 found
123 | ERROR | [x] Space before opening parenthesis of function call prohibited
123 | ERROR | [x] There should be no white space after an opening "("
123 | ERROR | [x] There should be no white space before a closing ")"
125 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
127 | ERROR | [x] Space before opening parenthesis of function call prohibited
127 | ERROR | [x] There should be no white space after an opening "("
127 | ERROR | [x] There should be no white space before a closing ")"
130 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
131 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
132 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
133 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
137 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
138 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
139 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
140 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
142 | ERROR | [x] Functions must not contain multiple empty lines in a row; found 2 empty lines
144 | WARNING | [x] There must be no blank line following an inline comment
144 | WARNING | [x] 'todo we could add config for additional route names to allow $show on.' should match the format '@todo Fix problem X here.'
144 | WARNING | [ ] There must be no blank line following an inline comment
144 | ERROR | [x] Inline comments must start with a capital letter
148 | WARNING | [x] There must be no blank line following an inline comment
148 | WARNING | [ ] There must be no blank line following an inline comment
153 | WARNING | [x] 'Todo: this fails on user route?' should match the format '@todo Fix problem X here.'
168 | ERROR | [x] Namespaced classes/interfaces/traits should be referenced with use statements
175 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
179 | ERROR | [x] Expected newline after closing brace
180 | ERROR | [x] Expected one space after the comma, 0 found
185 | WARNING | [x] There must be no blank line following an inline comment
185 | WARNING | [ ] There must be no blank line following an inline comment
193 | ERROR | [x] Expected newline after closing brace
202 | ERROR | [x] Expected newline after closing brace
203 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
-----------------------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 51 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/editoria11y.routing.yml
----------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------
168 | ERROR | [x] Expected 1 newline at end of file; 2 found
----------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/tests/src/Functional/InstallTest.php
-------------------------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AND 1 WARNING AFFECTING 4 LINES
-------------------------------------------------------------------------------------------------------------------------------------------
16 | ERROR | [ ] Missing member variable doc comment
26 | WARNING | [ ] Possible useless method overriding detected
45 | ERROR | [x] Missing function doc comment
46 | ERROR | [ ] The array declaration extends to column 91 (the limit is 80). The array content should be split up over multiple lines
-------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/tests/src/Traits/UserTraits.php
----------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AND 4 WARNINGS AFFECTING 5 LINES
----------------------------------------------------------------------------------------------------------
5 | WARNING | [x] Unused use statement
6 | WARNING | [x] Unused use statement
8 | ERROR | [x] Missing trait doc comment
10 | ERROR | [x] Missing function doc comment
11 | WARNING | [x] There must be no blank line following an inline comment
11 | WARNING | [ ] There must be no blank line following an inline comment
11 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 0
11 | ERROR | [x] 4 spaces found before inline comment; expected "// $user =" but found "// $user ="
----------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/editoria11y/tests/src/Kernel/SchemaTest.php
------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AND 4 WARNINGS AFFECTING 7 LINES
------------------------------------------------------------------------------------------------
6 | WARNING | [x] Unused use statement
7 | WARNING | [x] Unused use statement
8 | WARNING | [x] Unused use statement
9 | WARNING | [x] Unused use statement
37 | ERROR | [x] Expected 1 space before opening brace; found 0
43 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
44 | ERROR | [x] Expected newline after closing brace
------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------

📌 Task
Status

Fixed

Version

2.0

Component

Bugs

Created by

🇮🇳India himanshu_jhaloya Mandsaur

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

Merge Requests

Comments & Activities

Production build 0.69.0 2024