LUA help

[thebigman]

When making a custom gossip npc, that adds and deletes spells, I finally made the script and it all worked except:

When you choose an option, It doesnt do what it is supposed, does nothing.

Also it says this above the option menu:

ID:70 is a bad Worldstring TEXT!

help? heres a copy of the lua. I dont think theres anything wrong with it though.

function On_Gossip(unit, event, player)
unit:GossipCreateMenu(3544, player, 0)
unit:GossipMenuAddItem(2, "Remove Rezz Sickness", 1, 0)
unit:GossipMenuAddItem(2, "Remove Deserter", 3, 0)
unit:GossipMenuAddItem(2, "Add Master Hammersmith", 4, 0)
unit:GossipMenuAddItem(2, "Add Master Swordsmith", 5, 0)
unit:GossipMenuAddItem(2, "Add Master Axesmith", 6, 0)
unit:GossipMenuAddItem(2, "Add Goblin Engineering", 7, 0)
unit:GossipMenuAddItem(2, "Add Gnomish Engineering", 8, 0 )
unit:GossipMenuAddItem(2, "Made by Furious / Armageddon of WoW-Hazzard", 999, 3)
unit:GossipSendMenu(player)
end

function Gossip_Submenus(unit, event, player, id, intid, code)
end

if(intid == 3544) then
function On_Gossip(unit, event, player)
unit:GossipCreateMenu(3544, player, 0)
unit:GossipMenuAddItem(2, "Remove Rezz Sickness", 1, 0)
unit:GossipMenuAddItem(2, "Remove Coward", 3, 0)
unit:GossipMenuAddItem(2, "Add Master Hammersmith", 4, 0)
unit:GossipMenuAddItem(2, "Add Master Swordsmith", 5, 0)
unit:GossipMenuAddItem(2, "Add Master Axesmith", 6, 0)
unit:GossipMenuAddItem(2, "Add Goblin Engineering", 7, 0)
unit:GossipMenuAddItem(2, "Add Gnomish Engineering", 8, 0)
unit:GossipSendMenu(player)
end

if(intid == 1) then
player:LearnSpell(15007)
player:UnlearnSpell(15007)
end

if(intid == 3) then
player:LearnSpell(26013)
player:UnlearnSpell(26013)
end

if(intid == 4) then
player:LearnSpell(17040)
player:UnlearnSpell(17041)
player:UnlearnSpell(17039)
end

if(intid == 5) then
player:LearnSpell(17039)
player:UnlearnSpell(17040)
player:UnlearnSpell(17041)
end

if(intid == 6) then
player:LearnSpell(17041)
player:UnlearnSpell(17040)
player:UnlearnSpell(17039)
end

if(intid == 7) then
player:LearnSpell(20221)
player:UnlearnSpell(20220)
end

if(intid == 8) then
player:LearnSpell(20220)
player:UnlearnSpell(20221)
end
end

RegisterUnitGossipEvent(555555, 1, "On_Gossip")
RegisterUnitGossipEvent(555555, 2, "Gossip_Submenus")

[bigjohnson4]

function Gossip_Submenus(unit, event, player, id, intid, code)
end

Don't end the function before you even start it...

[Kaidos]

You need to use a unique ID everytime: so you cant use:

function On_Gossip(unit, event, player)
unit:GossipCreateMenu(3544, player, 0)
unit:GossipMenuAddItem(2, "Remove Rezz Sickness", 1, 0)
unit:GossipMenuAddItem(2, "Remove Deserter", 3, 0)
unit:GossipMenuAddItem(2, "Add Master Hammersmith", 4, 0)
unit:GossipMenuAddItem(2, "Add Master Swordsmith", 5, 0)
unit:GossipMenuAddItem(2, "Add Master Axesmith", 6, 0)
unit:GossipMenuAddItem(2, "Add Goblin Engineering", 7, 0)
unit:GossipMenuAddItem(2, "Add Gnomish Engineering", 8, 0 )
unit:GossipMenuAddItem(2, "Made by Furious / Armageddon of WoW-Hazzard", 999, 3)
unit:GossipSendMenu(player)
end

function Gossip_Submenus(unit, event, player, id, intid, code)
end

if(intid == 3544) then
function On_Gossip(unit, event, player)
unit:GossipCreateMenu(3544, player, 0)
unit:GossipMenuAddItem(2, "Remove Rezz Sickness", 1, 0)
unit:GossipMenuAddItem(2, "Remove Coward", 3, 0)
unit:GossipMenuAddItem(2, "Add Master Hammersmith", 4, 0)
unit:GossipMenuAddItem(2, "Add Master Swordsmith", 5, 0)
unit:GossipMenuAddItem(2, "Add Master Axesmith", 6, 0)
unit:GossipMenuAddItem(2, "Add Goblin Engineering", 7, 0)
unit:GossipMenuAddItem(2, "Add Gnomish Engineering", 8, 0)
unit:GossipSendMenu(player)
end

if(intid == 1) then
player:LearnSpell(15007)
player:UnlearnSpell(15007)
end

if(intid == 3) then
player:LearnSpell(26013)
player:UnlearnSpell(26013)
end

if(intid == 4) then
player:LearnSpell(17040)
player:UnlearnSpell(17041)
player:UnlearnSpell(17039)
end

if(intid == 5) then
player:LearnSpell(17039)
player:UnlearnSpell(17040)
player:UnlearnSpell(17041)
end

if(intid == 6) then
player:LearnSpell(17041)
player:UnlearnSpell(17040)
player:UnlearnSpell(17039)
end

if(intid == 7) then
player:LearnSpell(20221)
player:UnlearnSpell(20220)
end

if(intid == 8) then
player:LearnSpell(20220)
player:UnlearnSpell(20221)
end
end

RegisterUnitGossipEvent(555555, 1, "On_Gossip")
RegisterUnitGossipEvent(555555, 2, "Gossip_Submenus")

from where did you get the 3544?
You need one of the red numbers at the position from the 3544.

[thebigman]

There is a unique id per each one. Any help from the ID:70 wise part? apparently that doesnt work even with a reloaded worldstring

[thebigman]

Don't end the function before you even start it...

If i Dont have an end there the script doesnt work AT ALL and logonserver says it needed an "end" at line 14, which is that line.

edit: shit nvm that was one of the problems, so i added an extra end to the bottom and its ok now.

[thebigman]

ok so the npc works now but the text on it still bothers me, it used to say how can I help you N$? now it says

ID:70 is a bad worldstring TEXT

how do i rid myself of that? it looks tacky.

[jay77]

Repost the entire script, I'll take a look at it.

[Sounddead]

EDIT: Nvm, seems you already fixed your problem with the script

[thebigman]

yea, now all i need is an sql fix ><