[Lua] My first own Boss Script does not work...

[Dark Guild]

Hello everybody,

i have an problem with my own first Boss script. My Boss (namely Pandorra) does nothing. He does only attacking and the arcemu-world console only means

scripts\Pandorra.lua:20: '(' expected near 'Unit'

Here is the script:

function Pandorra_OnCombat(pUnit, Event)
pUnit:SendChatMessage(14, 0, "You insignificant nats cannot stop my rise! My awakening draws near!")
pUnit:RegisterEvent("Pandorra_DeathTouch", 300, 1)
pUnit:RegisterEvent("Pandorra_SoulBleeding", 60, 1)
pUnit:RegisterEvent("Pandorra_Shackle", 40, 1)
pUnit:RegisterEvent("Pandorra_Felstreak", 70, 1)
end

function Pandorra_DeathTouch(pUnit, Event)
pUnit:FullCastSpellOnTarget(5, pUnit:GetRandomPlayer(0))
pUnit:SendChatMessage(14, 8, "Your death is only a stepping stone for the massive fail you all will have.")
end

function Pandorra_SoulBleeding(pUnit, Event)
pUnit:FullCastSpellOnTarget(41303, pUnit:GetRandomPlayer(4))
pUnit:SendChatMessage(12, 0, "Everything drains from you.")
end

function Pandorra_Shackle
pUnit:FullCastSpellOnTarget(38051, pUnit:GetRandomPlayer(7))
pUnit:SendChatMessage(14, 8, "Be shackled!")
end

function Pandorra_FelStreak
pUnit:FullCastSpellOnTarget(67044, pUnit:GetRandomPlayer(0))
end

function Pandorra_SummonMinion
pUnit:CastSpell(38876)
pUnit:SendChatMessage(14, 8, "Minons, assist your master!")
end

function Pandorra_RainofFire
pUnit:CastSpell(47820)
end

function Pandorra_OnLeaveCombat(pUnit, Event)
pUnit:SendChatMessage(14, 0, "So the pUnited forces of Sacredos have failed. Now the hour of my rise comes near."
pUnit:SendChatMessage(14, 8, "Come my minions. Empower you master.")
pUnit:RemoveEvents() 
end

function Pandorra_OnKilledTarget(pUnit, Event)
pUnit:SendChatMessage(14, 0, "Pathetic!")
end

function Pandorra_OnDied(pUnit, Event)
pUnit:RemoveEvents()
pUnit:SendChatMessage(14, 0, "No...NO! How could you weaken me? Why have you the power to imprison me again? WHY?")
end

RegisterUnitEvent(509358, 1, "Pandorra_OnCombat")
RegisterUnitEvent(509358, 2, "Pandorra_OnLeaveCombat")
RegisterUnitEvent(509358, 3, "Pandorra_OnKilledTarget")
RegisterUnitEvent(509358, 4, "Pandorra_OnDied")

Yes, he does not have any phases, but that should be.

Can anyone skilled look over that "noobish" script and can help me to improve the script?
I'm using P1ratens Repack.

And i will +Rep if i can =)

Dark Guild

EDIT: And i have two question:

1. How can i root an npc with lua? Do i need using "plr:root"?
2. How can i add an buff to an NPC?

[gospag]

replace

function Pandorra_Shackle
pUnit:FullCastSpellOnTarget(38051, pUnit:GetRandomPlayer(7))
pUnit:SendChatMessage(14, 8, "Be shackled!")
end

function Pandorra_FelStreak
pUnit:FullCastSpellOnTarget(67044, pUnit:GetRandomPlayer(0))
end

function Pandorra_SummonMinion
pUnit:CastSpell(38876)
pUnit:SendChatMessage(14, 8, "Minons, assist your master!")
end

function Pandorra_RainofFire
pUnit:CastSpell(47820)
end

with

function Pandorra_Shackle(pUnit, Event)
pUnit:FullCastSpellOnTarget(38051, pUnit:GetRandomPlayer(7))
pUnit:SendChatMessage(14, 8, "Be shackled!")
end

function Pandorra_FelStreak(pUnit, Event)
pUnit:FullCastSpellOnTarget(67044, pUnit:GetRandomPlayer(0))
end

function Pandorra_SummonMinion(pUnit, Event)
pUnit:CastSpell(38876)
pUnit:SendChatMessage(14, 8, "Minons, assist your master!")
end

function Pandorra_RainofFire(pUnit, Event)
pUnit:CastSpell(47820)
end

[P1raten]

replace

function Pandorra_Shackle
pUnit:FullCastSpellOnTarget(38051, pUnit:GetRandomPlayer(7))
pUnit:SendChatMessage(14, 8, "Be shackled!")
end

function Pandorra_FelStreak
pUnit:FullCastSpellOnTarget(67044, pUnit:GetRandomPlayer(0))
end

function Pandorra_SummonMinion
pUnit:CastSpell(38876)
pUnit:SendChatMessage(14, 8, "Minons, assist your master!")
end

function Pandorra_RainofFire
pUnit:CastSpell(47820)
end

with

function Pandorra_Shackle(pUnit, Event)
pUnit:FullCastSpellOnTarget(38051, pUnit:GetRandomPlayer(7))
pUnit:SendChatMessage(14, 8, "Be shackled!")
end

function Pandorra_FelStreak(pUnit, Event)
pUnit:FullCastSpellOnTarget(67044, pUnit:GetRandomPlayer(0))
end

function Pandorra_SummonMinion(pUnit, Event)
pUnit:CastSpell(38876)
pUnit:SendChatMessage(14, 8, "Minons, assist your master!")
end

function Pandorra_RainofFire(pUnit, Event)
pUnit:CastSpell(47820)
end

^this

You need to always use parameters.

[Ground Zero]

You can root the npc with either pUnit:Root() and make the npc casts buffs on itself by pUnit:CastSpell(xxxx) or on players with player:CastSpell(xxxx) you can also make the NPC cast it on a target by defining the target and using pUnit:CastSpellOnTarget(spell, target)

[Dark Guild]

/facepalm

Oh my god... why i haven't saw that i've forgotten the parameters? Well, thanks for you help and thanks for your answers, GroundZero, but i have a new question:

Pandorra shall cast FelStreak every 30 Seconds. But he does cast it only once after 30 Seconds an then never again.

How can i let him cast FelStreak every 30 Seconds? By modifying the parameters?

[Reflection]

I'm not a lua expert so I can only tell you what to do, not how to do it. When you register a function you can define the amount of time the spells are cast I believe. Someone else would have to come help you with the real code but that's the way to do it. At least I think so, ha.

[Ground Zero]

You can register a new function like so

unit:RegisterEvent(functionname, time, repeat)
so
unit:RegisterEvent("recast", 30000, 0)

function recast (unit, event)
unit:CastSpell(ID)
end

[RyeRye]

You always have to make sure you have some type of ().