New to LUA need help with script

[nickelo]

 function phase_1(pUnit, Event)
if pUnit:GetHealthPct() < 85 then
pUnit:RemoveEvents()
pUnit:GetMainTank()
pUnit:SendChatMessage (11, 0, "THE SCOURGE IS DOMINANT!")
pUnit:CastSpellOnTarget(40876)
pUnit:RegisterEvent("phase_2",1000, 0)
end
end

function phase_2(pUnit, Event)
if pUnit:GetHealthPct() < 85 then
pUnit:GetMainTank()
pUnit:RemoveEvents()
pUnit:CastSpellOnTarget(40876)
end
end

function Boss_Start(pUnit, Event)
pUnit:RegisterEvent("phase_1",1000, 0)
pUnit:SendChatMessage (11, 0, "YOU ARE A FOOL TO CHALLENGE THE DAMNED!")
end
RegisterUnitEvent(99012, 1, "Boss_Start")

However at 85% he casts both spells on himself, what did i do wrong?

[TheSpidey]

pUnit:GetMainTank() is useless if you don't use the value it gives you. CastSpellOnTarget expects two parameters: Spell and target.


pUnit:CastSpellOnTarget(40876, pUnit:GetMainTank())

[BrantX]

Try this

 function phase_1(pUnit, Event)
if pUnit:GetHealthPct() < 85 then
pUnit:RemoveEvents()
pUnit:GetMainTank()
pUnit:SendChatMessage (11, 0, "THE SCOURGE IS DOMINANT!")
pUnit:CastSpellOnTarget(40876,pUnit:GetClosestPlayer())
pUnit:RegisterEvent("phase_2",1000, 0)
end
end

function phase_2(pUnit, Event)
if pUnit:GetHealthPct() < 85 then
pUnit:GetMainTank()
pUnit:RemoveEvents()
pUnit:CastSpellOnTarget(40876,pUnit:GetClosestPlayer())
end
end

function Boss_Start(pUnit, Event)
pUnit:RegisterEvent("phase_1",1000, 0)
pUnit:SendChatMessage (11, 0, "YOU ARE A FOOL TO CHALLENGE THE DAMNED!")
end
RegisterUnitEvent(99012, 1, "Boss_Start")

[TheSpidey]

Again, why call pUnit:GetMainTank() without storing it in a var or something? :P

[nickelo]

true, thanks guys +rep 2 both