Revision of https://git.drupal.org/project/cg_payment.git from Fri, 01/11/2019 - 07:11

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 e449427):
  • 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 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/cg_payment.module
--------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------
 17 | ERROR | [x] A unary operator statement must not be followed by a
    |       |     space
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/README.md
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 6 WARNINGS AFFECTING 6 LINES
--------------------------------------------------------------------------
 35 | WARNING | Line exceeds 80 characters; contains 86 characters
 36 | WARNING | Line exceeds 80 characters; contains 89 characters
 40 | WARNING | Line exceeds 80 characters; contains 88 characters
 52 | WARNING | Line exceeds 80 characters; contains 107 characters
 58 | WARNING | Line exceeds 80 characters; contains 82 characters
 59 | WARNING | Line exceeds 80 characters; contains 86 characters
--------------------------------------------------------------------------


FILE: ...endor/drupal/pareviewsh/pareview_temp/src/Manager/RequestManager.php
--------------------------------------------------------------------------
FOUND 28 ERRORS AFFECTING 25 LINES
--------------------------------------------------------------------------
  20 | ERROR | [x] Doc comment short description must end with a full
     |       |     stop
  21 | ERROR | [x] There must be exactly one blank line before the tags
     |       |     in a doc comment
  33 | ERROR | [ ] Missing short description in doc comment
  38 | ERROR | [ ] Missing short description in doc comment
  43 | ERROR | [ ] Missing short description in doc comment
  93 | ERROR | [x] Inline comments must end in full-stops, exclamation
     |       |     marks, colons, question marks, or closing parentheses
  95 | ERROR | [x] Inline comments must end in full-stops, exclamation
     |       |     marks, colons, question marks, or closing parentheses
  97 | ERROR | [x] Inline comments must end in full-stops, exclamation
     |       |     marks, colons, question marks, or closing parentheses
 101 | ERROR | [ ] Missing short description in doc comment
 108 | ERROR | [x] Inline comments must end in full-stops, exclamation
     |       |     marks, colons, question marks, or closing parentheses
 113 | ERROR | [x] Inline comments must end in full-stops, exclamation
     |       |     marks, colons, question marks, or closing parentheses
 129 | ERROR | [x] Inline comments must end in full-stops, exclamation
     |       |     marks, colons, question marks, or closing parentheses
 154 | ERROR | [ ] Inline doc block comments are not allowed; use "/*
     |       |     Comment */" or "// Comment" instead
 186 | ERROR | [ ] Missing short description in doc comment
 194 | ERROR | [x] Inline comments must end in full-stops, exclamation
     |       |     marks, colons, question marks, or closing parentheses
 241 | ERROR | [ ] Missing short description in doc comment
 242 | ERROR | [ ] Missing parameter comment
 243 | ERROR | [ ] Description for the @return value is missing
 243 | ERROR | [x] Data types in @return tags need to be fully namespaced
 250 | ERROR | [ ] Missing short description in doc comment
 251 | ERROR | [ ] Missing parameter comment
 252 | ERROR | [ ] Description for the @return value is missing
 252 | ERROR | [x] Data types in @return tags need to be fully namespaced
 259 | ERROR | [ ] Missing short description in doc comment
 260 | ERROR | [ ] Missing parameter comment
 261 | ERROR | [ ] Description for the @return value is missing
 261 | ERROR | [x] Data types in @return tags need to be fully namespaced
 268 | ERROR | [ ] Missing short description in doc comment
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 12 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...r/drupal/pareviewsh/pareview_temp/src/Manager/TransactionManager.php
--------------------------------------------------------------------------
FOUND 4 ERRORS AND 1 WARNING AFFECTING 5 LINES
--------------------------------------------------------------------------
  7 | WARNING | [x] Unused use statement
 10 | ERROR   | [x] Doc comment short description must end with a full
    |         |     stop
 11 | ERROR   | [x] There must be exactly one blank line before the tags
    |         |     in a doc comment
 15 | ERROR   | [ ] Missing short description in doc comment
 33 | ERROR   | [x] Return comment indentation must be 3 spaces, found 2
    |         |     spaces
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...101/web/vendor/drupal/pareviewsh/pareview_temp/src/CgChargeEvent.php
--------------------------------------------------------------------------
FOUND 9 ERRORS AFFECTING 9 LINES
--------------------------------------------------------------------------
  8 | ERROR | [x] Missing class doc comment
 12 | ERROR | [ ] Missing short description in doc comment
 17 | ERROR | [ ] Missing short description in doc comment
 22 | ERROR | [ ] Parameter $request is not described in comment
 25 | ERROR | [ ] Missing parameter comment
 32 | ERROR | [ ] Missing short description in doc comment
 33 | ERROR | [ ] Description for the @return value is missing
 39 | ERROR | [ ] Missing short description in doc comment
 40 | ERROR | [ ] Description for the @return value is missing
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...dor/drupal/pareviewsh/pareview_temp/src/Utility/TransactionTrait.php
--------------------------------------------------------------------------
FOUND 8 ERRORS AFFECTING 8 LINES
--------------------------------------------------------------------------
  6 | ERROR | [x] Doc comment short description must end with a full stop
 11 | ERROR | [x] Doc comment short description must end with a full stop
 13 | ERROR | [ ] Missing parameter comment
 15 | ERROR | [x] Parameter comment must end with a full stop
 17 | ERROR | [ ] Return type "\Drupal\cg_payment\TransactionInterface |
    |       |     bool" must not contain spaces
 20 | ERROR | [ ] Type hint "array" missing for $statuses
 21 | ERROR | [x] Inline comments must end in full-stops, exclamation
    |       |     marks, colons, question marks, or closing parentheses
 26 | ERROR | [x] Inline comments must end in full-stops, exclamation
    |       |     marks, colons, question marks, or closing parentheses
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...iew_temp/src/Plugin/Validation/Constraint/ValidTerminalValidator.php
--------------------------------------------------------------------------
FOUND 6 ERRORS AFFECTING 6 LINES
--------------------------------------------------------------------------
 17 | ERROR | [x] Doc comment short description must end with a full stop
 46 | ERROR | [ ] Inline doc block comments are not allowed; use "/*
    |       |     Comment */" or "// Comment" instead
 51 | ERROR | [x] Inline comments must end in full-stops, exclamation
    |       |     marks, colons, question marks, or closing parentheses
 56 | ERROR | [x] Inline comments must end in full-stops, exclamation
    |       |     marks, colons, question marks, or closing parentheses
 64 | ERROR | [x] Inline comments must end in full-stops, exclamation
    |       |     marks, colons, question marks, or closing parentheses
 77 | ERROR | [x] Expected 1 newline at end of file; 0 found
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...wsh/pareview_temp/src/Plugin/Validation/Constraint/ValidTerminal.php
--------------------------------------------------------------------------
FOUND 8 ERRORS AND 1 WARNING AFFECTING 8 LINES
--------------------------------------------------------------------------
  1 | ERROR   | [x] The PHP open tag must be followed by exactly one
    |         |     blank line
 18 | ERROR   | [ ] Missing short description in doc comment
 21 | ERROR   | [ ] Class property $terminal_id_field_name should use
    |         |     lowerCamel naming without underscores
 23 | ERROR   | [ ] Missing short description in doc comment
 26 | ERROR   | [ ] Class property $mid_field_name should use lowerCamel
    |         |     naming without underscores
 28 | ERROR   | [ ] Missing short description in doc comment
 34 | WARNING | [x] A comma should follow the last multiline array item.
    |         |     Found: 'mid_field_name'
 37 | ERROR   | [x] Expected 1 newline at end of file; 0 found
 37 | ERROR   | [x] The closing brace for the class must have an empty
    |         |     line before it
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...upal/pareviewsh/pareview_temp/src/Controller/PaymentCompletePage.php
--------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
--------------------------------------------------------------------------
  25 | ERROR | [ ] Missing short description in doc comment
  62 | ERROR | [ ] Missing short description in doc comment
 169 | ERROR | [x] Data types in @param tags need to be fully namespaced
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: .../web/vendor/drupal/pareviewsh/pareview_temp/src/RequestInterface.php
--------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------
 26 | ERROR | [ ] Return type "string url" must not contain spaces
 28 | ERROR | [x] There must be no blank lines after the function comment
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: .../web/vendor/drupal/pareviewsh/pareview_temp/src/Form/CreditGuard.php
--------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
--------------------------------------------------------------------------
 16 | ERROR | Missing short description in doc comment
 21 | ERROR | Missing short description in doc comment
 33 | ERROR | Missing parameter comment
--------------------------------------------------------------------------


FILE: ...te1101/web/vendor/drupal/pareviewsh/pareview_temp/cg_payment.api.php
--------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------
 11 | ERROR | Doc comment short description must be on a single line,
    |       | further text should be a separate paragraph
--------------------------------------------------------------------------

Time: 2.34 secs; Memory: 6Mb