FILE: /var/www/html/user_logout_confirm/user_logout_confirm.module
---------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------
86 | WARNING | Messages are user facing text and must run through t() for translation
---------------------------------------------------------------------------------------

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

Comments

rohit-drupal created an issue. See original summary.

ash2303’s picture

ash2303 commented
StatusFileSize
new user_logout_confirm-3104751-2.patch4.2 KB

Fixed with patch submitted for https://www.drupal.org/project/user_logout_confirm/issues/3104751. Attaching here as well.

ash2303’s picture

ash2303 commented
Status: Active » Needs review
watergate’s picture

watergate commented
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

watergate commented
Related issues: +#3112765: Implement Drupal 8 version of this module by resolving the @FIXME comments