Severity minor, Drupal Commenting Standards, Internationalization, Drupal Security Checks, Drupal SQL Standards, Drupal Coding Standards sites/all/modules/pareview_temp/test_candidate/./includes/jqgrid.callbacks.inc: +66: [minor] in most cases, replace the string function with the drupal_ equivalent string functions Status Messages: Coder found 7 projects, 7 files, 1 minor warnings, 0 warnings were flagged to be ignored FILE: .../drupal-7/sites/all/modules/pareview_temp/test_candidate/css/jqgrid.css -------------------------------------------------------------------------------- FOUND 11 ERROR(S) AFFECTING 7 LINE(S) -------------------------------------------------------------------------------- 3 | ERROR | CSS colours must be defined in lowercase; expected #cecece but | | found #CECECE 16 | ERROR | CSS colours must be defined in lowercase; expected #ddd but found | | #DDD 21 | ERROR | Multiple selectors should each be on a single line 21 | ERROR | Whitespace found at end of line 21 | ERROR | Selectors must be on a single line 22 | ERROR | Multiple selectors should each be on a single line 22 | ERROR | Whitespace found at end of line 22 | ERROR | Selectors must be on a single line 23 | ERROR | Multiple selectors should each be on a single line 25 | ERROR | Expected 1 space before opening brace of class definition; 0 | | found 26 | ERROR | CSS colours must be defined in lowercase; expected #ddd but found | | #DDD -------------------------------------------------------------------------------- FILE: .../all/modules/pareview_temp/test_candidate/includes/jqgrid.callbacks.inc -------------------------------------------------------------------------------- FOUND 5 ERROR(S) AFFECTING 5 LINE(S) -------------------------------------------------------------------------------- 52 | ERROR | BREAK statements must be followed by a single blank line 64 | ERROR | BREAK statements must be followed by a single blank line 76 | ERROR | BREAK statements must be followed by a single blank line 77 | ERROR | Empty CASE statements are not allowed 79 | ERROR | BREAK statements must be followed by a single blank line -------------------------------------------------------------------------------- FILE: ...sites/all/modules/pareview_temp/test_candidate/includes/jqgrid.json.inc -------------------------------------------------------------------------------- FOUND 7 ERROR(S) AFFECTING 6 LINE(S) -------------------------------------------------------------------------------- 59 | ERROR | No space found after comma in function call 64 | ERROR | No space before comment text; expected "// $view->jqgrid_ajax = | | TRUE;" but found "//$view->jqgrid_ajax = TRUE;" 64 | ERROR | There must be no blank line following an inline comment 128 | ERROR | Expected a valid @param data type, but found unknown_type 155 | ERROR | Whitespace found at end of line 156 | ERROR | Expected a valid @param data type, but found unknown_type 160 | ERROR | A unary operator statement must not be followed by a space -------------------------------------------------------------------------------- FILE: ...ites/all/modules/pareview_temp/test_candidate/includes/jqgrid.views.inc -------------------------------------------------------------------------------- FOUND 4 ERROR(S) AFFECTING 4 LINE(S) -------------------------------------------------------------------------------- 9 | ERROR | Expected 3 space(s) before asterisk; 1 found 10 | ERROR | Expected 3 space(s) before asterisk; 1 found 152 | ERROR | Whitespace found at end of line 158 | ERROR | Whitespace found at end of line -------------------------------------------------------------------------------- FILE: ...l/modules/pareview_temp/test_candidate/includes/jqgrid_views_plugin.inc -------------------------------------------------------------------------------- FOUND 13 ERROR(S) AND 1 WARNING(S) AFFECTING 13 LINE(S) -------------------------------------------------------------------------------- 11 | ERROR | Class name must begin with a capital letter 11 | ERROR | Class name must use UpperCamel naming without underscores 12 | ERROR | Whitespace found at end of line 32 | ERROR | Method name "jqgrid_views_plugin::option_definition" is not in | | lowerCamel format, it must not contain underscores 62 | ERROR | Missing parameter type at position 1 63 | ERROR | Parameter comment indentation must be 2 additional spaces at | | position 1 65 | ERROR | Missing parameter type at position 2 66 | ERROR | Parameter comment indentation must be 2 additional spaces at | | position 2 71 | ERROR | Method name "jqgrid_views_plugin::sanitize_columns" is not in | | lowerCamel format, it must not contain underscores 96 | ERROR | Method name "jqgrid_views_plugin::options_form" is not in | | lowerCamel format, it must not contain underscores 99 | WARNING | A comma should follow the last multiline array item. Found: | | '/js/jqgrid.admin.js' 133 | ERROR | Whitespace found at end of line 217 | ERROR | If the line declaring an array spans longer than 80 | | characters, each element should be broken into its own line 225 | ERROR | There must be a single space before an operator statement -------------------------------------------------------------------------------- FILE: .../drupal-7/sites/all/modules/pareview_temp/test_candidate/jqgrid.install -------------------------------------------------------------------------------- FOUND 2 ERROR(S) AND 1 WARNING(S) AFFECTING 2 LINE(S) -------------------------------------------------------------------------------- 25 | WARNING | A comma should follow the last multiline array item. Found: ) 78 | ERROR | Array indentation error, expected 4 spaces but found 6 78 | ERROR | Array closing indentation error, expected 4 spaces but found 6 -------------------------------------------------------------------------------- FILE: ...e/drupal-7/sites/all/modules/pareview_temp/test_candidate/jqgrid.module -------------------------------------------------------------------------------- FOUND 16 ERROR(S) AFFECTING 12 LINE(S) -------------------------------------------------------------------------------- 65 | ERROR | Array closing indentation error, expected 2 spaces but found 4 138 | ERROR | Whitespace found at end of line 139 | ERROR | Doc comment for var object does not match actual variable name | | $account at position 1 139 | ERROR | Parameter comment must be on the next line at position 1 156 | ERROR | There should be no white space after an opening "(" 165 | ERROR | Array closing indentation error, expected 4 spaces but found 6 181 | ERROR | Whitespace found at end of line 182 | ERROR | Doc comment for var $uid does not match actual variable name | | $conditions at position 1 182 | ERROR | Missing comment for param "$uid" at position 1 183 | ERROR | Doc comment for var $view_name does not match actual variable | | name $setting at position 2 183 | ERROR | Missing comment for param "$view_name" at position 2 184 | ERROR | Superfluous doc comment at position 3 184 | ERROR | Missing comment for param "$setting" at position 3 199 | ERROR | Expected "else {\n"; found "else{\n" 206 | ERROR | Whitespace found at end of line 207 | ERROR | Parameter comment must be on the next line at position 1 -------------------------------------------------------------------------------- FILE: ...ce/drupal-7/sites/all/modules/pareview_temp/test_candidate/js/jqgrid.js -------------------------------------------------------------------------------- FOUND 8 ERROR(S) AFFECTING 8 LINE(S) -------------------------------------------------------------------------------- 35 | ERROR | Inline control structures are not allowed 36 | ERROR | Inline control structures are not allowed 41 | ERROR | Whitespace found at end of line 54 | ERROR | Whitespace found at end of line 58 | ERROR | Whitespace found at end of line 72 | ERROR | Whitespace found at end of line 73 | ERROR | Expected 1 space after second semicolon of FOR loop; 0 found 92 | ERROR | Whitespace found at end of line -------------------------------------------------------------------------------- FILE: ...drupal-7/sites/all/modules/pareview_temp/test_candidate/theme/theme.inc -------------------------------------------------------------------------------- FOUND 1 ERROR(S) AND 1 WARNING(S) AFFECTING 2 LINE(S) -------------------------------------------------------------------------------- 42 | WARNING | Line exceeds 80 characters; contains 95 characters 136 | ERROR | There must be no blank line following an inline comment --------------------------------------------------------------------------------