Can anyone tell me if this is right?

[matyj]

if (CurrentNPC == XXXXXX)
{
SendPacket(General.MyPackets.NPCSay("Hello,Welcome to BlizzardCo,you are no longer a stranger to the world, Can you do me a favor?"));
SendPacket(General.MyPackets.NPCLink("Yes", 1));
SendPacket(General.MyPackets.NPCLink("No", 255));
SendPacket(General.MyPackets.NPCSay("Please dilever this letter to Jade in BirdIsland A.S.A.P"));
SendPacket(General.MyPackets.NPCLink("Okay", 1));
SendPacket(General.MyPackets.NPCSetFace(30));
SendPacket(General.MyPackets.NPCFinish());
}

if (CurrentNPC == XXXXXX)
{
MyChar.AddItem("721115-0-0-0-0-0", 0, (uint)General.Rand.Next(36457836));
}
}

Please, this is a quest npc, and idk if i am doing right, first time i really made a npc talk, and give things.

 

[ryuchetval]

Put this on region NPC say or something like this.

if (CurrentNPC == XXXXXX)
{
SendPacket(General.MyPackets.NPCSay("Hello,Welcome to BlizzardCo,you are no longer a stranger to the world, Can you do me a favor?"));
SendPacket(General.MyPackets.NPCLink("Yes", 1));
SendPacket(General.MyPackets.NPCLink("No", 255));
SendPacket(General.MyPackets.NPCSetFace(30));
SendPacket(General.MyPackets.NPCFinish());
}

Then on NPC do region put this:

if (CurrentNPC == XXXXXX)
{
[SIZE=2][COLOR=#800000][/COLOR][/SIZE][SIZE=2][COLOR=#0000ff]if[/COLOR][/SIZE][SIZE=2][COLOR=#800000] (Control == 1)
[/COLOR][/SIZE]{
SendPacket(General.MyPackets.NPCSay("Please dilever this letter to Jade in BirdIsland A.S.A.P"));
SendPacket(General.MyPackets.NPCLink("Okay", 2));
SendPacket(General.MyPackets.NPCSetFace(30));
SendPacket(General.MyPackets.NPCFinish());
}
[COLOR=#0000ff]if[/COLOR][SIZE=2][COLOR=#800000] (Control == 2)[/COLOR][/SIZE]
[COLOR=#800000]{[/COLOR]
MyChar.AddItem("721115-0-0-0-0-0", 0, (uint)General.Rand.Next(36457836));
}
}

I don't know if this is very good but i think it is

 

[petertoft]

Looks good to me just copy and past and edit lmao

 

[Ne[o]x]

Nah it cant be copy and pasted, b cuz you need to replace some lines etc etc


Cya