Need help on function for new combat log stuffs..

[halpmeh]

function Innervate_OnEvent(e)
	if (e == "COMBAT_LOG_EVENT_UNFILTERED") then
		local event,destName = arg2,arg7
		if(event == "SPELL_AURA_APPLIED") then
			local spellID = arg10
			local isEnemy = (bit.band(arg5, COMBATLOG_OBJECT_REACTION_HOSTILE) == COMBATLOG_OBJECT_REACTION_HOSTILE)
			
			spell,_,_ = GetSpellInfo(spellID)
			if (spell == "Innervate" and isEnemy) then
				ZoneTextString:SetText("INNERVATE!!");
				ZoneTextFrame.startTime = GetTime()
				ZoneTextString:SetTextColor(0, 1, 0);
				PVPInfoTextString:SetText("");
				ZoneTextFrame:Show()
				
			end
		end
	end
end

Alright, the code is fairly straightforward, I'm trying to set it up to announce when "Innervate" ( http://www.wowhead.com/?spell=29166 ) is gained on an enemy.
It doesn't seem to be working and I believe I've narrowed down my problem to

if(event == "SPELL_AURA_APPLIED") then
			local spellID = arg10

but it could be wrong.

Ideally, I would like to just search for the spell using spellID's instead of SpellName but every time i try it fails. Can anyone shine some light on this for me pretty please? I'm not sure where I'm messing up but I know I am because it's not working and it's really buggin the heck outta me Thanks !

[Akryn]

have you tried:

spell,_,_ = GetSpellInfo(spellID)
ChatFrame1:AddMessage(spell)

?

btw the _,_ isn't necessary since you only want the first return value
also i don't think there's any disadvantage to just using the spellName that you get from COMBAT_LOG_EVENT_UNFILTERED.

[halpmeh]

Originally Posted by Akryn also i don't think there's any disadvantage to just using the spellName that you get from COMBAT_LOG_EVENT_UNFILTERED.

Thanks for the advice.

Could you provide a quick example on how i could just use the spellName instead of my current method?

regards

[Maccaa]

Have you tested to make sure SpellID is arg10, I thought it was arg9

[Akryn]

Originally Posted by Maccaa Have you tested to make sure SpellID is arg10, I thought it was arg9

doh, this. i should have checked that. :P

arg10 is spellName

[Maccaa]

Another error i believe is in your isEnemy check.

Checking it against srcFlags rather than dstFlags.

here is some code that works for me.

function Innervate_OnEvent(self, event, ...)
	if event == "COMBAT_LOG_EVENT_UNFILTERED" then
		local _, cmbEvent, _, _, _, _, dstName, dstFlags = select(1, ...)
		if cmbEvent == "SPELL_AURA_APPLIED" then
			local isEnemy = (bit.band(dstFlags, COMBATLOG_OBJECT_REACTION_HOSTILE) == COMBATLOG_OBJECT_REACTION_HOSTILE)
			local spellID = arg9
			if spellID == 29166 --[[Innervate]] and isEnemy then
				ChatFrame1:AddMessage("Vate Casted on "..dstName)
			end
		end
	end
end

[halpmeh]

Originally Posted by Maccaa Another error i believe is in your isEnemy check. Checking it against srcFlags rather than dstFlags. here is some code that works for me. function Innervate_OnEvent(self, event, ...) if event == "COMBAT_LOG_EVENT_UNFILTERED" then local _, cmbEvent, _, _, _, _, dstName, dstFlags = select(1, ...) if cmbEvent == "SPELL_AURA_APPLIED" then local isEnemy = (bit.band(dstFlags, COMBATLOG_OBJECT_REACTION_HOSTILE) == COMBATLOG_OBJECT_REACTION_HOSTILE) local spellID = arg9 if spellID == 29166 --[[Innervate]] and isEnemy then ChatFrame1:AddMessage("Vate Casted on "..dstName) end end end end

Sent ya a msg, thx for the help guys.