Arena Master Script not working as intended

[chris999]

Hey all. Ok so basically I've made this script where you talk to an NPC who gives out a quest for the arena. If you accept the quest then choose the first option upon talking to him again, you are moved down to the center of the arena and then after 5 seconds THIS SHOULD HAPPEN (but doesn't ): The battlemaster should speak (see code if you want to know what he says) and then spawn an NPC.

--[[
        Arena Battlemaster Script
]]

-- Variables
local NPC_ID = 50000

-- On Triggers

function exampleGossipOnTalk(Unit, Event, player)
        Unit:GossipCreateMenu(100, player, 0)
        Unit:GossipMenuAddItem(0, "Let's do this!", 1, 0)
        Unit:GossipMenuAddItem(0, "Forget it.", 2, 0)
        Unit:GossipSendMenu(player)
end

function exampleGossipOnSelect(Unit, Event, player, id, intid, code, pMisc)
        if (intid == 1)then
            if player:HasQuest(50000) then
                player:MovePlayerTo(-13208.142578, 263.146149, 21.858044, 1.000000, 4096)
                Unit:RegisterEvent("Initiation_RoundOne", 5000, 1)
                player:GossipComplete()
            else
                player:SendBroadcastMessage("You currently have no active arena quest.")
                player:GossipComplete()
            end
        end
        if (intid == 2) then
            player:GossipComplete()
        end
end

function Initiation_RoundOne(Unit, Event)
    Unit:SendChatMessage(14, 0, "Good luck, challenger! Commence round one!!")
    Unit:SpawnCreature(118, -13194.796875, 290.456818, 21.858200, 4.300000, 2, 0)
end

        


RegisterUnitGossipEvent(NPC_ID, 1, "exampleGossipOnTalk")
RegisterUnitGossipEvent(NPC_ID, 2, "exampleGossipOnSelect")
RegisterUnitEvent(NPC_ID, 0, "Initiation_RoundOne")

Any ideas? Thankyou in advance

[The-Eradicator]

I don't know what the issue is as I don't work with Lua serverside, however, try some basic debugging.

Ensure the function is actually firing, and see if the variables match the values you think they're supposed to have:

function exampleGossipOnSelect(Unit, Event, player, id, intid, code, pMisc)
        player:SendBroadcastMessage("exampleGossipOnSelect fired! id: " .. tostring(id) .. "; intid: " .. tostring(intid) .. ";")
end

My apologies if the Lua syntax is incorrect, I don't remember it very well.

[chris999]

Ok, well I just tested both OnSelect possibilities (Let's do this and Forget it) and they both execute properly. The

Unit:RegisterEvent("Initiation_RoundOne", 5000, 1)

however just doesn't seem to pass the registration to

function Initiation_RoundOne(Unit, Event)
    Unit:SendChatMessage(14, 0, "Good luck, challenger! Commence round one!!")
    Unit:SpawnCreature(118, -13194.796875, 290.456818, 21.858200, 4.300000, 2, 0)
end

EDIT: Just changed the Event Registers down the bottom of the script from

RegisterUnitGossipEvent(NPC_ID, 1, "exampleGossipOnTalk")
RegisterUnitGossipEvent(NPC_ID, 2, "exampleGossipOnSelect")
RegisterUnitEvent(NPC_ID, 0, "Initiation_RoundOne")

to

RegisterUnitGossipEvent(NPC_ID, 1, "exampleGossipOnTalk").
RegisterUnitGossipEvent(NPC_ID, 2, "exampleGossipOnSelect")
RegisterUnitGossipEvent(NPC_ID, 2, "Initiation_RoundOne")

Doing this properly passes the SendChatMessage for the NPC and spawns the other NPC as it should, however the player is no longer moved to the center of the arena.

[stoneharry]

A historical error with Lua in emulation is that you can not actually register events inside a gossip event. That function will never be called.

Alternatively, you can do something like this:

local Count = 0

function KaelSummon_OnGossip(pUnit, event, player)
    pUnit:GossipCreateMenu(1260722, player, 0)
    pUnit:GossipMenuAddItem(0, "Kael'thas we are ready.", 4, 0)
    pUnit:GossipMenuAddItem(0, "Nevermind.", 3, 0)
    pUnit:GossipSendMenu(player)
end


function KaelSummon_GossipSubmenus(pUnit, event, player, id, intid, code)
    if(intid == 4) then
        pUnit:SendChatMessage(12, 0, "Stand back, mortals. I will summon the Lord and master.")
        pUnit:SetNPCFlags(2)
        Count = 1
        player:GossipComplete()
    end
    if(intid == 3) then
        player:GossipComplete()
    end
end

RegisterUnitGossipEvent(24855, 1, "KaelSummon_OnGossip")
RegisterUnitGossipEvent(24855, 2, "KaelSummon_GossipSubmenus")

function On_Spawn(pUnit, Event)
    pUnit:RegisterEvent("CheckForGossip", 2000, 0)
end

function CheckForGossip(pUnit)
    if Count == 1 then
    Count = 0
    pUnit:RemoveEvents()
    pUnit:SendChatMessage(12,0,"So, you decided to click my gossip menu, did you?!")
    pUnit:RegisterEvent("NowContinueWithTheScript", 5000, 1)
    end
end

RegisterUnitEvent(24855, 18, "On_Spawn")