SA-MP Forums Archive
My gm only kick ,no ban players - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+--- Thread: My gm only kick ,no ban players (/showthread.php?tid=640257)


My gm only kick ,no ban players - timsosa09 - 30.08.2017

My server only kick players does not ban them because?

Код:
CMD:ban(playerid, params[])
{
    if(Player[playerid][Level] < 3) return SendClientMessage(playerid,-1,"{FFFFFF}Error: {0099FF}You need to be a higher admin level.");
	if(AllowStartBase == false) return SendClientMessage(playerid,-1,"{FFFFFF}Error: {0099FF}Can't kick now. Please wait.");

	new Params[2][128], iString[180];
	#if PLUGINS == 1
		sscanf(params, "s[128]s[128]", Params[0], Params[1]);
	#else
	    sscanf(params, "ss", Params[0], Params[1]);
	#endif
	if(isnull(Params[0]) || !IsNumeric(Params[0])) return SendClientMessage(playerid,-1,"{FFFFFF}USAGE: {0099FF}/ban [Player ID] [Reason]");
	new pID = strval(Params[0]);

	if(!IsPlayerConnected(pID)) return SendClientMessage(playerid,-1,"{FFFFFF}Error: {0099FF}That player isn't connected.");
	if(Player[pID][Level] >= Player[playerid][Level]) return SendClientMessage(playerid,-1,"{FFFFFF}Error: {0099FF}Can't ban someone of same or higher admin level.");
	
	new IP[50];
	GetPlayerIp(pID, IP, sizeof(IP));

	new bool:GiveReason;
	if(isnull(Params[1])) GiveReason = false;
	else GiveReason = true;


	if(GiveReason == false) {
		format(iString, sizeof(iString), "{FFFFFF}%s {0099FF}has banned {FFFFFF}%s (IP: %s) {0099FF}| Reason: {FFFFFF}No Reason Given", Player[playerid][Name], Player[pID][Name], IP);
		SendClientMessageToAll(-1, iString);
		Ban(pID);///
	} else {
		format(iString, sizeof(iString), "{FFFFFF}%s {0099FF}has banned {FFFFFF}%s (IP: %s) {0099FF}| Reason: {FFFFFF}%s", Player[playerid][Name], Player[pID][Name], IP, Params[1]);
		SendClientMessageToAll(-1, iString);
		BanEx(pID, Params[1]);///
	}

    SetTimerEx("OnPlayerBan", 1000, false, "i", pID);

	return 1;
}
Users

Код:
Password=344907E89B981CAF221D05F597EB57A6AF408F15F4DD7895BBD1B96A2938EC24A7DCF23ACB94ECE0B6D7B0640358BC56BDB448194B9305311AFF038A834A079F
Level=0
Weather=0
Time=10
ChatChannel=-1


Re: My gm only kick ,no ban players - mongi - 01.09.2017

Show OnPlayerBan function

Also showing the user IP is a real BAD idea. You should remove it

Re: My gm only kick ,no ban players - ISmokezU - 01.09.2017

Your code could be compressed. Probably even more than how I compressed it

PHP код:
CMD:ban(playerid, params[])
{
    new target, string[144], reason[78];
    if(Player[playerid][Level] < 3) return SendClientMessage(playerid,-1,"{FFFFFF}Error: {0099FF}You need to be a higher admin level.");
    if(AllowStartBase == false) return SendClientMessage(playerid,-1,"{FFFFFF}Error: {0099FF}Can't kick now. Please wait.");// ...
    if(sscanf(params, "us[78]", target, reason)) return SendClientMessage(playerid,-1,"{FFFFFF}USAGE: {0099FF}/ban [Player ID] [Reason]");
    if(target == INVALID_PLAYER_ID) return SendClientMessage(playerid,-1,"{FFFFFF}Error: {0099FF}That player isn't connected.");
    if(Player[target][Level] >= Player[playerid][Level]) return SendClientMessage(playerid,-1,"{FFFFFF}Error: {0099FF}Can't ban someone of same or higher admin level.");

    new IP[16];
    GetPlayerIp(target, IP, sizeof(IP));

    if(isnull(params)) {
        format(string, sizeof string, "{FFFFFF}%s {0099FF}has banned {FFFFFF}%s (IP: %s) {0099FF}| Reason: {FFFFFF}No Reason Given", Player[playerid][Name], Player[target][Name], IP);
        SendClientMessageToAll(-1, iString);
    } else {
        format(string, sizeof string, "{FFFFFF}%s {0099FF}has banned {FFFFFF}%s (IP: %s) {0099FF}| Reason: {FFFFFF}%s", Player[playerid][Name], Player[target][Name], IP, Params[1]);
        SendClientMessageToAll(-1, string);
    }
    SetTimerEx("OnPlayerBan", 10, false, "i", target);
    return true;
} 


Re: My gm only kick ,no ban players - mongi - 01.09.2017

Quote:
Originally Posted by ISmokezU
Посмотреть сообщение
Your code could be compressed. Probably even more than how I compressed it

PHP код:
CMD:ban(playerid, params[])
{
    new target, string[144], reason[78];
    if(Player[playerid][Level] < 3) return SendClientMessage(playerid,-1,"{FFFFFF}Error: {0099FF}You need to be a higher admin level.");
    if(AllowStartBase == false) return SendClientMessage(playerid,-1,"{FFFFFF}Error: {0099FF}Can't kick now. Please wait.");// ...
    if(sscanf(params, "us[78]", target, reason)) return SendClientMessage(playerid,-1,"{FFFFFF}USAGE: {0099FF}/ban [Player ID] [Reason]");
    if(target == INVALID_PLAYER_ID) return SendClientMessage(playerid,-1,"{FFFFFF}Error: {0099FF}That player isn't connected.");
    if(Player[target][Level] >= Player[playerid][Level]) return SendClientMessage(playerid,-1,"{FFFFFF}Error: {0099FF}Can't ban someone of same or higher admin level.");
    new IP[16];
    GetPlayerIp(target, IP, sizeof(IP));
    if(isnull(params)) {
        format(string, sizeof string, "{FFFFFF}%s {0099FF}has banned {FFFFFF}%s (IP: %s) {0099FF}| Reason: {FFFFFF}No Reason Given", Player[playerid][Name], Player[target][Name], IP);
        SendClientMessageToAll(-1, iString);
    } else {
        format(string, sizeof string, "{FFFFFF}%s {0099FF}has banned {FFFFFF}%s (IP: %s) {0099FF}| Reason: {FFFFFF}%s", Player[playerid][Name], Player[target][Name], IP, reason);
        SendClientMessageToAll(-1, string);
    }
    SetTimerEx("OnPlayerBan", 10, false, "i", target);
    return true;
} 
I guess you mean this ( isnull(reason) instead of isnull(params) )

PHP код:
    if(isnull(reason)) {
        format(string, sizeof string, "{FFFFFF}%s {0099FF}has banned {FFFFFF}%s (IP: %s) {0099FF}| Reason: {FFFFFF}No Reason Given", Player[playerid][Name], Player[target][Name], IP);
        SendClientMessageToAll(-1, iString);
    } else {
        format(string, sizeof string, "{FFFFFF}%s {0099FF}has banned {FFFFFF}%s (IP: %s) {0099FF}| Reason: {FFFFFF}%s", Player[playerid][Name], Player[target][Name], IP, Params[1]);
        SendClientMessageToAll(-1, string);
    }
}