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: +91: [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 5 ERROR(S) AFFECTING 3 LINE(S) -------------------------------------------------------------------------------- 21 | ERROR | Multiple selectors should each be on a single line 21 | ERROR | Selectors must be on a single line 22 | ERROR | Multiple selectors should each be on a single line 22 | ERROR | Selectors must be on a single line 23 | ERROR | Multiple selectors should each be on a single line -------------------------------------------------------------------------------- FILE: ...sites/all/modules/pareview_temp/test_candidate/includes/jqgrid.json.inc -------------------------------------------------------------------------------- FOUND 1 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 156 | ERROR | Last parameter comment requires a blank newline after it -------------------------------------------------------------------------------- FILE: ...l/modules/pareview_temp/test_candidate/includes/jqgrid_views_plugin.inc -------------------------------------------------------------------------------- FOUND 11 ERROR(S) AND 1 WARNING(S) AFFECTING 11 LINE(S) -------------------------------------------------------------------------------- 11 | ERROR | Class name must begin with a capital letter 11 | ERROR | Class name must use UpperCamel naming without underscores 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' 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 0 ERROR(S) AND 1 WARNING(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 25 | WARNING | A comma should follow the last multiline array item. Found: ) -------------------------------------------------------------------------------- FILE: ...e/drupal-7/sites/all/modules/pareview_temp/test_candidate/jqgrid.module -------------------------------------------------------------------------------- FOUND 5 ERROR(S) AFFECTING 3 LINE(S) -------------------------------------------------------------------------------- 180 | ERROR | Missing comment for param "$conditions" at position 1 181 | ERROR | Last parameter comment requires a blank newline after it 181 | ERROR | Missing comment for param "$setting" at position 2 205 | ERROR | Whitespace found at end of line 205 | ERROR | Last parameter comment requires a blank newline after it -------------------------------------------------------------------------------- FILE: ...ce/drupal-7/sites/all/modules/pareview_temp/test_candidate/js/jqgrid.js -------------------------------------------------------------------------------- FOUND 2 ERROR(S) AFFECTING 2 LINE(S) -------------------------------------------------------------------------------- 35 | ERROR | Inline control structures are not allowed 36 | ERROR | Inline control structures are not allowed --------------------------------------------------------------------------------