[midnight9]

Originally Posted by paul181516 Doesnt work, im already tryed

Try adding some debug prints to the code to see what is happening.

[DarkDeviL]

Originally Posted by shanapu according to https://wiki.alliedmods.net/Generic_...yer_disconnect the reason should be "self" and not "Disconnect by user."

I believe that wiki may be quite old, and might be in need of an update:

The reason has been "Disconnect by user" in CS:S for the past ~5 years, when an user quits manually on his/her own decision, but of course, it isn't completely impossible that Valve might do the same thing differently in two different games.

Maybe try OnClientDisconnect instead.

[shanapu]

Originally Posted by arne1288 I believe that wiki may be quite old, and might be in need of an update: The reason has been "Disconnect by user" in CS:S for the past ~5 years, when an user quits manually on his/her own decision, but of course, it isn't completely impossible that Valve might do the same thing differently in two different games. Maybe try OnClientDisconnect instead.

good to know! that's why referred to the wiki.
Maybe the issue comes from the extra point "." in 'StrEqual'
"Disconnect by user." != "Disconnect by user"

As midnight9 said, add some debug messages to see if the event fires and if the PUB check works fine.
Maybe also try PugSetup_GetGameState() in combination

[DarkDeviL]

Originally Posted by shanapu good to know! that's why referred to the wiki. Maybe the issue comes from the extra point "." in 'StrEqual' "Disconnect by user." != "Disconnect by user" As midnight9 said, add some debug messages to see if the event fires and if the PUB check works fine. Maybe also try PugSetup_GetGameState() in combination

My fault, the correct one is the one with the dot in the end.

[paul181516]

Anyone, the reason of user disconnect is just "Disconnect". I have to put a log that prints the reason of disconnection, and the plugin show me "Kick by admin" "Disconnected" "Timed out" and "Error" so i rewrite that little part and add some other things to the plugin. Anyway i have this little problem, all PrintToChatAll and LogMessage do twice, no idea why, thanks a lot shanapu and midnight9 for help me.

I add some features like a cvar to control time of ban and admin inmunitty. Also i steal a little function from another plugins, and this functions get clients count and start plugin only when players are more than the cvar. But i dont know if i do correctly

PHP Code:

#include <cstrike>
#include <sourcemod>
#include "include/pugsetup.inc"

/* Handles to convars used by plugin */
ConVar sm_matchkick_bantime;
ConVar sm_matchkick_minplayer;

public Plugin:myinfo = {
name = "Matchkick",
author = "paul181516",
description = "Bans a player who is disconnected on a live match",
version = "1.0",
url = "http://CounterBucaraman.ga"
}

public OnPluginStart()
{
    sm_matchkick_bantime = CreateConVar("sm_matchkick_bantime", "360", "Duracion del baneo por abandonar el PUG.");
    sm_matchkick_minplayer = CreateConVar("sm_matchkick_minplayer", "7", "Jugadores necesarios para iniciar el plugin.");
    HookEvent("player_disconnect", Event_PlayerDisconnect, EventHookMode_Post);
}

stock PlayerCountIsCorrect()
{
    new k;
    for(new i = 1; i <= MaxClients; i++)
    {
        if(k++ >= GetConVarInt(sm_matchkick_minplayer))
        {
            return true;
        }
    }
    
    return false;
}

public Action:Event_PlayerDisconnect(Handle:event, String:name[], bool:dontBroadcast)
{
    if(PugSetup_IsMatchLive() && PlayerCountIsCorrect())
    {
        new client = GetClientOfUserId(GetEventInt(event, "userid"));
        if(GetUserAdmin(client) == INVALID_ADMIN_ID)
        {
            decl String:reason[128];
            GetEventString(event, "reason", reason, sizeof(reason));
            if (StrEqual(reason, "Disconnect"))
            {
                BanClient(client, GetConVarInt(sm_matchkick_bantime), BANFLAG_AUTHID, "Fue baneado por salirse en medio de un PUG!", "Fuiste baneado por salirte en medio de un PUG! ¿Donde esta tu honor, basura?");
                LogMessage("%L fue baneado por salirse en medio de un PUG", client);
                PrintToChatAll("%N fue baneado por salirse en medio de un PUG", client);
            }
        }
    }
    return Plugin_Handled;
} 


