Why does this print to chat twice?

Kanyin · 01-11-09, 01:43 AM

local CL = CreateFrame("Frame", "extCombatlog", UIParent)

CL:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED")
CL:SetScript("OnEvent", function()

-- Player vs. Player
-- Druid
if event == "COMBAT_LOG_EVENT_UNFILTERED" then
if arg2 == "SPELL_AURA_APPLIED" then
if arg7 == UnitName("player") then
if arg10 == "Cyclone" then
SendChatMessage("Do Not Dispel: Cyclone[Immune]")
end
end
end
end
end)

So far this addon works. The problem is it performs the SendChatMessage(Do Not Dispel: Cycline[Immune]") part to the chat frame twice. Take into account I know nothing about LUA and I'm tryin to write an addon to learn a little bit. Thanks for any help in advance.

Mera · 01-11-09, 04:53 AM

probably the event happening twice you will have to code it with more precision, first take DevTools and you can now debug events in realtime with /dtevents command.

Kanyin · 01-12-09, 01:33 AM

The event does fire twice but they both have the same calls. Could someone give me some tips on how to write an event handler? I've heard that thrown around to fix problems like this but I have no clue how to do it.

Kanyin · 01-12-09, 04:22 AM

Woohoo figured it out. Don't need help anymore

DDD

Codermik · 01-12-09, 04:30 AM

Originally Posted by Kanyin

local CL = CreateFrame("Frame", "extCombatlog", UIParent)

CL:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED")
CL:SetScript("OnEvent", function()

-- Player vs. Player
-- Druid
if event == "COMBAT_LOG_EVENT_UNFILTERED" then
if arg2 == "SPELL_AURA_APPLIED" then
if arg7 == UnitName("player") then
if arg10 == "Cyclone" then
SendChatMessage("Do Not Dispel: Cyclone[Immune]")
end
end
end
end
end)

So far this addon works. The problem is it performs the SendChatMessage(Do Not Dispel: Cycline[Immune]") part to the chat frame twice. Take into account I know nothing about LUA and I'm tryin to write an addon to learn a little bit. Thanks for any help in advance.

Glad you got it working, thing to note though is that testing for the spell name is not really a suitable way to be honest unless its for personal use only. Remember that in other countries that dont use the english client this test would fail.

Good luck with the coding.