Revision of https://git.drupal.org/project/amp_carousel.git from Tue, 07/09/2019 - 21:01

Revisions allow you to track differences between multiple versions of your content, and revert back to older versions.

Review of the 7.x-1.x branch (commit a2ed067):
  • Remove "version" from the ./amp_carousel.info file, it will be added by drupal.org packaging automatically.
  • Remove "project" from the ./amp_carousel.info file, it will be added by drupal.org packaging automatically.
  • Remove "datestamp" from the ./amp_carousel.info file, it will be added by drupal.org packaging automatically.
  • Coder Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment.
  • No automated test cases were found, did you consider writing Simpletests or PHPUnit tests? This is not a requirement but encouraged for professional software development.
This automated report was generated with PAReview.sh, your friendly project application review script.

FILE: ...ite1101/web/vendor/drupal/pareviewsh/pareview_temp/amp_carousel.info
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 1 LINE
--------------------------------------------------------------------------
 1 | WARNING | Remove "project" from the info file, it will be added by
   |         | drupal.org packaging automatically
 1 | WARNING | Remove "version" from the info file, it will be added by
   |         | drupal.org packaging automatically
--------------------------------------------------------------------------


FILE: ...rupal/pareviewsh/pareview_temp/theme/views-view-amp-carousel.tpl.php
--------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 3 LINES
--------------------------------------------------------------------------
 1 | ERROR | [x] The PHP open tag must be followed by exactly one blank
   |       |     line
 3 | ERROR | [x] Doc comment short description must be on the first line
 3 | ERROR | [x] Doc comment short description must start with a capital
   |       |     letter
 4 | ERROR | [ ] Doc comment short description must be on a single line,
   |       |     further text should be a separate paragraph
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...00000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/README.txt
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------
 5 | WARNING | Line exceeds 80 characters; contains 84 characters
--------------------------------------------------------------------------


FILE: ...01/web/vendor/drupal/pareviewsh/pareview_temp/amp_carousel.views.inc
--------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------
 10 | ERROR | [x] There must be no blank lines after the function comment
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...e1101/web/vendor/drupal/pareviewsh/pareview_temp/amp_carousel.module
--------------------------------------------------------------------------
FOUND 6 ERRORS AND 2 WARNINGS AFFECTING 8 LINES
--------------------------------------------------------------------------
  72 | WARNING | [ ] Line exceeds 80 characters; contains 86 characters
 145 | ERROR   | [ ] Concatenating translatable strings is not allowed,
     |         |     use placeholders instead and only one string literal
 146 | ERROR   | [x] Whitespace found at end of line
 147 | ERROR   | [x] Whitespace found at end of line
 148 | ERROR   | [x] Whitespace found at end of line
 149 | WARNING | [x] A comma should follow the last multiline array item.
     |         |     Found: ]
 244 | ERROR   | [x] Whitespace found at end of line
 289 | ERROR   | [x] Whitespace found at end of line
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...vendor/drupal/pareviewsh/pareview_temp/amp_carousel_style_plugin.inc
--------------------------------------------------------------------------
FOUND 92 ERRORS AFFECTING 35 LINES
--------------------------------------------------------------------------
 11 | ERROR | [ ] Class name must begin with a capital letter
 11 | ERROR | [ ] Class name must use UpperCamel naming without
    |       |     underscores
 16 | ERROR | [ ] Public method name
    |       |     "amp_carousel_style_plugin::option_definition" is not
    |       |     in lowerCamel format
 32 | ERROR | [ ] Public method name
    |       |     "amp_carousel_style_plugin::options_form" is not in
    |       |     lowerCamel format
 36 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 36 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 36 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 37 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 37 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 37 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 38 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 38 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 38 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 39 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 39 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 39 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 42 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 42 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 42 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 43 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 43 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 43 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 44 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 44 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 44 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 45 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 45 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 45 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 48 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 48 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 48 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 49 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 49 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 49 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 50 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 50 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 50 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 51 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 51 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 51 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 52 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 52 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 52 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 53 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 53 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 53 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 56 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 56 | ERROR | [x] Array indentation error, expected 6 spaces but found 5
 57 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 57 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 57 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 58 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 58 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 58 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 59 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 59 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 59 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 60 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 60 | ERROR | [x] Array indentation error, expected 5 spaces but found 4
 61 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 61 | ERROR | [x] Array indentation error, expected 6 spaces but found 4
 62 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 63 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 66 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 66 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 66 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 67 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 67 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 67 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 68 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 68 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 68 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 69 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 69 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 69 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 72 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 72 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 72 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 73 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 73 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 73 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 74 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 74 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 74 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 75 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 75 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces,
    |       |     found 3
 75 | ERROR | [x] Array indentation error, expected 6 spaces but found 3
 77 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 77 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 1
 77 | ERROR | [x] Closing brace indented incorrectly; expected 2 spaces,
    |       |     found 1
 78 | ERROR | [x] Spaces must be used to indent lines; tabs are not
    |       |     allowed
 78 | ERROR | [x] Whitespace found at end of line
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 88 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------

Time: 850ms; Memory: 6Mb