PK Point Npc [5165]

[ryski1]

Hi, I Was wondering if you could help me with this, I want to make an NPC for it to take away pks for 20cps, i got this so far:

#region PKPointsNPC
case 10065:
{
    if (Control == 0)
    {

        GC.AddSend(Packets.NPCSay("You have pkpoints let me take 30pks from you for the price of 20cps."));
        GC.AddSend(Packets.NPCLink("Yes please", 1));
        GC.AddSend(Packets.NPCLink("No Thanks", 255));
        GC.AddSend(Packets.NPCSetFace(N.Avatar));
        GC.AddSend(Packets.NPCFinish());
    }

    if (Control == 1)
    {
        if (GC.MyChar.CPs >= 20)
            GC.MyChar.CPs -= 20;
        GC.MyChar.PKPoints -= 30;
    }

    break;
}
#endregion

But if u have less than 30pkps it gives you 60k something pk points, i have searched for the working code and then my last option is to ask here, or even if you can help me make the Penitence amulet work it would be much appreciated

~Ryski

[Zeroxelli]

Change

        if (GC.MyChar.CPs >= 20)
            GC.MyChar.CPs -= 20;
        GC.MyChar.PKPoints -= 30;

to

    if (GC.MyChar.PKPoints >= 30 && GC.MyChar.CPs >= 20)
    {
        GC.MyChar.CPs -= 20;
        GC.MyChar.PKPoints -= 30;
    }

[I don't have a username]

And use a switch for your options as well.

[ryski1]

Originally Posted by Zeroxelli

Change

        if (GC.MyChar.CPs >= 20)
            GC.MyChar.CPs -= 20;
        GC.MyChar.PKPoints -= 30;

to

    if (GC.MyChar.PKPoints >= 30 && GC.MyChar.CPs >= 20)
    {
        GC.MyChar.CPs -= 20;
        GC.MyChar.PKPoints -= 30;
    }

Thanks mate this works