Closed (fixed)
Project:
Automated Logout
Version:
6.x-4.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
3 Apr 2011 at 18:07 UTC
Updated:
10 Jun 2014 at 13:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jlstreckerThe attached patch displays a message using drupal_set_message.
See also a fix for 6.x-2.5: http://drupal.org/node/1444260
Comment #2
nedjoShould the module be calling user_logout() instead of doing its own custom handling here?
If not, would calling
session_destroy()and thendrupal_goto()to let core handle session creation be preferable to creating a new session?Comment #3
nedjo#886660: Logout message doubles suggests this functionality already exists?
Comment #4
nedjoRelated core issue: #754560: Messages are lost if set just prior to logout.
Comment #5
johnennew commentedHere's an alternative approach using a query string in the redirect instead of trying to store the message. This works in more situations going forwards, such as in the case of multiple tabs being open.
Comment #6
johnennew commentedSorry, patch above is for the 7.x-4.x branch
Comment #7
johnennew commentedI have applied this auto logout message patch in #5 to the 7.x-4.x branch as part of #1990308
Setting back to the 6.x-4.x branch so it can be backported there.
Comment #8
johnennew commentedBack ported for 6.x-4.x and committed.