If pressflow is installed the _autologout_logout() function causes an error when trying to fire due to pressflows handling of cookies / session.

Comments

tayzlor’s picture

StatusFileSize
new713 bytes

patch to fix this issue.
I've also added another line -

$_SESSION['autologout'] = TRUE;

And move the session destroy to after the user module invoke.
This allows other modules to catch when an autologout has taken place and act upon it inside hook_user() in the 'logout' OP.

tayzlor’s picture

Status: Active » Needs review
mstef’s picture

Got a patch for 2.x?

mstef’s picture

This should apply for 2.5

jrz’s picture

Drupal gives me the following error when applying autologout.pressflow-1270950.patch.

Fatal error: Call to undefined function drupal_session_destroy()

Any ideas?

thanks

dstol’s picture

Version: 6.x-4.0 » 6.x-2.x-dev
StatusFileSize
new1.22 KB

Here is a patch for 6.x-2.x that supports both Pressflow and Drupal.

dstol’s picture

Version: 6.x-2.x-dev » 6.x-4.x-dev
StatusFileSize
new953 bytes

Same patch but for 6.x-4.x.

johnennew’s picture

Status: Needs review » Fixed

This seems fairly straightforward so have committed to the 6.x-4.x branch.

I don't think there needs to be a 7.x-4.x version - post a new patch if needed.

Status: Fixed » Closed (fixed)

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