NPC Lua script not working. (For unknown reason)

[Joakim Karbing]

Dialogue/Spells wont trigger. No errors in world.exe. (Using ArcEmu)

Any suggestions? ;(

local Name = Pontus
local NPCID = 100001

---------------Standard---------------------
function Pontus_OnCombat(pUnit, event, player)
pUnit:SendChatMessage(12, 0, "Ni kommer försent!")
pUnit:RegisterEvent("Pontus_Phase1", 1000, 1)
end

function Pontus_OnLeaveCombat(pUnit, event, player)
pUnit:SendChatMessage(14, 0, "Har ni gjort färdigt era gymnasiearbeten?")
pUnit:RemoveEvents()
end

function Pontus_OnDeath(pUnit, event, player)
pUnit:SendChatMessage(12, 0, "Det här ska ni skita i...")
pUnit:RemoveEvents()
end
----------------------------------------------
--------------------Phases--------------------
function Pontus_Phase1(pUnit, event, player)
if pUnit:GetHealthPct() == 80 then
pUnit:SendChatMessage(12, 0, "Ni slösar bort eran tid!")
pUnit:RegisterEvent("Pontus_RandomAoeGround", 2000, 0)
pUnit:SetCombatCapable(1)
pUnit:ChannelSpell(47855, pUnit:GetCreatureNearestCoords(2759.096680, -3096.876953, 268.675476, 100003))
pUnit:RegisterEvent("Pontus_Phase2", 1000, 1)
end
end

function Pontus_Phase2(pUnit, event, player)
if pUnit:GetHealthPct() == 60 then
pUnit:SendChatMessage(12, 0, "Håll Käften!")
pUnit:CastSpell(64189)
pUnit:SetCombatCapable(0)
pUnit:RegisterEvent("Pontus_Phase3", 1000, 1)
end
end

function Pontus_Phase3(pUnit, event, player)
if pUnit:GetHealthPct() == 40 then
pUnit:SendChatMessage(12, 0, "Ni kommer få indraget CSN!")
pUnit:ChannelSpell(47855, pUnit:GetCreatureNearestCoords(2759.284668, -3103.452881, 268.674347, 100003))
pUnit:RegisterEvent("Pontus_RandomAoeGround", 2000, 0)
pUnit:RegisterEvent("Pontus_Volley", 10000, 0)
pUnit:CastSpell(57381)
pUnit:RegisterEvent("Pontus_Phase4", 1000, 1)
pUnit:SetCombatCapable(1)
end
end

function Pontus_Phase4(pUnit, event, player)
if pUnit:GetHealthPct() == 20 then
pUnit:SendChatMessage(12, 0, "SKA JAG TA KONTAKT MED ERA FÖRÄLDRAR?")
pUnit:ChannelSpell(47855, pUnit:GetCreatureNearestCoords(2759.408448, -3098.933350, 286.500397, 100003))
pUnit:RegisterEvent("Pontus_RandomAoeGround", 2000, 0)
pUnit:CastSpell(68872)
pUnit:SetCombatCapable(0)
end
end
-----------------------------------------------

------------------------Spells----------------
function Pontus_RandomAoeGround(pUnit, event, player)
pUnit:FullCastSpellOnTarget(50752, pUnit:GetRandomPlayer(0))
end

function Pontus_Volley(pUnit, event, player)
pUnit:CastSpell(57381)
end
----------------------------------------------
RegisterUnitEvent(100001, 1, "Pontus_OnCombat")
RegisterUnitEvent(100001, 2, "Pontus_OnLeaveCombat")
RegisterUnitEvent(100001, 4, "Pontus_OnDeath")

[Joakim Karbing]

fixed it.

the problem was this: pUnit:GetCreatureNearestCoords
was supposed to be only Unit:GetCreatureNearestCoords

[Joakim Karbing]

But only the first OnCombat part worked, nothing else triggers.

[stoneharry]

I hate to go off topic like this but, I'm going to anyway.

ArcEmu is really bad and hasn't even been developed for going on 5 years it must be now. The Lua engine has known stability issues that were never fixed.

TrinityCore is pretty much the most you could ever ask for out of a emulator. 3.3.5a default checkout can run 2000 players stable. With the Eluna project you can even have a Lua engine, which is what I use. For example, something I am working on at the moment for a scaling health system:

local playerDistance = 80
local minPlayerScaling = 5

function Creature:GetScaledDamage(damage)
	local p = self:GetPlayersInRange(playerDistance)
	if not p or #p < 5 then
		return damage * 3.5
	end
	return damage * (1 + (#p / 2))
end

--[[
function Creature:GetMaxScaledHealth()
	local hp = maxHealthMap[self:GetGUIDLow()]
	local p = self:GetPlayersInRange(playerDistance)
	if not p or #p < 5 then
		return hp * 5
	end
	return hp * (1 + (#p / 2))
end
]]

function Creature:UpdateScaledHealth()
	local scaling = self:GetValue("currentScaling") or 1
	local maxHealth = self:GetMaxHealth() / scaling;
	local currentHealth = self:GetHealth() / scaling;
	local players = #(self:GetPlayersInRange(playerDistance) or {});
	scaling = math.max(minPlayerScaling, 1 + players / 2);
	self:SetMaxHealth(maxHealth * scaling);
	self:SetHealth(currentHealth * scaling);
	self:SetValue("currentScaling", scaling);
end

function Creature:RegisterScalingHealth(hp)
	if self then
		local guid = self:GetGUID()
		table.insert(healthMap, guid)
	end
end

local function ScaleCreatureHealths(_, _, _)
	for _,v in pairs(healthMap) do
		local pUnit = GetCreatureByGUID(v)
		if pUnit and pUnit:IsAlive() then
			pUnit:UpdateScaledHealth()
		end
	end
end

CreateLuaEvent(ScaleCreatureHealths, 5000, 0)

[Tikki100]

As Stoneharry said, I would recommend Eluna as well

I did notice something in your code, however

pUnit:RegisterEvent("Pontus_Phase1", 1000, 1)

That command calls the Phase1 only once, after 1 second. When it jumps down to Phase 1, the unit hasn't lost enough health, and the script stops there.

Change 1 to a 0, so that it becomes:

pUnit:RegisterEvent("Pontus_Phase1", 1000, 0)

This should also be changed in

pUnit:RegisterEvent("Pontus_Phase2", 1000, 1)

to

pUnit:RegisterEvent("Pontus_Phase2", 1000, 0)

You should also include a RemoveEvents, so that the phaes stops being called, once a new phase is activated.

Try this (changed marked in red):

local Name = "Pontus" --Is there any reason, that this is a variable? If so, it should be a string. 
local NPCID = 100001

---------------Standard---------------------
function Pontus_OnCombat(pUnit, event, player)
pUnit:SendChatMessage(12, 0, "Ni kommer försent!")
pUnit:RegisterEvent("Pontus_Phase1", 1000, 0)
end

function Pontus_OnLeaveCombat(pUnit, event, player)
pUnit:SendChatMessage(14, 0, "Har ni gjort färdigt era gymnasiearbeten?")
pUnit:RemoveEvents()
end

function Pontus_OnDeath(pUnit, event, player)
pUnit:SendChatMessage(12, 0, "Det här ska ni skita i...")
pUnit:RemoveEvents()
end
----------------------------------------------
--------------------Phases--------------------
function Pontus_Phase1(pUnit, event, player)
if pUnit:GetHealthPct() == 80 then
pUnit:RemoveEvents() --To prevent the function to be called, once we go to phase 2.
pUnit:SendChatMessage(12, 0, "Ni slösar bort eran tid!")
pUnit:RegisterEvent("Pontus_RandomAoeGround", 2000, 0)
pUnit:SetCombatCapable(1)
pUnit:ChannelSpell(47855, pUnit:GetCreatureNearestCoords(2759.096680, -3096.876953, 268.675476, 100003))
pUnit:RegisterEvent("Pontus_Phase2", 1000, 0)
end
end

function Pontus_Phase2(pUnit, event, player)
if pUnit:GetHealthPct() == 60 then
pUnit:SendChatMessage(12, 0, "Håll Käften!")
pUnit:CastSpell(64189)
pUnit:SetCombatCapable(0)
pUnit:RegisterEvent("Pontus_Phase3", 1000, 0)
end
end

function Pontus_Phase3(pUnit, event, player)
if pUnit:GetHealthPct() == 40 then
pUnit:SendChatMessage(12, 0, "Ni kommer få indraget CSN!")
pUnit:ChannelSpell(47855, pUnit:GetCreatureNearestCoords(2759.284668, -3103.452881, 268.674347, 100003))
pUnit:RegisterEvent("Pontus_RandomAoeGround", 2000, 0)
pUnit:RegisterEvent("Pontus_Volley", 10000, 0)
pUnit:CastSpell(57381)
pUnit:RegisterEvent("Pontus_Phase4", 1000, 0)
pUnit:SetCombatCapable(1)
end
end

function Pontus_Phase4(pUnit, event, player)
if pUnit:GetHealthPct() == 20 then
pUnit:SendChatMessage(12, 0, "SKA JAG TA KONTAKT MED ERA FÖRÄLDRAR?")
pUnit:ChannelSpell(47855, pUnit:GetCreatureNearestCoords(2759.408448, -3098.933350, 286.500397, 100003))
pUnit:RegisterEvent("Pontus_RandomAoeGround", 2000, 0)
pUnit:CastSpell(68872)
pUnit:SetCombatCapable(0)
end
end
-----------------------------------------------

------------------------Spells----------------
function Pontus_RandomAoeGround(pUnit, event, player)
pUnit:FullCastSpellOnTarget(50752, pUnit:GetRandomPlayer(0))
end

function Pontus_Volley(pUnit, event, player)
pUnit:CastSpell(57381)
end
----------------------------------------------
RegisterUnitEvent(100001, 1, "Pontus_OnCombat")
RegisterUnitEvent(100001, 2, "Pontus_OnLeaveCombat")
RegisterUnitEvent(100001, 4, "Pontus_OnDeath")