Revision of https://git.drupal.org/project/spectra_connect.git from Fri, 03/15/2019 - 01:35

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

Review of the 8.x-1.x branch (commit 4b47f42):
  • The "?>" PHP delimiter at the end of files is discouraged, see https://www.drupal.org/node/318#phptags
    ./src/SpectraConnectInterface.php
    
  • Coder Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment.
  • DrupalPractice has found some issues with your code, but could be false positives.
    
    FILE: ...ndor/drupal/pareviewsh/pareview_temp/src/Form/SpectraConnectForm.php
    --------------------------------------------------------------------------
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    --------------------------------------------------------------------------
     46 | WARNING | \Drupal calls should be avoided in classes, use
        |         | dependency injection instead
    --------------------------------------------------------------------------
    
    Time: 879ms; Memory: 4Mb
    
  • No automated test cases were found, did you consider writing 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: ...ndor/drupal/pareviewsh/pareview_temp/src/SpectraConnectUtilities.php
--------------------------------------------------------------------------
FOUND 8 ERRORS AFFECTING 8 LINES
--------------------------------------------------------------------------
   9 | ERROR | [x] Doc comment short description must end with a full
     |       |     stop
  10 | ERROR | [x] There must be exactly one blank line before the tags
     |       |     in a doc comment
  31 | ERROR | [ ] Public method name
     |       |     "SpectraConnectUtilities::spectra_queue_delete" is not
     |       |     in lowerCamel format
  51 | ERROR | [ ] Public method name
     |       |     "SpectraConnectUtilities::spectra_delete" is not in
     |       |     lowerCamel format
  90 | ERROR | [ ] Public method name
     |       |     "SpectraConnectUtilities::spectra_get" is not in
     |       |     lowerCamel format
 105 | ERROR | [x] Inline comments must end in full-stops, exclamation
     |       |     marks, colons, question marks, or closing parentheses
 132 | ERROR | [ ] Public method name
     |       |     "SpectraConnectUtilities::spectra_queue_post" is not
     |       |     in lowerCamel format
 152 | ERROR | [ ] Public method name
     |       |     "SpectraConnectUtilities::spectra_post" is not in
     |       |     lowerCamel format
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...eb/vendor/drupal/pareviewsh/pareview_temp/src/SpectraConnectTest.php
--------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------
 13 | ERROR | [x] Doc comment short description must end with a full stop
 14 | ERROR | [x] There must be exactly one blank line before the tags in
    |       |     a doc comment
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...ndor/drupal/pareviewsh/pareview_temp/src/SpectraConnectInterface.php
--------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------
  9 | ERROR | [x] There must be exactly one blank line before the tags in
    |       |     a doc comment
 15 | ERROR | [x] Expected 1 newline at end of file; 0 found
 15 | ERROR | [x] A closing tag is not permitted at the end of a PHP file
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...wsh/pareview_temp/src/Plugin/QueueWorker/SpectraConnectQueuePost.php
--------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
--------------------------------------------------------------------------
  5 | WARNING | [x] Unused use statement
 17 | ERROR   | [x] Expected 1 newline at end of file; 0 found
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...review_temp/src/Plugin/QueueWorker/SpectraConnectQueueDeleteBase.php
--------------------------------------------------------------------------
FOUND 6 ERRORS AND 1 WARNING AFFECTING 5 LINES
--------------------------------------------------------------------------
  7 | WARNING | [x] Unused use statement
  9 | ERROR   | [x] There must be one blank line after the last USE
    |         |     statement; 2 found;
 12 | ERROR   | [x] Missing class doc comment
 17 | ERROR   | [x] Doc comment short description must start with a
    |         |     capital letter
 17 | ERROR   | [x] Doc comment short description must end with a full
    |         |     stop
 42 | ERROR   | [x] Expected 1 newline at end of file; 0 found
 42 | ERROR   | [x] The closing brace for the class must have an empty
    |         |     line before it
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...pareview_temp/src/Plugin/QueueWorker/SpectraConnectQueuePostBase.php
--------------------------------------------------------------------------
FOUND 6 ERRORS AND 1 WARNING AFFECTING 5 LINES
--------------------------------------------------------------------------
  7 | WARNING | [x] Unused use statement
  9 | ERROR   | [x] There must be one blank line after the last USE
    |         |     statement; 2 found;
 12 | ERROR   | [x] Missing class doc comment
 17 | ERROR   | [x] Doc comment short description must start with a
    |         |     capital letter
 17 | ERROR   | [x] Doc comment short description must end with a full
    |         |     stop
 42 | ERROR   | [x] Expected 1 newline at end of file; 0 found
 42 | ERROR   | [x] The closing brace for the class must have an empty
    |         |     line before it
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...h/pareview_temp/src/Plugin/QueueWorker/SpectraConnectQueueDelete.php
--------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
--------------------------------------------------------------------------
  5 | WARNING | [x] Unused use statement
 17 | ERROR   | [x] Expected 1 newline at end of file; 0 found
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...pareviewsh/pareview_temp/src/Controller/SpectraConnectController.php
--------------------------------------------------------------------------
FOUND 7 ERRORS AFFECTING 6 LINES
--------------------------------------------------------------------------
  1 | ERROR | [x] The PHP open tag must be followed by exactly one blank
    |       |     line
  2 | ERROR | [x] Namespaced classes, interfaces and traits should not
    |       |     begin with a file doc comment
  2 | ERROR | [ ] Missing short description in doc comment
 14 | ERROR | [x] Doc comment short description must end with a full stop
 19 | ERROR | [x] Opening brace should be on the same line as the
    |       |     declaration
 21 | ERROR | [x] Missing function doc comment
 56 | ERROR | [x] Expected 1 newline at end of file; 0 found
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...ndor/drupal/pareviewsh/pareview_temp/src/Form/SpectraConnectForm.php
--------------------------------------------------------------------------
FOUND 8 ERRORS AND 2 WARNINGS AFFECTING 10 LINES
--------------------------------------------------------------------------
   1 | ERROR   | [x] The PHP open tag must be followed by exactly one
     |         |     blank line
   2 | ERROR   | [x] Namespaced classes, interfaces and traits should not
     |         |     begin with a file doc comment
  12 | WARNING | [x] Unused use statement
  42 | ERROR   | [x] Doc comment short description must end with a full
     |         |     stop
  45 | ERROR   | [x] Short array syntax must be used to define arrays
  46 | ERROR   | [x] Expected 1 space after FOREACH keyword; 0 found
  47 | ERROR   | [x] Inline comments must start with a capital letter
  48 | ERROR   | [x] Namespaced classes/interfaces/traits should be
     |         |     referenced with use statements
  50 | ERROR   | [x] Namespaced classes/interfaces/traits should be
     |         |     referenced with use statements
 139 | WARNING | [ ] Line exceeds 80 characters; contains 91 characters
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 9 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...rupal/pareviewsh/pareview_temp/src/Form/SpectraConnectDeleteForm.php
--------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
--------------------------------------------------------------------------
  3 | ERROR | [x] Namespaced classes, interfaces and traits should not
    |       |     begin with a file doc comment
 19 | ERROR | [x] Expected 1 space before opening brace; found 2
 25 | ERROR | [x] Short array syntax must be used to define arrays
 47 | ERROR | [x] Short array syntax must be used to define arrays
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...pal/pareviewsh/pareview_temp/src/Form/SpectraConnectSettingsForm.php
--------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
--------------------------------------------------------------------------
  1 | ERROR | [x] The PHP open tag must be followed by exactly one blank
    |       |     line
  2 | ERROR | [x] Namespaced classes, interfaces and traits should not
    |       |     begin with a file doc comment
 26 | ERROR | [x] Expected 1 blank line before function; 0 found
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...1/web/vendor/drupal/pareviewsh/pareview_temp/spectra_connect.install
--------------------------------------------------------------------------
FOUND 8 ERRORS AND 2 WARNINGS AFFECTING 9 LINES
--------------------------------------------------------------------------
  1 | ERROR   | [x] The PHP open tag must be followed by exactly one
    |         |     blank line
  2 | ERROR   | [ ] Missing short description in doc comment
  9 | WARNING | [ ] Line exceeds 80 characters; contains 102 characters
 13 | ERROR   | [x] Line indented incorrectly; expected 2 spaces, found 4
 14 | ERROR   | [x] Line indented incorrectly; expected 2 spaces, found 4
 21 | ERROR   | [x] Line indented incorrectly; expected 2 spaces, found 4
 25 | WARNING | [ ] Line exceeds 80 characters; contains 89 characters
 25 | ERROR   | [x] Doc comment short description must end with a full
    |         |     stop
 28 | ERROR   | [x] Namespaced classes/interfaces/traits should be
    |         |     referenced with use statements
 30 | ERROR   | [x] Expected 1 newline at end of file; 0 found
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...endor/drupal/pareviewsh/pareview_temp/spectra_connect.links.task.yml
--------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------
 15 | ERROR | [x] Expected 1 newline at end of file; 0 found
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...01/web/vendor/drupal/pareviewsh/pareview_temp/spectra_connect.module
--------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------
 1 | ERROR | [x] The PHP open tag must be followed by exactly one blank
   |       |     line
 2 | ERROR | [ ] Missing short description in doc comment
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------

Time: 1.63 secs; Memory: 4Mb