Print

Please login or register.
1 Hour 1 Day 1 Week 1 Month Forever
Login with username, password and session length

[Maketextgames.com]

Simple Make-Up here, probably could have used the laston column in users but my math is bleh so doing the calculations would be a pain.

This is much simpler.


All this does is check an active type column that is updated + 1 daily.

When that column is > 30 it deletes the user.  If the user Logs in that column goes back to 0.

Nothing major.  Quick inserts.


First add the SQL

ALTER TABLE `users` ADD `oncheck` INT(11) NOT NULL DEFAULT 0;

Then open up Authenticate.php

find.

Code: [Select]

$IP = $_SERVER['REMOTE_ADDR'];
$db->query("UPDATE users SET lastip_login='$IP',last_login=unix_timestamp() WHERE userid={$mem['userid']}");

Note**** IF your IP isnt set like the above, or mysql_real_escaped change it for security reasons.

Below the code above add

Code: [Select]

$db->query("UPDATE `users` SET `oncheck` = 0 WHERE `userid`=".$mem['userid']."");

Close/Save.

Next open up Cron Day.

Above ?>

Add the following

Code: [Select]

$checkout = $db->query("SELECT u.`userid`,us.`userid` FROM `users` u LEFT JOIN `userstats` us ON u.`userid`=us.`userid` WHERE `oncheck`>30");
while($check = $db->fetch_row($checkout))
{
$db->query("DELETE FROM `users` WHERE `userid`={$check['userid']}");
$db->query("DELETE FROM `userstats` WHERE `userid`={$check['userid']}");
}

Save.  Done.


Simple really.=)

[Jordan]

>,< stupid (N)

[Spudinski]

It's not really that hard math, though.

SQL Query:

Code: [Select]

DELETE FROM `users` WHERE `laston` <= (`laston` - 2592000)
I'm pretty sure that does just as well a job.

[Jordan]

Deleting user's because there inactive is STUPID

Although Spuds right.. All that code when you all ya need is one query really >,<

[Maketextgames.com]

I wasn't entirely sure how the last-on bit worked as it's several integers even if I just logged in, so wasnt grapsing the concept.

@Pudda, someone wanted this, i just slapped it together for them and then threw on here "shrug"

code isnt largely shorter ^

As you still need 2 db queries one to remove from userstats and one from users :p

The only thing differant is you wouldnt need

$checkout = $db->query("SELECT u.`userid`,us.`userid` FROM `users` u LEFT JOIN `userstats` us ON u.`userid`=us.`userid` WHERE `oncheck`>30");
while($check = $db->fetch_row($checkout))
{

and you wouldnt need

$db->query("UPDATE `users` SET `oncheck` = 0 WHERE `userid`=".$mem['userid']."");

really not a huge differance as far as "speed" clutter" goes.

[Zeddicus]

If you use this, make sure *everything* user related is deleted aswell. (Gangs,items) ect ect.

[Maketextgames.com]

Aye it all depends on what mods etc they have.  (:

Good point.

I will post queries you may need such as delete from mail, inventory, etc.

[Spudinski]

Yea, I still want to make a bot for that type of thing.
Based on the userID, it search all tables for that userID, and delete the related row - but since mod' developers aren't always correct in the way they do things, it's difficult as the column name that holds the userID isn't always the same.

Not even a regex search can do.

[Maketextgames.com]

Hm, Would definitely be a resourceful idea.

[CrimGame.com]

$db->query("DELETE FROM `users` WHERE `userid`={$check['userid']}");
$db->query("DELETE FROM `userstats` WHERE `userid`={$check['userid']}");

Wow reminds me of code from 2 years back on MWG, you forgot afew tables, events, mail, logs, .... and so on

[Maketextgames.com]

Everything Reminds you of everything, considering a dummy could code this up with minimal knowledge of PHP I sure hope you aren't implying that i actually took time to go through mwg and find something that does something so simple

You really are on yourself arent you?

[CrimGame.com]

Immortal in no way did i imply you stole any code i just simply said it reminds me of code from 2 years ago, Same discussion i participated in on MWG/CE two years ago came up with similar conclusions. I then offered my help with giving you a idea of advancing it further.

I didn't mean any disrespect, im sorry if you misinterpreted it.