Problem/Motivation

FILE: /home/thallesvf/Desktop/lando/d7-contrib/sites/all/modules/imce/inc/imce.admin.inc
-------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 74 ERRORS AND 6 WARNINGS AFFECTING 42 LINES
-------------------------------------------------------------------------------------------------------------------------------------------------
  15 | ERROR   | [ ] If the line declaring an array spans longer than 80 characters, each element should be broken into its own line
  26 | ERROR   | [ ] If the line declaring an array spans longer than 80 characters, each element should be broken into its own line
  26 | ERROR   | [ ] If the line declaring an array spans longer than 80 characters, each element should be broken into its own line
  39 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
  57 | ERROR   | [x] No space found before comment text; expected "// roles profiles" but found "//roles profiles"
  57 | ERROR   | [x] Inline comments must start with a capital letter
  57 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
  67 | ERROR   | [x] No space found before comment text; expected "// common settings" but found "//common settings"
  67 | ERROR   | [x] Inline comments must start with a capital letter
  67 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
  79 | WARNING | [ ] Avoid backslash escaping in translatable strings when possible, use "" quotes instead
  94 | WARNING | [x] A comma should follow the last multiline array item. Found: )
 103 | WARNING | [x] A comma should follow the last multiline array item. Found: )
 134 | ERROR   | [x] No space found before comment text; expected "// add each stream wrapper as a column" but found "//add each stream wrapper
     |         |     as a column"
 134 | ERROR   | [x] Inline comments must start with a capital letter
 134 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 142 | ERROR   | [x] No space found before comment text; expected "// in case we need profile weights." but found "//in case we need profile
     |         |     weights."
 142 | ERROR   | [x] Inline comments must start with a capital letter
 200 | ERROR   | [x] No space found before comment text; expected "// delete" but found "//delete"
 200 | ERROR   | [x] Inline comments must start with a capital letter
 200 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 205 | ERROR   | [x] No space found before comment text; expected "// add-edit" but found "//add-edit"
 205 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 226 | ERROR   | [x] No space found before comment text; expected "// import profile" but found "//import profile"
 226 | ERROR   | [x] Inline comments must start with a capital letter
 226 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 231 | ERROR   | [x] Comments may not appear after statements
 231 | ERROR   | [x] No space found before comment text; expected "// preserve the original name." but found "//preserve the original name."
 231 | ERROR   | [x] Inline comments must start with a capital letter
 235 | WARNING | [ ] There must be no blank line following an inline comment
 235 | ERROR   | [x] Comments may not appear after statements
 235 | ERROR   | [x] No space found before comment text; expected "// store the original profile just in case." but found "//store the original
     |         |     profile just in case."
 235 | ERROR   | [x] Inline comments must start with a capital letter
 295 | ERROR   | [x] No space found before comment text; expected "// Directories" but found "//Directories"
 295 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 302 | ERROR   | [x] Expected 1 space before "+"; 0 found
 302 | ERROR   | [x] Expected 1 space after "+"; 0 found
 304 | ERROR   | [x] No space found before comment text; expected "// Thumbnails" but found "//Thumbnails"
 304 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 311 | ERROR   | [x] Expected 1 space before "+"; 0 found
 311 | ERROR   | [x] Expected 1 space after "+"; 0 found
 328 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 332 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 348 | ERROR   | [x] No space found before comment text; expected "// unset empty fields of directories and thumbnails." but found "//unset
     |         |     empty fields of directories and thumbnails."
 348 | ERROR   | [x] Inline comments must start with a capital letter
 351 | ERROR   | [x] No space found before comment text; expected "// save profile." but found "//save profile."
 351 | ERROR   | [x] Inline comments must start with a capital letter
 359 | ERROR   | [x] Doc comment short description must start with a capital letter
 359 | ERROR   | [x] Doc comment short description must end with a full stop
 363 | ERROR   | [ ] If the line declaring an array spans longer than 80 characters, each element should be broken into its own line
 413 | ERROR   | [x] No space found before comment text; expected "// directory path" but found "//directory path"
 413 | ERROR   | [x] Inline comments must start with a capital letter
 413 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 417 | ERROR   | [x] No space found before comment text; expected "// permissions" but found "//permissions"
 417 | ERROR   | [x] Inline comments must start with a capital letter
 417 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 450 | ERROR   | [x] Doc comment short description must start with a capital letter
 450 | ERROR   | [x] Doc comment short description must end with a full stop
 454 | ERROR   | [ ] If the line declaring an array spans longer than 80 characters, each element should be broken into its own line
 512 | ERROR   | [x] Doc comment short description must end with a full stop
 540 | ERROR   | [x] Doc comment short description must end with a full stop
 559 | ERROR   | [x] Doc comment short description must end with a full stop
 592 | WARNING | [ ] Do not concatenate strings to translatable strings, they should be part of the t() argument and you should use placeholders
 625 | ERROR   | [x] No space found before comment text; expected "// add or update" but found "//add or update"
 625 | ERROR   | [x] Inline comments must start with a capital letter
 625 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 627 | ERROR   | [x] Expected 1 space before "+"; 0 found
 627 | ERROR   | [x] Expected 1 space after "+"; 0 found
 631 | ERROR   | [x] No space found before comment text; expected "// delete" but found "//delete"
 631 | ERROR   | [x] Inline comments must start with a capital letter
 631 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 634 | ERROR   | [x] Expected 1 space before "+"; 0 found
 634 | ERROR   | [x] Expected 1 space after "+"; 0 found
 635 | ERROR   | [x] Expected 1 space before "-"; 0 found
 635 | ERROR   | [x] Expected 1 space after "-"; 0 found
 702 | WARNING | [x] A comma should follow the last multiline array item. Found: 0
 718 | ERROR   | [x] Expected 1 space before "-"; 0 found
 718 | ERROR   | [x] Expected 1 space after "-"; 0 found
 746 | ERROR   | [x] No space found before comment text; expected "// Include core profile functions." but found "//Include core profile
     |         |     functions."
 747 | ERROR   | [x] Expected 1 newline at end of file; 0 found
-------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 72 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------------

Comments

thalles created an issue. See original summary.

thalles’s picture

Status: Needs work » Needs review
StatusFileSize
new402 bytes

Follow the patch!

  • thalles committed 20657a5 on 7.x-1.x
    Issue #3062528 by thalles: Clear code on inc/imce.admin.inc
    
thalles’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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