[leonardo1434]

This may work, a little thing, your code is pretty messed up, try fixing this mess, it may help you alot for prevent future mistakes.

pawn Код:

CMD:cuff(playerid, params[])
{
    new ID,pname[24],opname[24],string[200];
    if(!IsPlayerACop(playerid)) return SendClientMessage(playerid, COLOR_RED,"You are not a law enforcement.");
    if(IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, COLOR_RED, "You cannot use this command in a vehicle.");
    if(sscanf(params, "u", ID)) return SendClientMessage(playerid, COLOR_RED, "Usage: /cuff (ID)");
    if(!IsPlayerConnected(ID)) return SendClientMessage(playerid, COLOR_RED, "Invalid ID.");
    GetPlayerName(ID, opname, 24); GetPlayerName(playerid, pname, 24);
    if(GetDistanceBetweenPlayers(playerid, ID) > 3) return SendClientMessage(playerid, COLOR_RED, "That player is too far away to be cuffed.");
    if(IsPlayerInAnyVehicle(ID)) return SendClientMessage(playerid, COLOR_RED, "That player is in a vehicle. You cannot cuff him.");
    if(IsPlayerACop(ID)) return SendClientMessage(playerid, COLOR_RED, "You cannot cuff other Law Enforcements.");
    TogglePlayerControllable(ID, 0);
    GameTextForPlayer(ID, "~b~HANDCUFFED!", 5000, 1);
    format(string, sizeof(string), "[POLICE] Officer %s(%d) has put hand cuffs on you. You cannot move.", pname, playerid);
    SendClientMessage(ID, COLOR_LIGHTBLUE, string);
    format(string, 200, "[DISPATCH] Officer %s(%d) has cuffed player %s(%d)", pname, playerid, opname, ID);
    SendClientMessage(playerid, COLOR_LIGHTBLUE, "[POLICE] You have cuffed a suspect. You have 30 seconds before they get uncuffed.");
    SetTimerEx(#cuffed,2000*10,false,"i",ID); // it gonna repate once, since is set to false, and only for the ID, which is the victim.
    return 1;
}
forward cuffed(playerid);
public cuffed(playerid)
{
    SendClientMessage(playerid,-1,#you're free);
    return TogglePlayerControllable(playerid, 1); // unfreezing him.
}

@ These guys that posted above are kinda wrong, first one is not required a loop or even call the settimer every second, is just required once, and doesn't require to destroy it, cause it's once.

second, why we would put a settimer on ongamemodeinit? to call it every 20 seconds, kinda useless.