FILE: /var/www/html/user_logout_confirm/src/Routing/RouteSubscriber.php
-----------------------------------------------------------------------------------------------------------
FOUND 6 ERRORS AND 2 WARNINGS AFFECTING 7 LINES
-----------------------------------------------------------------------------------------------------------
1 | ERROR | [x] The PHP open tag must be followed by exactly one blank line
1 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
2 | ERROR | [x] Expected 7 space(s) before asterisk; 1 found
3 | ERROR | [x] Expected 7 space(s) before asterisk; 1 found
4 | ERROR | [x] Expected 7 space(s) before asterisk; 1 found
20 | ERROR | [ ] Inline doc block comments are not allowed; use "/* Comment */" or "// Comment" instead
22 | WARNING | [ ] Line exceeds 80 characters; contains 84 characters
24 | WARNING | [ ] Line exceeds 80 characters; contains 83 characters
-----------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------

FILE: /var/www/html/user_logout_confirm/user_logout_confirm.module
---------------------------------------------------------------------------
FOUND 10 ERRORS AFFECTING 10 LINES
---------------------------------------------------------------------------
20 | ERROR | [ ] Missing short description in doc comment
46 | ERROR | [x] There must be no blank lines after the function comment
64 | ERROR | [x] Short array syntax must be used to define arrays
68 | ERROR | [x] Short array syntax must be used to define arrays
72 | ERROR | [x] Short array syntax must be used to define arrays
87 | ERROR | [x] Short array syntax must be used to define arrays
102 | ERROR | [x] Closing brace must be on a line by itself
107 | ERROR | [x] Closing brace must be on a line by itself
112 | ERROR | [x] Closing brace must be on a line by itself
117 | ERROR | [x] Closing brace must be on a line by itself
---------------------------------------------------------------------------
PHPCBF CAN FIX THE 9 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------

CommentFileSizeAuthor
#2 user_logout_confirm-3104751-2.patch4.2 KBash2303

Comments

rohit-drupal created an issue. See original summary.

ash2303’s picture

StatusFileSize
new4.2 KB

All the phpcs error for `Drupal` & `DrupalPractice` Standards are corrected.

ash2303’s picture

Status: Active » Needs review
watergate’s picture

Status: Needs review » Closed (outdated)

I'm marking this as outdated, since the patch is for an incomplete port of the Drupal 7 version of this module. In #3112765 I've added a patch for an initial Drupal 8 version of this module (which adheres to Drupal's coding standards).

watergate’s picture