----------------------------------------------------------------------
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
----------------------------------------------------------------------

CommentFileSizeAuthor
#2 fix_code_review_items-2896572-2.patch7.38 KBrenatog
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RenatoG created an issue. See original summary.

renatog’s picture

Patch with the fix.

  • RenatoG committed 6b0bac7 on 7.x-2.x
    Issue #2896572 by RenatoG: Fix code review items for sub Custom...
renatog’s picture

Status: Active » Fixed

Fixed.

Committed to dev branch.

Good Work and Good Weekend.

Regards.

Status: Fixed » Closed (fixed)

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