Review of the 7.x-1.x branch: This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me. FILE: ...sites/all/modules/pareview_temp/test_candidate/shopintegrator.admin.inc -------------------------------------------------------------------------------- FOUND 3 ERROR(S) AND 1 WARNING(S) AFFECTING 3 LINE(S) -------------------------------------------------------------------------------- 10 | WARNING | Format should be "* Implements hook_foo()." or "Implements | | hook_foo_BAR_ID_bar() for xyz_bar()." 10 | ERROR | Expected 2 space(s) before asterisk; 1 found 11 | ERROR | Expected 2 space(s) before asterisk; 1 found 50 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...view/sites/all/modules/pareview_temp/test_candidate/shopintegrator.info -------------------------------------------------------------------------------- FOUND 2 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 4 | ERROR | Files must end in a single new line character 4 | ERROR | It's only necessary to declare files[] if they declare a class or | | interface. -------------------------------------------------------------------------------- FILE: ...ew/sites/all/modules/pareview_temp/test_candidate/shopintegrator.module -------------------------------------------------------------------------------- FOUND 51 ERROR(S) AND 5 WARNING(S) AFFECTING 26 LINE(S) -------------------------------------------------------------------------------- 16 | ERROR | No space before comment text; expected "// TODO: Add the Shop | | Integrator logo as the URL link" but found "//TODO: Add the | | Shop Integrator logo as the URL link" 16 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 68 | ERROR | Additional blank line found at the end of doc comment 125 | ERROR | No space before comment text; expected "// Add the required | | base JS library" but found "//Add the required base JS | | library" 125 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 127 | ERROR | Whitespace found at end of line 128 | ERROR | Line indented incorrectly; expected 2 spaces, found 3 128 | ERROR | No space before comment text; expected "// The Shop Integrator | | client ID" but found "//The Shop Integrator client ID" 128 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 131 | ERROR | No space before comment text; expected "// TODO: Use Drupal | | Theme functions" but found "//TODO: Use Drupal Theme | | functions" 131 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 146 | ERROR | No space before comment text; expected "// Add the required | | base JS library" but found "//Add the required base JS | | library" 146 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 148 | ERROR | Whitespace found at end of line 149 | ERROR | No space before comment text; expected "// The Shop Integrator | | client ID" but found "//The Shop Integrator client ID" 149 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 152 | ERROR | No space before comment text; expected "// TODO: Use Drupal | | Theme functions" but found "//TODO: Use Drupal Theme | | functions" 152 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 153 | ERROR | Expected 1 space before "="; 0 found 165 | ERROR | There should be no white space after an opening "(" 165 | ERROR | No space before comment text; expected "// Machine name of the | | formatter" but found "//Machine name of the formatter" 165 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 165 | ERROR | Comments may not appear after statements. 167 | WARNING | Line exceeds 80 characters; contains 82 characters 167 | ERROR | No space before comment text; expected "// This will only be | | available to text fields" but found "//This will only be | | available to text fields" 167 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 167 | ERROR | Comments may not appear after statements. 177 | WARNING | Line exceeds 80 characters; contains 104 characters 177 | ERROR | No space before comment text; expected "// This gets the | | view_mode where our settings are stored" but found "//This | | gets the view_mode where our settings are stored" 177 | ERROR | Comments may not appear after statements. 178 | ERROR | No space before comment text; expected "// This gets the | | actual settings" but found "//This gets the actual settings" 178 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 178 | ERROR | There must be no blank line following an inline comment 178 | ERROR | Comments may not appear after statements. 183 | WARNING | Line exceeds 80 characters; contains 87 characters 183 | ERROR | Comments may not appear after statements. 184 | ERROR | Comments may not appear after statements. 185 | WARNING | Line exceeds 80 characters; contains 92 characters 185 | ERROR | Comments may not appear after statements. 186 | WARNING | Line exceeds 80 characters; contains 98 characters 186 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 186 | ERROR | Comments may not appear after statements. 209 | ERROR | Inline comments must start with a capital letter 209 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 209 | ERROR | Comments may not appear after statements. 218 | ERROR | No space before comment text; expected "// Add the required | | base JS library" but found "//Add the required base JS | | library" 218 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 222 | ERROR | Comments may not appear after statements. 222 | ERROR | Inline comments must start with a capital letter 223 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 223 | ERROR | There must be no blank line following an inline comment 223 | ERROR | Comments may not appear after statements. 229 | ERROR | No space before comment text; expected "// Generate the Shop | | Integrator Button Script" but found "//Generate the Shop | | Integrator Button Script" 229 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 232 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 235 | ERROR | Files must end in a single new line character --------------------------------------------------------------------------------
Source: http://ventral.org/pareview - PAReview.sh online service