----------------------------------------------------------------------
FOUND 2 ERRORS AND 2 WARNINGS AFFECTING 4 LINES
----------------------------------------------------------------------
  9 | WARNING | [ ] Line exceeds 80 characters; contains 84
    |         |     characters
 34 | ERROR   | [x] Inline control structures are not allowed
 50 | ERROR   | [x] Expected 1 space after "?"; 2 found
 82 | WARNING | [ ] Line exceeds 80 characters; contains 90
    |         |     characters
----------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------


----------------------------------------------------------------------
FOUND 44 ERRORS AND 15 WARNINGS AFFECTING 55 LINES
----------------------------------------------------------------------
  10 | ERROR   | [ ] Doc comment short description must be on a
     |         |     single line, further text should be a separate
     |         |     paragraph
  20 | ERROR   | [x] There should be no white space after an opening
     |         |     "("
  20 | ERROR   | [x] There should be no white space before a closing
     |         |     ")"
  39 | ERROR   | [ ] Missing parameter type
  41 | ERROR   | [ ] Missing parameter type
  43 | ERROR   | [ ] Missing parameter type
  44 | WARNING | [ ] Line exceeds 80 characters; contains 90
     |         |     characters
  45 | ERROR   | [ ] Missing parameter type
  47 | ERROR   | [ ] Missing parameter type
  48 | WARNING | [ ] Line exceeds 80 characters; contains 81
     |         |     characters
 112 | ERROR   | [ ] Missing parameter type
 115 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
 131 | ERROR   | [ ] Missing parameter type
 134 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
 148 | ERROR   | [ ] Missing parameter type
 149 | WARNING | [ ] Line exceeds 80 characters; contains 86
     |         |     characters
 151 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
 152 | WARNING | [ ] Line exceeds 80 characters; contains 85
     |         |     characters
 176 | ERROR   | [ ] Missing parameter type
 179 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
 195 | WARNING | [ ] Line exceeds 80 characters; contains 96
     |         |     characters
 212 | WARNING | [ ] Line exceeds 80 characters; contains 113
     |         |     characters
 218 | WARNING | [ ] Line exceeds 80 characters; contains 88
     |         |     characters
 232 | WARNING | [ ] Line exceeds 80 characters; contains 81
     |         |     characters
 233 | WARNING | [ ] Line exceeds 80 characters; contains 90
     |         |     characters
 233 | ERROR   | [ ] Doc comment short description must be on a
     |         |     single line, further text should be a separate
     |         |     paragraph
 235 | ERROR   | [ ] Missing parameter type
 237 | ERROR   | [ ] Missing parameter type
 239 | ERROR   | [ ] Missing parameter type
 240 | WARNING | [ ] Line exceeds 80 characters; contains 90
     |         |     characters
 241 | ERROR   | [ ] Missing parameter type
 244 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
 283 | ERROR   | [ ] Missing parameter type
 285 | ERROR   | [ ] Missing parameter type
 287 | ERROR   | [ ] Missing parameter type
 289 | ERROR   | [x] Separate the @param and @return sections by a
     |         |     blank line.
 289 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
 299 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
 317 | ERROR   | [ ] Missing parameter type
 319 | ERROR   | [ ] Missing parameter type
 320 | WARNING | [ ] Line exceeds 80 characters; contains 90
     |         |     characters
 321 | ERROR   | [ ] Missing parameter type
 322 | WARNING | [ ] Line exceeds 80 characters; contains 106
     |         |     characters
 323 | ERROR   | [ ] Missing parameter type
 325 | ERROR   | [ ] Missing parameter type
 328 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
 348 | ERROR   | [ ] If the line declaring an array spans longer than
     |         |     80 characters, each element should be broken
     |         |     into its own line
 363 | ERROR   | [ ] Missing parameter type
 365 | ERROR   | [ ] Missing parameter type
 366 | WARNING | [ ] Line exceeds 80 characters; contains 90
     |         |     characters
 367 | ERROR   | [ ] Missing parameter type
 368 | WARNING | [ ] Line exceeds 80 characters; contains 106
     |         |     characters
 369 | ERROR   | [ ] Missing parameter type
 371 | ERROR   | [ ] Missing parameter type
 374 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
 391 | ERROR   | [x] Expected 1 space after comma in function call; 2
     |         |     found
 391 | ERROR   | [x] Expected one space after the comma, 2 found
 394 | ERROR   | [ ] If the line declaring an array spans longer than
     |         |     80 characters, each element should be broken
     |         |     into its own line
 415 | WARNING | [ ] Line exceeds 80 characters; contains 109
     |         |     characters
----------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------


----------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
----------------------------------------------------------------------
  8 | ERROR | It's only necessary to declare files[] if they declare
    |       | a class or interface.
  9 | ERROR | It's only necessary to declare files[] if they declare
    |       | a class or interface.
 10 | ERROR | It's only necessary to declare files[] if they declare
    |       | a class or interface.
 11 | ERROR | It's only necessary to declare files[] if they declare
    |       | a class or interface.
----------------------------------------------------------------------