[shanapu]

Originally Posted by paul181516 ... Anyway i have this little problem, all PrintToChatAll and LogMessage do twice, no idea why,...

I faced this problem few times in the past, every time cause I was running the plugin two times.
Check your plugins folder and subfolders for a duplicate plugin, maybe with an old name or named after compiled on webcompiler (plugin(2).smx,...)

Originally Posted by paul181516 But i dont know if i do correctly

use https://sm.alliedmods.net/new-api/cl...GetClientCount instead your stock
just a fast edit, not tested, should work, still things to improve.

PHP Code:

#include <cstrike>
#include <sourcemod>
#include <include/pugsetup.inc>

/* Handles to convars used by plugin */
ConVar sm_matchkick_bantime;
ConVar sm_matchkick_minplayer;

public Plugin:myinfo = {
name = "Matchkick",
author = "paul181516",
description = "Bans a player who is disconnected on a live match",
version = "1.1",
url = "http://CounterBucaraman.ga"
}

public OnPluginStart()
{
    sm_matchkick_bantime = CreateConVar("sm_matchkick_bantime", "360", "Duracion del baneo por abandonar el PUG.");
    sm_matchkick_minplayer = CreateConVar("sm_matchkick_minplayer", "7", "Jugadores necesarios para iniciar el plugin.");
    HookEvent("player_disconnect", Event_PlayerDisconnect, EventHookMode_Post);
}

public Event_PlayerDisconnect(Handle:event, String:name[], bool:dontBroadcast)
{
    if(PugSetup_IsMatchLive())
    {
        if (GetClientCount(true) >= GetConVarInt(sm_matchkick_minplayer))
        {
            new client = GetClientOfUserId(GetEventInt(event, "userid"));
            if(GetUserAdmin(client) == INVALID_ADMIN_ID)
            {
                decl String:reason[128];
                GetEventString(event, "reason", reason, sizeof(reason));
                if (StrEqual(reason, "Disconnect"))
                {
                    BanClient(client, GetConVarInt(sm_matchkick_bantime), BANFLAG_AUTHID, "Fue baneado por salirse en medio de un PUG!", "Fuiste baneado por salirte en medio de un PUG! ¿Donde esta tu honor, basura?");
                    LogMessage("%L fue baneado por salirse en medio de un PUG", client);
                    PrintToChatAll("%N fue baneado por salirse en medio de un PUG", client);
                }
            }
        }
    }
} 


[paul181516]

