What am I doing wrong?

[Cowberty]

Okay. so I'm extremely new when i comes to writing addons. I've spent a lot of time reading up on wow api, but for the life of me i can't figure out what im doing wrong here. Can someone take a look at my code and tell me if i'm doing something wrong with my formatting?

TauntTell.xml

<Ui xmlns="http://www.blizzard.com/wow/ui/" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.blizzard.com/wow/ui/ C:\Projects\WoW\Bin\Interface\FrameXML\UI.xsd">

<Script file="TauntTell.lua"/>

<Frame name="TauntTell_Frame">
	<Scripts>
		<!-- event handler -->
		<OnLoad>TauntTell_OnLoad();</OnLoad>
		<OnEvent>TauntTell_OnEvent(self, event, ...);</OnEvent>
	</Scripts>

</Frame></Ui>

TauntTell.lua

function TauntTell_OnLoad()
	this:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED");
	DEFAULT_CHAT_FRAME:AddMessage("|cffffff00 TauntTell Loaded");
end

function TauntTell_OnEvent(self, event, ...)

	local timestamp, type, sourceGUID, sourceName, sourceFlags, destGUID, destName, destFlags = select(1, ...)

	if (event == "COMBAT_LOG_EVENT_UNFILTERED")
		local spellId, spellName, spellSchool = select(9, ...)
		if (type == "SPELL_CAST_SUCCESS") and ((spellId == 6795) or (spellId == 5209) or (spellId == 355) or (spellId == 1161) or (spellId == 694) or (spellId == 31789) or (spellId == 62124) or (spellId == 56222)) then
			if (destName) then
		  DEFAULT_CHAT_FRAME:AddMessage(sourceName.." taunted "..destName);
			else
		  DEFAULT_CHAT_FRAME:AddMessage(spellName.." used by ".. sourceName);
			end
		end
	end
end

Thanks in advance for the help.

[Akryn]

You should do it this way, for one:

<OnLoad>TauntTell_OnLoad(self)</OnLoad>

function TauntTell_OnLoad(self)
	self:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED")

although the way you have it now should still work. I looked over your code quickly and didn't see an obvious problem. Are you getting any errors in-game? What does your .toc file look like?

[Sythalin]

Originally Posted by Akryn self:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED") although the way you have it now should still work.

Although the change is correct, the latter is not. "this" was removed with 3.0, if I recall correctly, and is no longer recognized

[Akryn]

Originally Posted by ChaosInc Although the change is correct, the latter is not. "this" was removed with 3.0, if I recall correctly, and is no longer recognized

It's deprecated and has been for some time. In 3.0, Blizzard's UI was updated so that it no longer uses it, meaning that addons no longer *have* to use it to make use of some of the Blizzard built-in object templates and such. But AFAIK, all the old globals still exist and should technically be usable.

[Cowberty]

Okay, so sorry if i wasn't entirely clear initially. First off, here's my toc file.

## Interface: 30000
## Title: TauntTell
## Notes: Displays who taunted off you in the default chat frame.
## Author: Tankadyn
## Version: 1.0
## SavedVariablesPerCharacter: TauntTell
TauntTell.xml

As for error messages in-game, I get none. Actually, if i take out the second function, "TauntTell Loaded" shows up in the chat frame as expected. However, when i stick in the second function the "TauntTell Loaded" message goes away and nothing works. (this is why i think there's got to be some kind of formatting issue). As for what this is actually supposed to do, it should see when a taunt lands and print out in chat who taunted what.

Thanks for the feedback.

[Akryn]

Ugh...

if (event == "COMBAT_LOG_EVENT_UNFILTERED")

"then" expected near ')'

If you're not getting an error there, you need to turn Lua errors on in the interface options, and personally I suggest installing a better error handler such as http://www.wowinterface.com/download...5-BugSack.html

[Cowberty]

Ahah! yes! that was the problem. Thanks much sir. (tbh i completely forgot to turn lua errors back on when i was debugging )