----------------------------------------------------------------------
FOUND 7 ERRORS AND 5 WARNINGS AFFECTING 11 LINES
----------------------------------------------------------------------
  15 | WARNING | [ ] Line exceeds 80 characters; contains 112
     |         |     characters
  17 | WARNING | [ ] Line exceeds 80 characters; contains 120
     |         |     characters
  27 | ERROR   | [x] Array closing indentation error, expected 8
     |         |     spaces but found 10
  39 | ERROR   | [x] Array closing indentation error, expected 8
     |         |     spaces but found 10
  44 | ERROR   | [x] Inline comments must start with a capital letter
  44 | ERROR   | [x] Inline comments must end in full-stops,
     |         |     exclamation marks, colons, question marks, or
     |         |     closing parentheses
  52 | WARNING | [ ] Line exceeds 80 characters; contains 92
     |         |     characters
  63 | ERROR   | [ ] The $text argument to l() should be enclosed
     |         |     within t() so that it is translatable
 201 | WARNING | [ ] Line exceeds 80 characters; contains 105
     |         |     characters
 217 | WARNING | [ ] Line exceeds 80 characters; contains 106
     |         |     characters
 220 | ERROR   | [ ] If the line declaring an array spans longer than
     |         |     80 characters, each element should be broken
     |         |     into its own line
 221 | ERROR   | [ ] If the line declaring an array spans longer than
     |         |     80 characters, each element should be broken
     |         |     into its own line
----------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------


----------------------------------------------------------------------
FOUND 16 ERRORS AND 22 WARNINGS AFFECTING 35 LINES
----------------------------------------------------------------------
  10 | ERROR   | [ ] Doc comment short description must be on a
     |         |     single line, further text should be a separate
     |         |     paragraph
  13 | WARNING | [ ] Line exceeds 80 characters; contains 81
     |         |     characters
  29 | WARNING | [ ] Line exceeds 80 characters; contains 86
     |         |     characters
  30 | WARNING | [ ] Line exceeds 80 characters; contains 85
     |         |     characters
  31 | WARNING | [ ] Line exceeds 80 characters; contains 84
     |         |     characters
  32 | WARNING | [ ] Line exceeds 80 characters; contains 88
     |         |     characters
  42 | WARNING | [ ] Hook implementations should not duplicate
     |         |     @return documentation
  42 | ERROR   | [ ] Return type "an array with elements" must not
     |         |     contain spaces
  46 | WARNING | [ ] Line exceeds 80 characters; contains 91
     |         |     characters
  68 | ERROR   | [ ] Missing parameter type
  71 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
  89 | WARNING | [ ] Line exceeds 80 characters; contains 82
     |         |     characters
  91 | ERROR   | [ ] Missing parameter type
  94 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
 152 | WARNING | [ ] Line exceeds 80 characters; contains 115
     |         |     characters
 182 | WARNING | [ ] Line exceeds 80 characters; contains 94
     |         |     characters
 196 | WARNING | [ ] Line exceeds 80 characters; contains 84
     |         |     characters
 217 | ERROR   | [x] Case breaking statements must be followed by a
     |         |     single blank line
 240 | WARNING | [ ] Line exceeds 80 characters; contains 118
     |         |     characters
 456 | ERROR   | [ ] The $text argument to l() should be enclosed
     |         |     within t() so that it is translatable
 500 | WARNING | [ ] Hook implementations should not duplicate @param
     |         |     documentation
 500 | ERROR   | [ ] Missing parameter type
 503 | WARNING | [ ] Hook implementations should not duplicate
     |         |     @return documentation
 503 | ERROR   | [x] Expected "breadcrumbs" but found "$breadcrumbs"
     |         |     for function return type
 504 | WARNING | [ ] Line exceeds 80 characters; contains 83
     |         |     characters
 534 | WARNING | [ ] Line exceeds 80 characters; contains 91
     |         |     characters
 536 | ERROR   | [ ] Missing parameter type
 539 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
 540 | WARNING | [ ] Line exceeds 80 characters; contains 84
     |         |     characters
 544 | WARNING | [ ] Line exceeds 80 characters; contains 84
     |         |     characters
 545 | WARNING | [ ] Line exceeds 80 characters; contains 98
     |         |     characters
 556 | ERROR   | [ ] custom_breadcrumbs_taxonomy_enable() is an
     |         |     installation hook and must be declared in an
     |         |     install file
 557 | WARNING | [ ] Line exceeds 80 characters; contains 107
     |         |     characters
 565 | ERROR   | [x] Array closing indentation error, expected 6
     |         |     spaces but found 8
 575 | ERROR   | [x] Trailing punctuation for @see references is not
     |         |     allowed.
 578 | ERROR   | [ ] If the line declaring an array spans longer than
     |         |     80 characters, each element should be broken
     |         |     into its own line
 589 | WARNING | [ ] Line exceeds 80 characters; contains 105
     |         |     characters
 590 | WARNING | [ ] Line exceeds 80 characters; contains 111
     |         |     characters
----------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------




----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 8 | ERROR | It's only necessary to declare files[] if they declare a
   |       | class or interface.
----------------------------------------------------------------------


----------------------------------------------------------------------
FOUND 5 ERRORS AND 1 WARNING AFFECTING 5 LINES
----------------------------------------------------------------------
  11 | WARNING | [ ] Hook implementations should not duplicate
     |         |     @return documentation
  11 | ERROR   | [ ] Return type missing for @return tag in function
     |         |     comment
  23 | ERROR   | [x] Expected 1 space after "="; 2 found
  66 | ERROR   | [x] Case breaking statements must be followed by a
     |         |     single blank line
  75 | ERROR   | [x] Case breaking statements must be followed by a
     |         |     single blank line
 130 | ERROR   | [x] Line indented incorrectly; expected 4 spaces,
     |         |     found 6
----------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------
CommentFileSizeAuthor
#2 fix_pareview-2896972-2.patch36.71 KBrenatog
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RenatoG created an issue. See original summary.

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
FileSize
fix_pareview-2896972-2.patch36.71 KB

Patch with fixes.

  • RenatoG committed 1b826b5 on 7.x-2.x 
    Issue #2896972 by RenatoG: Fix pareview recommendations for Custom...
renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Active » Fixed

Fixed.

Committed to dev branch.

Good Work and Good Week.

Regards.

Status: Fixed » Closed (fixed)

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