[FilterScript] TAdmin - A MySQL administration system
#5

Quote:
Originally Posted by KinderClans
View Post
You're using latest MySQL plugin but not foreach, the classic for(new i = 0; i < MAX_PLAYERS; i ++).
Originally, I wanted to use as little includes/plugins as possible so it's easier for anyone to just add it so I decided to go with classic loops and not foreach. But in the end, I still ended up with a lot of includes and plugins so I'll probably just add foreach in a future update. Same with y_timers.

Quote:
Originally Posted by KinderClans
View Post
mysql_format(db, query, sizeof(query), "SELECT `pPass`, `pID` FROM `accounts` WHERE `pUsername` = '%e'", GetName(playerid));
Thanks! Will fix this in a future update.

Quote:
Originally Posted by KinderClans
View Post
pawn Code:
format(string, sizeof(string), "* %s (%d) has connected. Country: %s", GetName(playerid), playerid, GetPlayerCountry(playerid));
    SendAdminMessage(COLOR_ADMINMSG, string);
   
    format(string, sizeof(string), "* %s (%d) has connected.", GetName(playerid), playerid);
    SendClientMessageToAll(COLOR_GRAY, string);
That would send 2 messages to admins, no? That's why it's looping through the players. To find which one is an admin and which one is not so it sends the correct message.

Also yeah, not really happy with the current /ahelp layout (Spent a long time deciding it) so I'll work on improving it.

Quote:
Originally Posted by KinderClans
View Post
Also you're using Whirlpool without salting the passwords. Use SALT and SHA-256 to encrypt them. Better for you and better for the security of who'll use this system
Will add that as well, didn't spend enough time on the login/reg system.

Thanks for the criticism!
Reply


Messages In This Thread

Forum Jump:


Users browsing this thread: 1 Guest(s)