SA-MP Forums Archive
[Help] Server closed the connection on connecting - 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: [Help] Server closed the connection on connecting (/showthread.php?tid=585975)


[Help] Server closed the connection on connecting - javad - 18.08.2015

Hi,
I've made an kick script
When i kick myself (for testing) it works correctly but when i try to connect again, server says "Server closed the connection.".
PHP код:
CMD:kick(playerid, params[])
{
    new
        targetid,
        Reason[50],
        AdminName[MAX_PLAYERS],
        TargetName[MAX_PLAYER_NAME],
        kickmsg[128];
    if(PlayerInfo[pAdmin][playerid]>=1)
    {
         if(sscanf(params, "us[50]", targetid, Reason)) return SendClientMessage(playerid, 0xE90A0AFF, "[CMD] /kick [playerid] [reason]");
        if(!IsPlayerConnected(targetid)) return SendClientMessage(playerid, 0xE90A0AFF, "[ERROR] Invalid player ID.");
        GetPlayerName(playerid, AdminName, sizeof(AdminName));
        GetPlayerName(targetid, TargetName, sizeof(TargetName));
        format(kickmsg, sizeof(kickmsg), "[ADMIN] Admin %s has kicked %s [Reason: %s]", AdminName, TargetName, Reason);
        SendClientMessageToAll(0xE90A0AFF, kickmsg);
        SetTimerEx("KickPlayer", 500, 1, "i", playerid);
    }
    else
    {
        SendClientMessage(playerid, 0xE90A0AFF, "[ERROR] You are not authorized.");
    }
    return 1;
} 
PHP код:
forward KickPlayer(playerid);
public KickPlayer(playerid)
{
    Kick(playerid);
    return 1;
} 
In
Код:
Kick(playerid);
what should i write? (playerid or targetid?)
[/PHP]

Re: [Help] Server closed the connection on connecting - MarvinPWN - 18.08.2015

Write it like this:
PHP код:
CMD:kick(playerid,params[])
{
    new targetid,Reason[50],AdminName[MAX_PLAYER_NAME],TargetName[MAX_PLAYER_NAME],kickmsg[145];
    if(PlayerInfo[pAdmin][playerid] >= 1)
    {
        if(sscanf(params,"us[50]",targetid,Reason))return SendClientMessage(playerid,0xE90A0AFF,"[CMD] /kick [playerid] [reason]");
        if(!IsPlayerConnected(targetid))return SendClientMessage(playerid,0xE90A0AFF,"[ERROR] Invalid player ID.");
        GetPlayerName(playerid,AdminName,MAX_PLAYER_NAME);
        GetPlayerName(targetid,TargetName,MAX_PLAYER_NAME);
        format(kickmsg,sizeof kickmsg,"[ADMIN] Admin %s has kicked %s [Reason: %s]",AdminName,TargetName,Reason);
        SendClientMessageToAll(0xE90A0AFF,kickmsg);
        SetTimerEx("KickPlayer",500,0,"i",targetid);
        return 1;
    }
    SendClientMessage(playerid,0xE90A0AFF,"[ERROR] You are not authorized.");
    return 1;
}
forward KickPlayer(playerid);
public KickPlayer(playerid)
{
    Kick(playerid);
    return 1;
} 
You have to start the timer with targetid and the repeat as 0.

Re: [Help] Server closed the connection on connecting - CNMike - 18.08.2015

Kick(playerid) that is in side the timer stays exactly the same, because targetid is not defined globally and gets passed throw into the playerid parem anyway. AdminName[MAX_PLAYERS] should be AdminName[MAX_PLAYER_NAME]. However, it is just the size of the array ^ hes probably right about the timers. I don't use timers very often :P.

Re: [Help] Server closed the connection on connecting - MarvinPWN - 18.08.2015

@CNMike:
It doesn't crash. He start a timer with the repeat 1 and "playerid". So if he disconnected the timer is going on and when he connects again he will kick because the timer is already going on with the "playerd".

Re: [Help] Server closed the connection on connecting - javad - 18.08.2015

Quote:
Originally Posted by MarvinPWN
Посмотреть сообщение
Write it like this:
PHP код:
CMD:kick(playerid,params[])
{
    new targetid,Reason[50],AdminName[MAX_PLAYER_NAME],TargetName[MAX_PLAYER_NAME],kickmsg[145];
    if(PlayerInfo[pAdmin][playerid] >= 1)
    {
        if(sscanf(params,"us[50]",targetid,Reason))return SendClientMessage(playerid,0xE90A0AFF,"[CMD] /kick [playerid] [reason]");
        if(!IsPlayerConnected(targetid))return SendClientMessage(playerid,0xE90A0AFF,"[ERROR] Invalid player ID.");
        GetPlayerName(playerid,AdminName,MAX_PLAYER_NAME);
        GetPlayerName(targetid,TargetName,MAX_PLAYER_NAME);
        format(kickmsg,sizeof kickmsg,"[ADMIN] Admin %s has kicked %s [Reason: %s]",AdminName,TargetName,Reason);
        SendClientMessageToAll(0xE90A0AFF,kickmsg);
        SetTimerEx("KickPlayer",500,0,"i",targetid);
        return 1;
    }
    SendClientMessage(playerid,0xE90A0AFF,"[ERROR] You are not authorized.");
    return 1;
}
forward KickPlayer(playerid);
public KickPlayer(playerid)
{
    Kick(playerid);
    return 1;
} 
You have to start the timer with targetid and the repeat as 0.
Fixed.
Thanks, +REP