DeleteStalePlayerBackups() function typo(i think)

[Cripp]

Hey.. not sure if this was intented or not but...

these 2 functions are called on booting world..

sint32 SharedDatabase::DeleteStalePlayerCorpses() {
char errbuf[MYSQL_ERRMSG_SIZE];
char *query = 0;
int32 affected_rows = 0;

// 604800 seconds = 1 week
if (!RunQuery(query, MakeAnyLenString(&query, "Delete from player_corpses where (UNIX_TIMESTAMP() - UNIX_TIMESTAMP(timeofdeath)) > 604800 and not timeofdeath=0"), errbuf, 0, &affected_rows)) {
safe_delete_array(query);
return -1;
}
safe_delete_array(query);

return affected_rows;
}

sint32 SharedDatabase::DeleteStalePlayerBackups() {
char errbuf[MYSQL_ERRMSG_SIZE];
char *query = 0;
int32 affected_rows = 0;

// 1209600 seconds = 2 weeks
if (!RunQuery(query, MakeAnyLenString(&query, "Delete from player_corpses where (UNIX_TIMESTAMP() - UNIX_TIMESTAMP(timeofdeath)) > 1209600"), errbuf, 0, &affected_rows)) {
safe_delete_array(query);
return -1;
}
safe_delete_array(query);

return affected_rows;
}

shouldnt DeleteStalePlayerBackups() be querying the backup table instead of both of them querying the normal table?

sint32 SharedDatabase::DeleteStalePlayerBackups() {
char errbuf[MYSQL_ERRMSG_SIZE];
char *query = 0;
int32 affected_rows = 0;

// 1209600 seconds = 2 weeks
if (!RunQuery(query, MakeAnyLenString(&query, "Delete from player_corpses_backup where (UNIX_TIMESTAMP() - UNIX_TIMESTAMP(timeofdeath)) > 1209600"), errbuf, 0, &affected_rows)) {
safe_delete_array(query);
return -1;
}
safe_delete_array(query);

return affected_rows;
}

shrug.. let me know :)

[KLS]

That sounds right. Wonder how long it's been like that I didn't even notice it when I was tinkering near those functions not too long ago.

[Cripp]

i found this lookiing through the 4.4 eqemu source.. then looked at my 7.0 source and it was the same :D


soo id figure its been there for a while lol