[ilham92-cc-sakura]

How to prevent IsFakeClient() throw an error, I have put IsClientConnected() first

Error Log:

L 11/07/2020 - 00:11:29: [SM] Exception reported: Client 1 is not connected
L 11/07/2020 - 00:11:29: [SM] Blaming: myplugins.smx
L 11/07/2020 - 00:11:29: [SM] Call stack trace:
L 11/07/2020 - 00:11:29: [SM]   [0] IsFakeClient
L 11/07/2020 - 00:11:29: [SM]   [1] Line 278, F:\Project\DS_L4D2\Scripting\1.10.0-git6497\myplugins::CMD_CallBlock

Code:

PHP Code:

public void OnPluginStart()
{
    // Command Call Back
    AddCommandListener(CMD_CallBlock, "wait");
    AddCommandListener(CMD_CallBlock, "pause");
    AddCommandListener(CMD_CallBlock, "unpause");
    AddCommandListener(CMD_CallBlock, "setpause");
    AddCommandListener(CMD_CallBlock, "jointeam");
    AddCommandListener(CMD_CallBlock, "go_away_from_keyboard");
}

...

public Action CMD_CallBlock(int client, const char[] command, int argc)
{
    if (client == 0) \\Console
        return Plugin_Continue;
        
    if (IsClientConnected(client)) \\Execute only connected client
        return Plugin_Continue;

    if (IsFakeClient(client)) \\Bots
        return Plugin_Continue;

    return Plugin_Handled; 


[DJ Tsunami]

Your code currently returns Plugin_Continue if the client is connected. You want to return Plugin_Continue if the client is NOT connected.

[cravenge]

I don't think that's the case here. Also, you don't need the IsFakeClient check since bots can't do any of those commands and replace IsClientConnected with IsClientInGame since you're basically blocking no one that way. You can simply intercept all commands from being executed by connecting clients with:

PHP Code:

public void OnAllPluginsLoaded()
{
    AddCommandListener(OnAllCmds, "");
}

public Action OnAllCmds(int client, const char[] command, int argc)
{
    return (client == 0 || IsClientInGame(client)) ? Plugin_Continue : Plugin_Handled;
} 


[Psyk0tik]

Actually, DJ Tsunami is right. The error is caused by returning Plugin_Handled for someone that isn't even there. Plugin_Handled is only returned in this scenario to block the command from going through. If the client isn't connected, then there's nobody to block the command for, hence the error. The console and bot checks are correct, but the connection check is invalidating the OP's intended target(s) (connected players).