[TheFlyingApple]

Originally Posted by Marttt Ilusion, yeah you are correct. Well I made another version that minimizes that error, but still can "be exploited" in rare (impossible?) cases (between 2 seconds). Spoiler PHP Code: #include <sourcemod> #include <sdktools> #include <multicolors>  #pragma semicolon 1 #pragma newdecls required #define WAIT_TIME 30.0 static float g_fRoundStartTime; public void OnPluginStart() {     HookEvent("round_start", Round_Start_Hook);     HookEvent("round_end", Round_End_Hook); } public Action Round_Start_Hook(Handle event, const char[] name, bool dontBroadcast) {     g_fRoundStartTime = GetEngineTime();     CreateTimer(2.0, CheckPlayerZone, _, TIMER_REPEAT|TIMER_FLAG_NO_MAPCHANGE); } public Action Round_End_Hook(Handle event, const char[] name, bool dontBroadcast) {     g_fRoundStartTime = 0.0; } public Action CheckPlayerZone(Handle timer) {     if (g_fRoundStartTime == 0.0)         return Plugin_Stop;          if (GetEngineTime() - g_fRoundStartTime < WAIT_TIME)         return Plugin_Continue;     for (int i = 1; i <= MaxClients; i++)     {         if (IsClientConnected(i) && IsClientInGame(i) && IsPlayerAlive(i))         {             SlapPlayer(i, 20, true);             CPrintToChat(i, "{green}You are being slapped!");             //PrintToChat(i, "{green}You are being slapped!");         }     }     return Plugin_Continue; }

YES! Perfect! This worked... I see what you did ;) Thanks for your help everybody!

[Dragokas]

Marttt, good idea.
But, is there a reason to stop and re-create timer at all.

Spoiler

PHP Code:

#pragma semicolon 1
#pragma newdecls required

#include <sourcemod>
#include <sdktools>

#define WAIT_TIME 30.0

float g_fRoundStartTime;

public void OnPluginStart()
{
    HookEvent("round_start",             Event_RoundStart,         EventHookMode_PostNoCopy);
    HookEvent("round_end",                 Event_RoundEnd,         EventHookMode_PostNoCopy);
    
    CreateTimer(2.0, Timer_CheckPlayerZone, _, TIMER_REPEAT);
}

public Action Event_RoundStart(Handle event, const char[] name, bool dontBroadcast)
{
    g_fRoundStartTime = GetEngineTime();
}

public Action Event_RoundEnd(Handle event, const char[] name, bool dontBroadcast)
{
    g_fRoundStartTime = 0.0;
}

public void OnMapEnd()
{
    g_fRoundStartTime = 0.0;
}

Action Timer_CheckPlayerZone(Handle timer)
{
    if ( g_fRoundStartTime == 0.0 )
        return Plugin_Continue;
    
    if ( GetEngineTime() - g_fRoundStartTime < WAIT_TIME )
        return Plugin_Continue;
    
    for ( int i = 1; i <= MaxClients; i++ )
    {
        if ( IsClientInGame(i) && IsPlayerAlive(i) )
        {
            SlapPlayer(i, 20, true);
            PrintToChat(i, "You are being slapped!");
        }
    }
    return Plugin_Continue;
} 


[Dragokas]

Marttt, one more notice, just for the future, your timer run twice if round lost, or when round ends without change map (CS:GO or similar games).

[Marttt]

Thanks, I was just supporting, I know there is a reason to kill the timer.
For my l4d2 versus server it worked fine, I don't know which game the author is trying to implement.
Maybe hooking other events it could be achieved. Some things he must discover by himself.

[Silvers]

For testing I sometimes double restart (mp_restartgame 1) in Versus so Im back on the same team, using your code Marttt without killing the timer would probably lead to a duplicate timer running.