Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
----------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
----------------------------------------------------------------------
11 | ERROR | It's only necessary to declare files[] if they declare
| | a class or interface.
12 | ERROR | It's only necessary to declare files[] if they declare
| | a class or interface.
----------------------------------------------------------------------
----------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
----------------------------------------------------------------------
15 | WARNING | Line exceeds 80 characters; contains 88 characters
118 | ERROR | Doc comment short description must be on a single
| | line, further text should be a separate paragraph
----------------------------------------------------------------------
----------------------------------------------------------------------
FOUND 15 ERRORS AND 12 WARNINGS AFFECTING 22 LINES
----------------------------------------------------------------------
6 | ERROR | [x] There must be exactly one blank line after the
| | file comment
13 | WARNING | [ ] Hook implementations should not duplicate
| | @return documentation
13 | ERROR | [ ] Return type "an array with elements:" must not
| | contain spaces
15 | WARNING | [ ] Line exceeds 80 characters; contains 82
| | characters
17 | WARNING | [ ] Line exceeds 80 characters; contains 91
| | characters
33 | ERROR | [ ] Missing parameter type
36 | ERROR | [ ] Return type missing for @return tag in function
| | comment
96 | WARNING | [ ] Line exceeds 80 characters; contains 92
| | characters
116 | WARNING | [ ] Line exceeds 80 characters; contains 82
| | characters
118 | ERROR | [ ] Missing parameter type
119 | ERROR | [x] Parameter comment indentation must be 3 spaces,
| | found 4 spaces
119 | WARNING | [ ] Line exceeds 80 characters; contains 94
| | characters
120 | ERROR | [ ] Missing parameter type
121 | ERROR | [x] Parameter comment indentation must be 3 spaces,
| | found 4 spaces
121 | WARNING | [ ] Line exceeds 80 characters; contains 83
| | characters
122 | WARNING | [ ] Line exceeds 80 characters; contains 83
| | characters
135 | WARNING | [ ] Line exceeds 80 characters; contains 87
| | characters
172 | ERROR | [ ] Missing parameter type
175 | ERROR | [ ] Missing parameter type
178 | ERROR | [x] Expected "breadcrumbs" but found "$breadcrumbs"
| | for function return type
206 | ERROR | [ ] Missing parameter type
209 | ERROR | [x] Expected "page_match" but found "$page_match"
| | for function return type
210 | WARNING | [ ] Line exceeds 80 characters; contains 84
| | characters
238 | WARNING | [ ] Hook implementations should not duplicate @param
| | documentation
238 | ERROR | [ ] Missing parameter type
241 | WARNING | [ ] Hook implementations should not duplicate
| | @return documentation
241 | ERROR | [x] Expected "breadcrumbs" but found "$breadcrumbs"
| | for function return type
----------------------------------------------------------------------
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------
Comment | File | Size | Author |
---|---|---|---|
#2 | fix_code_review_items-2896572-2.patch | 7.38 KB | renatog |
Comments
Comment #2
renatogPatch with the fix.
Comment #4
renatogFixed.
Committed to dev branch.
Good Work and Good Weekend.
Regards.