Tell the user the session was automatically logged out so they know what happen in a modal dialog or a drupal_set_message().

Comments

jlstrecker’s picture

Status: Active » Needs review
StatusFileSize
new712 bytes

The attached patch displays a message using drupal_set_message.

See also a fix for 6.x-2.5: http://drupal.org/node/1444260

nedjo’s picture

Should the module be calling user_logout() instead of doing its own custom handling here?

If not, would calling session_destroy() and then drupal_goto() to let core handle session creation be preferable to creating a new session?

nedjo’s picture

#886660: Logout message doubles suggests this functionality already exists?

nedjo’s picture

johnennew’s picture

Version: 6.x-4.0 » 6.x-4.x-dev
StatusFileSize
new1.93 KB

Here'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.

johnennew’s picture

Version: 6.x-4.x-dev » 7.x-4.x-dev

Sorry, patch above is for the 7.x-4.x branch

johnennew’s picture

Version: 7.x-4.x-dev » 6.x-4.x-dev
Assigned: Unassigned » johnennew
Status: Needs review » Needs work

I 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.

johnennew’s picture

Status: Needs work » Fixed
StatusFileSize
new2.2 KB

Back ported for 6.x-4.x and committed.

Status: Fixed » Closed (fixed)

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

  • Commit d587b2c on 7.x-4.x, 8.x-1.x by ceng:
    Issue #1990308: Added fixes for #1114846 (display a message after auto...