Severity minor, Drupal Commenting Standards, Internationalization, Drupal Security Checks, Drupal SQL Standards, Drupal Coding Standards sites/all/modules/pareview_temp/test_candidate/./includes/pages/swiftmailer_admin_default.inc: +65: [critical] Potential problem: drupal_set_message() only accepts filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. Status Messages: Coder found 9 projects, 9 files, 1 critical warnings, 0 warnings were flagged to be ignored FILE: ...pace/drupal-7/sites/all/modules/pareview_temp/test_candidate/README.txt -------------------------------------------------------------------------------- FOUND 0 ERROR(S) AND 1 WARNING(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 36 | WARNING | Line exceeds 80 characters; contains 86 characters -------------------------------------------------------------------------------- FILE: ...al-7/sites/all/modules/pareview_temp/test_candidate/css/swiftmailer.css -------------------------------------------------------------------------------- FOUND 2 ERROR(S) AFFECTING 2 LINE(S) -------------------------------------------------------------------------------- 3 | ERROR | Multiple CSS properties should be listed in alphabetical order 8 | ERROR | Multiple CSS properties should be listed in alphabetical order -------------------------------------------------------------------------------- FILE: ...dules/pareview_temp/test_candidate/includes/classes/SWIFTMailSystem.inc -------------------------------------------------------------------------------- FOUND 13 ERROR(S) AND 2 WARNING(S) AFFECTING 15 LINE(S) -------------------------------------------------------------------------------- 13 | ERROR | The @see reference should not contain any additional text 77 | ERROR | The @see reference should not contain any additional text 100 | ERROR | "require_once" is a statement not a function; no parentheses | | are required 132 | ERROR | BREAK statements must be followed by a single blank line 135 | ERROR | BREAK statements must be followed by a single blank line 138 | ERROR | BREAK statements must be followed by a single blank line 141 | ERROR | BREAK statements must be followed by a single blank line 144 | ERROR | BREAK statements must be followed by a single blank line 212 | ERROR | Blank lines are not allowed after CASE statements 242 | ERROR | Blank lines are not allowed after CASE statements 253 | ERROR | Blank lines are not allowed after CASE statements 279 | ERROR | Missing parameter type at position 2 282 | WARNING | The use of private methods or properties is strongly | | discouraged, use "protected" instead 311 | ERROR | Missing parameter type at position 2 315 | WARNING | The use of private methods or properties is strongly | | discouraged, use "protected" instead -------------------------------------------------------------------------------- FILE: ...all/modules/pareview_temp/test_candidate/includes/helpers/utilities.inc -------------------------------------------------------------------------------- FOUND 1 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 122 | ERROR | Missing parameter type at position 1 -------------------------------------------------------------------------------- FILE: ...review_temp/test_candidate/includes/pages/swiftmailer_admin_default.inc -------------------------------------------------------------------------------- FOUND 0 ERROR(S) AND 1 WARNING(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 65 | WARNING | Avoid backslash escaping in translatable strings when possible, | | use "" quotes instead -------------------------------------------------------------------------------- FILE: .../pareview_temp/test_candidate/includes/pages/swiftmailer_admin_test.inc -------------------------------------------------------------------------------- FOUND 1 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 69 | ERROR | Expected 1 space between double arrow and "$form_state"; 2 found -------------------------------------------------------------------------------- FILE: ...view_temp/test_candidate/includes/pages/swiftmailer_admin_transport.inc -------------------------------------------------------------------------------- FOUND 3 ERROR(S) AND 1 WARNING(S) AFFECTING 4 LINE(S) -------------------------------------------------------------------------------- 57 | ERROR | If the line declaring an array spans longer than 80 | | characters, each element should be broken into its own line 158 | WARNING | A comma should follow the last multiline array item. Found: | | '

' 243 | ERROR | BREAK statements must be followed by a single blank line 248 | ERROR | BREAK statements must be followed by a single blank line -------------------------------------------------------------------------------- FILE: ...modules/pareview_temp/test_candidate/includes/theme/swiftmailer.tpl.php -------------------------------------------------------------------------------- FOUND 1 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 5 | ERROR | Missing file doc comment -------------------------------------------------------------------------------- FILE: ...rupal-7/sites/all/modules/pareview_temp/test_candidate/swiftmailer.info -------------------------------------------------------------------------------- FOUND 1 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 6 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...pal-7/sites/all/modules/pareview_temp/test_candidate/swiftmailer.module -------------------------------------------------------------------------------- FOUND 1 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 120 | ERROR | SWITCH statements must contain at least one CASE statement --------------------------------------------------------------------------------