[iggy1]

Try this, it's your code but simplified, leave no chance of 2 timers on same var. It toggles it on/off one command.

pawn Код:

if (strcmp("/detector", cmdtext, true, 11) == 0)
{
    if(PlayerInfo[playerid][pDetector])
    {
        if(IsPlayerInAnyVehicle(playerid))
        {
            if( !DetectorOn[playerid] )
            {
                DetectorTimer[playerid] = SetTimerEx("RunDetector", 2000, true, "i", playerid);
                SendClientMessage(playerid,-1,"Detector On");
                DetectorOn[playerid]=1;
                return 1;
            }
            else
            {
                SendClientMessage(playerid,-1,"Detector OFF");
                KillTimer(DetectorTimer[playerid]);
                DetectorOn[playerid]=0;
                return 1;
            }
        }
        SendClientMessage(playerid, COLOR_GREY, "** You aren't in a vehicle!");
        return 1;
    }
}