First Addon - need help

[opismahname]

Hey there, I'm making my first AddOn that will be a real simple one. The thing i want this AddOn to do is to print "Rigtheous Fury is now activated" in the chat window when you cast Righteous Fury.

I've been reading alot about how to do something like this but still can't get it...

The code I am doing is like this (i know, i suck at this, but it's like my first time):

local frame = CreateFrame("FRAME", "RFFrame");
frame:RegisterEvent("UNIT_AURA_PLAYER_RIGHTEOUS FURY");
local function eventHandler(self, event, ...)
 print("Rigtheous Fury is now activated " .. event);
end
frame:SetScript("OnEvent", eventHandler);

I know that this isn't right at all, but don't know what to do

Really need help...

[Akryn]

You are pretty close:

"UNIT_AURA_PLAYER_RIGHTEOUS FURY" is not an event. "UNIT_AURA" is, and that's all you should put there.

Therefore, in your OnEvent function, you need to check for...
1. Is this event firing for the player
2. If so, does the player currently have the aura.
3. If so, did the player not have it the last time I checked -- to prevent the addon from spamming every time you gain/loose an aura while that one is active.

The way to do #1 is to check the event's arg1, which is the unit the event fired for e.g.

if ... == "player" then

The way to do #2 is with UnitAura e.g.

UnitAura("player", "Righteous Fury")

The way to do #3 is to save the result of each check in a variable, and check it before announcing.

[opismahname]

Originally Posted by Akryn You are pretty close: "UNIT_AURA_PLAYER_RIGHTEOUS FURY" is not an event. "UNIT_AURA" is, and that's all you should put there. Therefore, in your OnEvent function, you need to check for... 1. Is this event firing for the player 2. If so, does the player currently have the aura. 3. If so, did the player not have it the last time I checked -- to prevent the addon from spamming every time you gain/loose an aura while that one is active. The way to do #1 is to check the event's arg1, which is the unit the event fired for e.g. if ... == "player" then The way to do #2 is with UnitAura e.g. UnitAura("player", "Righteous Fury") The way to do #3 is to save the result of each check in a variable, and check it before announcing.

Thanks for answering!
Don't really know how to do this work but something like this?

local frame = CreateFrame("FRAME", "RFFrame");
frame:RegisterEvent("UNIT_AURA");
local function eventHandler(self, event, ...)
 print("Rigtheous Fury is now activated " .. event);
end
frame:SetScript("OnEvent *something here?*", eventHandler);

Or could you post how the entire code would be?

[Akryn]

Originally Posted by opismahname Or could you post how the entire code would be?

I would be happy to if you'd like; although wouldn't that invalidate the point of trying to make your own addon?

Otherwise:

What you want to do is put some checks in your OnEvent function itself, the function that you call eventHandler.

frame:SetScript("OnEvent", eventHandler)

and

frame:RegisterEvent("UNIT_AURA");

are both already correct as they are. You need to put some logic in eventHandler to detect the three things I mentioned above:

local function eventHandler(self, event, ...)
--some stuff goes here
--your existing code
end

[opismahname]

Hmm... Think I at least get a little bit of what you wrote.

Will it be like this?

local frame = CreateFrame("FRAME", "RFFrame");
frame:RegisterEvent("UNIT_AURA");
local function eventHandler(self, event, ...)
 print("Rigtheous Fury is now activated " .. event);
end
frame:SetScript("OnEvent", eventHandler);
if Rigtheous Fury (or is it supposed to be...?) == "player" then
**Didnt't get "--your existing code, or do you mean everything that i already have written?"**
end

Haven't coded like this or something before so I am really bad at this :P

[Akryn]

What I mean is, that what you need to add is inside the function e.g.

local frame = CreateFrame("FRAME", "RFFrame");
frame:RegisterEvent("UNIT_AURA");
local function eventHandler(self, event, ...)
 --ADD YOUR CODE HERE
 print("Rigtheous Fury is now activated " .. event);
end
frame:SetScript("OnEvent", eventHandler);

Given what you've posted so far, you might want to start with something like http://www.wowwiki.com/AddOn_program...l/Introduction