Originally Posted by shanapu I faced this problem few times in the past, every time cause I was running the plugin two times. Check your plugins folder and subfolders for a duplicate plugin, maybe with an old name or named after compiled on webcompiler (plugin(2).smx,...) use https://sm.alliedmods.net/new-api/cl...GetClientCount instead your stock just a fast edit, not tested, should work, still things to improve. PHP Code: #include <cstrike> #include <sourcemod> #include <include/pugsetup.inc> /* Handles to convars used by plugin */ ConVar sm_matchkick_bantime; ConVar sm_matchkick_minplayer; public Plugin:myinfo = { name = "Matchkick", author = "paul181516", description = "Bans a player who is disconnected on a live match", version = "1.1", url = "http://CounterBucaraman.ga" } public OnPluginStart() {     sm_matchkick_bantime = CreateConVar("sm_matchkick_bantime", "360", "Duracion del baneo por abandonar el PUG.");     sm_matchkick_minplayer = CreateConVar("sm_matchkick_minplayer", "7", "Jugadores necesarios para iniciar el plugin.");     HookEvent("player_disconnect", Event_PlayerDisconnect, EventHookMode_Post); } public Event_PlayerDisconnect(Handle:event, String:name[], bool:dontBroadcast) {     if(PugSetup_IsMatchLive())     {         if (GetClientCount(true) >= GetConVarInt(sm_matchkick_minplayer))         {             new client = GetClientOfUserId(GetEventInt(event, "userid"));             if(GetUserAdmin(client) == INVALID_ADMIN_ID)             {                 decl String:reason[128];                 GetEventString(event, "reason", reason, sizeof(reason));                 if (StrEqual(reason, "Disconnect"))                 {                     BanClient(client, GetConVarInt(sm_matchkick_bantime), BANFLAG_AUTHID, "Fue baneado por salirse en medio de un PUG!", "Fuiste baneado por salirte en medio de un PUG! ¿Donde esta tu honor, basura?");                     LogMessage("%L fue baneado por salirse en medio de un PUG", client);                     PrintToChatAll("%N fue baneado por salirse en medio de un PUG", client);                 }             }         }     } }

Thanks a lot, but the first problem was not that your second solution was great, much easy.
I read and idea from a guy to stop the double plugin exec

Originally Posted by lake393 I was under the impression that this was normal for SourceMod, so what I did was create a string to contain the UserId of the person who triggered the event last. If the same userId is seen again, it aborts by returning.

But i dont know how save data on string, yes stupid things, but is my first plugin haha. Thanks a lot

[shanapu]

a user asked me for a warmod version...

PHP Code:

#include <cstrike>
#include <sourcemod>
#include <sdktools>
#include <warmod>

/* Handles to convars used by plugin */
ConVar sm_matchkick_bantime;
ConVar sm_matchkick_minplayer;

public Plugin:myinfo = {
name = "Matchkick for warmod",
author = "paul181516,shanapu",
description = "Bans a player who is disconnected on a live match",
version = "1.2",
url = ""
}
bool g_bIsLive = false;

public OnPluginStart()
{
    sm_matchkick_bantime = CreateConVar("sm_matchkick_bantime", "360", "Duracion del baneo por abandonar el PUG.");
    sm_matchkick_minplayer = CreateConVar("sm_matchkick_minplayer", "7", "Jugadores necesarios para iniciar el plugin.");
    HookEvent("player_disconnect", Event_PlayerDisconnect, EventHookMode_Post);
}

public void OnLiveOn3()
{
    g_bIsLive = true;
}

public void OnResetMatch()
{
    g_bIsLive = false;
}

public void OnEndMatch(const char[] ct_name, int ct_score, int t_score, const char[] t_name)
{
    g_bIsLive = false;
}

public Event_PlayerDisconnect(Handle:event, String:name[], bool:dontBroadcast)
{
    if(g_bIsLive)
    {
        if (GetClientCount(true) >= GetConVarInt(sm_matchkick_minplayer))
        {
            new client = GetClientOfUserId(GetEventInt(event, "userid"));
            if(GetUserAdmin(client) == INVALID_ADMIN_ID)
            {
                decl String:reason[128];
                GetEventString(event, "reason", reason, sizeof(reason));
                if (StrEqual(reason, "Disconnect"))
                {
                    BanClient(client, GetConVarInt(sm_matchkick_bantime), BANFLAG_AUTHID, "Fue baneado por salirse en medio de un PUG!", "Fuiste baneado por salirte en medio de un PUG! ¿Donde esta tu honor, basura?");
                    LogMessage("%L fue baneado por salirse en medio de un PUG", client);
                    PrintToChatAll("%N fue baneado por salirse en medio de un PUG", client);
                }
            }
        }
    }
} 


https://warmod.bitbucket.io/scriptin...ude/warmod.inc

still dirty and worthy of improvement

[Franc1sco]

Here a working and with more features version https://forums.alliedmods.net/showthread.php?t=319581