Option to delete never logged in users

What is the status on this? I really need this feature. When can we expect this to be tested and added to the official release code?

Thanks

+    // automatically delete never logged in users
+    if ($delete_time = variable_get('never_logged_in_user_auto_delete', 0)) {
+      $result = db_query('SELECT * FROM {users} WHERE access = 0 AND ((%d - created ) > %d) AND NOT (uid=1)', time(), $delete_time);

In the code above, the expression ((%d - created ) > %d) never gets TRUE and thus no user ever gets deleted. To test this, I replaced this expression with 0 and replaced the code that delete users by a line that blocks them. With my test mod, I could block all users that never loged in indeed.

So the problem lies in the value of $delete_time it seems. Hope this helps.

Changing status based on marcbe's comment.

Why the expression never gets TRUE?

I have tested this patch and it works fine for me, it is exactly doing what should hapen:

A = time() - created => timestamp from now until the user created the account so, 0 = really now, 3600 == hour ago, etc...

A > $time_deleted (time_deleted is 604800 for one week, 1209600 for 2 weeks etc...)

If time_deleted = 0 then the code will never execute because it's disabled.!

attached new version diff with the 1.4 release!

duplicate of #162176: will this email users who never logged on, find patch for 5.x-1.4 there