A CMD that bugs my server. - Printable Version

A CMD that bugs my server. - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+--- Thread: A CMD that bugs my server. (/showthread.php?tid=367347)

A CMD that bugs my server. - Strech - 09.08.2012

pawn Код:

CMD:heal(playerid, params[])
{
    if(!IsPlayerLoggedIn(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to login first before using any command.");
    if(!IsAMedic(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to be a Medic to use this command.");
    SetPlayerHealth(playerid, 100);
    SetPlayerMoney(playerid, -45);
    SendClientMessage(playerid, COLOR_GREY, "You have healed %s for 45$.");
    return 1;
}

Well i made this command for my new LSFMD faction,but after i type it,my server is restarting.Im using zGaming edit.Any ideas/

Re: A CMD that bugs my server. - PawnFox - 09.08.2012

Shouldn't be GivePlayerMoney( playerid, -45 ); instead of SetPlayerMoney ?

Re: A CMD that bugs my server. - Strech - 09.08.2012

Quote:

Originally Posted by [V]PawnFox

Shouldn't be GivePlayerMoney( playerid, -45 ); instead of SetPlayerMoney ?

It's still restarting.

Re: A CMD that bugs my server. - Jstylezzz - 09.08.2012

so.. let me just check, you are sure of this command?

pawn Код:

CMD:heal(playerid, params[])
{
    if(!IsPlayerLoggedIn(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to login first before using any command.");
    if(!IsAMedic(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to be a Medic to use this command.");
//until here i get it, but this is strange
    SetPlayerHealth(playerid, 100);//you set your own health?
    SetPlayerMoney(playerid, -45);//you take away money from yourself?
    SendClientMessage(playerid, COLOR_GREY, "You have healed %s for 45$.");//%s in a sendclientmessage? this won't work as you want..
    return 1;
}

Use this command:

pawn Код:

CMD:heal(playerid, params[])
{
    if(!IsPlayerLoggedIn(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to login first before using any command.");
    if(!IsAMedic(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to be a Medic to use this command.");
    new id;
    if (sscanf(params, "u", id)) SendClientMessage(playerid, 0xFF0000AA, "Usage: /heal <playerid>");
    if (id == INVALID_PLAYER_ID) return SendClientMessage(playerid, 0xFF0000AA, "Player not found");
    
    SetPlayerHealth(id, 100.0);
    SendClientMessage(id, 0x00FF00AA, "You have been healed by a Medic!");
    SendClientMessage(playerid, 0x00FF00AA, "You have healed the player.");
    
    return 1;
}

use this one, and tell me what happens

Re: A CMD that bugs my server. - Dawnz - 09.08.2012

If its a zGaming edit, it should be like this:

pawn Код:

CMD:heal(playerid, params[])
{
        new playerb, string[128];
    if(!IsPlayerLoggedIn(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to login first before using any command.");
        if(sscanf(params, "u", playerb)) return SendClientMessage(playerid, COLOR_WHITE, "SYNTAX: /heal [playerid]");
    if(!IsAMedic(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to be a Medic to use this command.");
    SetPlayerHealth(playerb, 100);
    SetPlayerMoney(playerb, -45);
        format(string, sizeof(string), "You have healed %s for 45$.", RPN(playerb));
    SendClientMessage(playerid, COLOR_GREY, string);
    return 1;
}

Re: A CMD that bugs my server. - Strech - 09.08.2012

The problem now is that /heal takes 45$ but refunds them back again..

Re: A CMD that bugs my server. - Dawnz - 09.08.2012

Fix:

pawn Код:

CMD:heal(playerid, params[])
{
        new playerb, string[128];
    if(!IsPlayerLoggedIn(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to login first before using any command.");
        if(sscanf(params, "u", playerb)) return SendClientMessage(playerid, COLOR_WHITE, "SYNTAX: /heal [playerid]");
    if(!IsAMedic(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to be a Medic to use this command.");
    SetPlayerHealth(playerb, 100);
    GiveZaiatMoney(playerb, -45);
        format(string, sizeof(string), "You have healed %s for 45$.", RPN(playerb));
    SendClientMessage(playerid, COLOR_GREY, string);
    return 1;
}