[Rezant]

I've been working on an LUA script for a friend's server lately and I can't fix an issue. The NPC is supposed to basically do everything needed: Teleporting, Proffessions, Skills, Banking, AH, etc. The one thing I'm having trouble with is the shop part of the script; the shop will not be displayed to the player. I am getting no errors in the console or in game.

function AllNPC_OnGossip(pUnit, event, player)
	if (player:IsInCombat() == true) then
		pUnit:SendBroadcastMessage("You are in combat!")
	end
	if(player:IsInCombat() == false) then
		pUnit:VendorRemoveAllItems()
		pUnit:GossipCreateMenu(100, player, 0)
		pUnit:GossipMenuAddItem(0, "Teleports", 1, 0)
		pUnit:GossipMenuAddItem(0, "Shop", 2, 0)
		pUnit:GossipMenuAddItem(0, "Proffessions", 3, 0)
		pUnit:GossipMenuAddItem(0, "Training", 4, 0)
		pUnit:GossipMenuAddItem(0, "Bank", 5, 0)
		pUnit:GossipMenuAddItem(0, "Set Hearthstone", 6, 0)
		if (player:HasAura(15007) == true) then
			pUnit:GossipMenuAddItem(0, "Remove Rez Sickness", 7, 0)
		end
		pUnit:GossipMenuAddItem(0, "Reset Talents", 8, 0)
		if(pUnit:GetPlayerClass(player) == 9) then
			pUnit:GossipMenuAddItem(0, "Reset Pet Talents", 10, 0)
		end
		pUnit:GossipSendMenu(player)
	end
end
function AllNPC_OnSelect(pUnit, event, player, id, intid, code)
        local class = pUnit:GetPlayerClass(player)
	if(intid == 2) then
		pUnit:VendorRemoveAllItems()
		pUnit:VendorAddItem(51344,1,2329)
		player:SendVendorWindow(pUnit)
	end	
end

RegisterUnitGossipEvent(npcid, 1, "AllNPC_OnGossip")
RegisterUnitGossipEvent(npcid, 2, "AllNPC_OnSelect")

Any assistance with the SendVendorWindow command would be appreciated.

[Cursed_Ace]

yeah I have the same problem
nothing happens if i klick the menuitem
but it in the world console it says something like: player is a nil value
if the script is player:SendVendorWindow(unit)
or: unit is a nil value
if the script is unit:SendVendorWindow(player)
and so on.. I think you get the point

it doesn't change anything if I make a vendor-table for the npc or if I just give it to him by .npc vendoradditem .. ..
it also doesn't change anything if I set his flags to 128 (vendor) or 129 (vendor + Gossip)

help please!

[Foereaper]

Try player:SendVendorWindow(player)

[kjanko]

Shouldn't this player:SendVendorWindow(pUnit)
be player:SendVendorWindow()
or pUnit:SendVendorWindow(player) ?

[Foereaper]

its player:SendVendorWindow(pUnit)

I googled for another script and found This, tried it out real quick, and it worked all good.. So according to his script, it should technically work

[Cursed_Ace]

WoOhOo
when I try what Foereaper said the world crashes
when I try your methods, Kjanko nothing happens
but then I tested Rezant's method and it's just like foereaper says:

	if(intid == 1)then
		player:SendVendorWindow(pUnit)
		player:GossipComplete(player)
	end

works


thx guys!

But y wouldn't it work with Rezant?

[Foereaper]

Must be something else on his side, maybe an outdated LUA engine or something, I'm not really sure why it doesn't work for him