Instead of only deleting 1 old session, delete all old sessions

batje - September 4, 2009 - 14:05
Project:Session Limit
Version:6.x-1.0
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs review
Description

When I switched on the module on a year-old website, I was forced to manually one-by-one delete my previous 50+ sessions.

This patch wipes all previous sessions up to the amount except session_limit_max , when session_limit_auto_drop is enabled.

AttachmentSize
session_limit_disconnect_all_v1.patch1.32 KB

#1

deekayen - September 4, 2009 - 19:27
Status:needs review» needs work

Maybe I'm just reading the patch wrong, but it looks like it finds only 1 row that is the oldest session and kills that one. Shouldn't it disconnect *all* the oldest sessions leaving only the sessions left in the permissible limit range?

#2

batje - September 7, 2009 - 08:47

Indeed, i did swap the two filenames while creating the patch.

AttachmentSize
session_limit_disconnect_all_v2.patch 1.32 KB

#3

batje - September 14, 2009 - 12:19
Status:needs work» needs review

changed status

 
 

Drupal is a registered trademark of Dries Buytaert.