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: +92: [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 2 ERROR(S) AFFECTING 2 LINE(S) -------------------------------------------------------------------------------- 22 | ERROR | Selectors must be on a single line 24 | ERROR | Selectors must be on a single line -------------------------------------------------------------------------------- FILE: ...l/modules/pareview_temp/test_candidate/includes/jqgrid_views_plugin.inc -------------------------------------------------------------------------------- FOUND 11 ERROR(S) AFFECTING 10 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 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: ...e/drupal-7/sites/all/modules/pareview_temp/test_candidate/jqgrid.module -------------------------------------------------------------------------------- FOUND 5 ERROR(S) AFFECTING 3 LINE(S) -------------------------------------------------------------------------------- 179 | 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 --------------------------------------------------------------------------------