http://git.drupal.org/project/masquerade 8.x-2.x

Primary tabs

Git errors: Review of the 8.x-2.x branch (commit b409976): This automated report was generated with PAReview.sh, your friendly project application review script.

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


FILE: ...eviewsh/pareview_temp/tests/src/Functional/MasqueradeWebTestBase.php
--------------------------------------------------------------------------
FOUND 25 ERRORS AND 1 WARNING AFFECTING 14 LINES
--------------------------------------------------------------------------
  40 | ERROR   | [ ] Class property $admin_user should use lowerCamel
     |         |     naming without underscores
  40 | ERROR   | [ ] There must not be more than one property declared
     |         |     per statement
  40 | ERROR   | [ ] Class property $auth_user should use lowerCamel
     |         |     naming without underscores
  40 | ERROR   | [ ] Class property $editor_user should use lowerCamel
     |         |     naming without underscores
  40 | ERROR   | [ ] Class property $masquerade_user should use
     |         |     lowerCamel naming without underscores
  40 | ERROR   | [ ] Class property $moderator_user should use lowerCamel
     |         |     naming without underscores
  47 | ERROR   | [ ] Class property $admin_role should use lowerCamel
     |         |     naming without underscores
  47 | ERROR   | [ ] There must not be more than one property declared
     |         |     per statement
  47 | ERROR   | [ ] Class property $editor_role should use lowerCamel
     |         |     naming without underscores
  47 | ERROR   | [ ] Class property $masquerade_role should use
     |         |     lowerCamel naming without underscores
  47 | ERROR   | [ ] Class property $moderator_role should use lowerCamel
     |         |     naming without underscores
 136 | ERROR   | [x] No space found before comment text; expected "//
     |         |     $this->drupalGet('user/' . $account->id() .
     |         |     '/masquerade', [" but found
     |         |     "//$this->drupalGet('user/' . $account->id() .
     |         |     '/masquerade', ["
 138 | WARNING | [ ] Line exceeds 80 characters; contains 85 characters
 138 | ERROR   | [ ] Comment indentation error, expected only 2 spaces
 140 | ERROR   | [x] No space found before comment text; expected "//
     |         |     ]);" but found "//]);"
 141 | ERROR   | [x] No space found before comment text; expected "//
     |         |     $this->assertSession()->statusCodeEquals(200);" but
     |         |     found
     |         |     "//$this->assertSession()->statusCodeEquals(200);"
 148 | ERROR   | [x] No space found before comment text; expected "//
     |         |     $this->loggedInUser = $account;" but found
     |         |     "//$this->loggedInUser = $account;"
 149 | ERROR   | [x] No space found before comment text; expected "//
     |         |     $this->loggedInUser->session_id =
     |         |     $this->session_id;" but found
     |         |     "//$this->loggedInUser->session_id =
     |         |     $this->session_id;"
 149 | ERROR   | [x] Inline comments must end in full-stops, exclamation
     |         |     marks, colons, question marks, or closing
     |         |     parentheses
 165 | ERROR   | [x] No space found before comment text; expected "//
     |         |     $this->drupalGet('unmasquerade', [" but found
     |         |     "//$this->drupalGet('unmasquerade', ["
 167 | ERROR   | [ ] Comment indentation error, expected only 2 spaces
 169 | ERROR   | [x] No space found before comment text; expected "//
     |         |     ]);" but found "//]);"
 170 | ERROR   | [x] No space found before comment text; expected "//
     |         |     $this->assertSession()->statusCodeEquals(200);" but
     |         |     found
     |         |     "//$this->assertSession()->statusCodeEquals(200);"
 177 | ERROR   | [x] No space found before comment text; expected "//
     |         |     $this->loggedInUser = $account;" but found
     |         |     "//$this->loggedInUser = $account;"
 178 | ERROR   | [x] No space found before comment text; expected "//
     |         |     $this->loggedInUser->session_id =
     |         |     $this->session_id;" but found
     |         |     "//$this->loggedInUser->session_id =
     |         |     $this->session_id;"
 178 | ERROR   | [x] Inline comments must end in full-stops, exclamation
     |         |     marks, colons, question marks, or closing
     |         |     parentheses
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 12 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------


FILE: ...ite1101/web/vendor/drupal/pareviewsh/pareview_temp/masquerade.module
--------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------
 191 | ERROR | Doc comment short description must start with a capital
     |       | letter
--------------------------------------------------------------------------

Time: 2.3 secs; Memory: 6Mb