[Guss]

Hi there, sorry but I need help whit this:

PHP Code:

dcmd_jail(playerid, params[])
{
      new jailedid, string[128];
      if(sscanf(params, "u", jailedid)) return SendClientMessage(playerid, -1, "/jail <playerid>");
      if(jailedid == INVALID_PLAYER_ID) return SendClientMessage(playerid, -1, "Player is not online.");
      if(Jailed[playerid] == true) return SendClientMessage(playerid, -1, "Player is already jailed.");
    Jailed[playerid]=true;
     SetTimerEx("unjail", 180000, false, "d", jailedid);
      format(string, sizeof(string), "%s has been jailed 3 minutes", GetPlayerNameEx(jailedid));
     SendClientMessageToAll(COLOR_RED, string);
     SetPlayerPos(jailedid, -27.1455, 2321.1648, 24.3034);
     SetPlayerInterior(jailedid, 0);
     SendClientMessage(jailedid, COLOR_RED,   "You are jailed by an administrator");
     SendClientMessage(jailedid, COLOR_RED, "Please read /rules");
     SendClientMessage(jailedid, COLOR_RED,   "....................................");
     return 1;
} 


PHP Code:

forward unjail(playerid);
public unjail(playerid)
{
  new string[128];
  SendClientMessage(playerid, COLOR_GREEN, "You're free!");
  SetPlayerPos(playerid, -20.8889, 2334.4663, 24.1406);
  SetPlayerInterior(playerid, 0);
  format(string, sizeof(string), "%s now is free", GetPlayerNameEx(playerid));
  SendClientMessageToAll(COLOR_GREEN, string);
  Jailed[playerid] = false;
  return 1;
} 


but when he imprisoned someone who finishes his sentence 3 minutes to try again imprisoned not let me and says "Player is already jailed"

Help please, regards!!

[The__]

In your jail command you're setting playerid Jailed to 1 instead of JailedID.

pawn Code:

Jailed[playerid]=true;

to

pawn Code:

Jailed[jailedid]=true;

[zT KiNgKoNg]

Quote:

Originally Posted by The__ In your jail command you're setting playerid Jailed to 1 instead of JailedID. pawn Code: Jailed[playerid]=true; to pawn Code: Jailed[jailedid]=true;

You're semi-right, He'll be setting the value to {X} where the value of {X} being his/the players current ID.

[JuanStone]

This.

PHP Code:

if(Jailed[playerid] == true) return SendClientMessage(playerid, -1, "Player is already jailed.");
//
Jailed[playerid]=true; 


The correct way would be:

PHP Code:

if(Jailed[jailedid] == true) return SendClientMessage(playerid, -1, "Player is already jailed.");
//
Jailed[jailedid]=true; 


This will work better

PHP Code:

dcmd_jail(playerid, params[])
{
    if(sscanf(params, "u", params[0]))
        return SendClientMessage(playerid, -1, "/jail <playerid>");
    if(params[0] == INVALID_PLAYER_ID)
        return SendClientMessage(playerid, -1, "Player is not online.");
        
    if(Jailed[params[0]] == true)
        return SendClientMessage(playerid, -1, "Player is already jailed.");
    new string[83];
    SetTimerEx("unjail", 180000, false, "d", params[0]);
    format(string, sizeof(string), "%s has been jailed 3 minutes", GetPlayerNameEx(params[0]));
    SendClientMessageToAll(COLOR_RED, string);
    
    SetPlayerInterior(params[0], 0);
    SetPlayerPos(params[0], -27.1455, 2321.1648, 24.3034);
    
    SendClientMessage(params[0], COLOR_RED,   "You are jailed by an administrator");
    SendClientMessage(params[0], COLOR_RED, "Please read /rules");
    SendClientMessage(params[0], COLOR_RED,   "....................................");
    
    Jailed[params[0]] = true;
    return true;
}