Problem/Motivation

Currently, the Drupal 7 version of this module has been copied to the Drupal 8 branch. The functionality doesn't work.

Proposed resolution

Implement the code as suggested by the @FIXME comments.

Remaining tasks

  1. Provide patch containing the fixes.
  2. Test the patch and apply it, so that the an initial Drupal 8 version of this module can be released.
CommentFileSizeAuthor
#4 user_logout_confirm-initial-drupal-8-version-3112765-4.patch8.44 KBwatergate
#2 user_logout_confirm-initial-drupal-8-version-3112765-2.patch8.43 KBwatergate

Comments

Watergate created an issue. See original summary.

watergate’s picture

watergate commented
Status: Needs work » Needs review
StatusFileSize
new user_logout_confirm-initial-drupal-8-version-3112765-2.patch8.43 KB

Attached patch provides a Drupal 8 version of this module.

The added route subscriber, alters the user.logout route by replacing Drupal's original controller by this modules confirm form. When agreeing to the confirm form, the original controller's functionality is used to log off the user correctly.

Please test and review the patch.

watergate’s picture

watergate commented
Assigned: watergate » Unassigned
watergate’s picture

watergate commented
StatusFileSize
new user_logout_confirm-initial-drupal-8-version-3112765-4.patch8.44 KB

Fixed a small typo.

watergate’s picture

watergate commented
Issue summary: View changes
geoanders’s picture

geoanders commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.