New to this, need help writing a simple addon.

[Eetabeetay]

Ok, my goal is to write an addon that types in raid chat whenever someone has died, but with nicknames for each person. Its meant as a joke addon to make fun of people who die in our raids. I also want it to be spam proof in case we wipe, so like have it auto disable itself for 30 seconds when 5 people die in 5 seconds or less. I've been trying to figure this out on wowwiki, but I'm still really confused.
All I have so far is a .toc file with this:
## Interface: 30300
## Title: (addonname)
## Notes: (addonname) has died.
(addonname).lua
(addonname).xml

i removed the addon name because its named after someone who I need to keep anonymous.

[Xubera]

well lets say in your xml file you have this

*.xml

...

<Frame name="(addonname)Frame">
    <Scripts>
        <OnLoad>self:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED")
        <OnEvent>(addonname)_OnEvent(self,event,...)</OnEvent>
    </Scripts>
</Frame>

...

and for your *.lua you wanna do

*.lua

local nicknames {}
  nickname["Actualname"] = "Fluffypinkbunnyman"
  nickname["Xubera"] = "Xub"

local counter = 0
local firstDeath = false

function (addonname)_OnEvent(self,event,...)
    local timestamp, type, sourceGUID, sourceName, sourceFlags, destGUID, destName, destFlags = ...

    if type == "UNIT_DIED" then
       if nickname[destName] and counter <= 5 then
           SendChatMessage("EL OH EL!!! "..nickname[destName].." just got roflpwned by that boss guys!!", GetNumRaidMembers()>0 and "RAID" or GetNumPartyMembers()>0 and "PARTY" or "SAY")
           counter = counter + 1
       end
       if not firstDeath then
           firstDeath = GetTime()
       elseif firstDeath and firstDeath + 300 > GetTime() then --if its been 5 minutes since the first death, reset the counter and clock
           firstDeath = false
           counter = 0
       end
   end
end

I havent tested this, and if you have any questions, please feel free to ask me what ive done, why ive done it, and whats the point

**side note, in the .xml file, i left out the <Ui> tags, if you need helps on those too, just ask

[Eetabeetay]

First of all, thanks for the help, but sadly it's not working for me.

is this supposed to be plural?
this is whats called a user defined function right?

local nicknames {}

also I was reading on wowwiki, trying to learn some more stuff. I came across this page - http://www.wowwiki.com/API_COMBAT_LOG_EVENT . is this unnecessary or did it just get left out?

if (event=="COMBAT_LOG_EVENT_UNFILTERED") then

I'm probably completely wrong, but I changed it up to this. Its still not working so I'm guessing I'm wrong.

*.lua

local nickname {}
  nickname["Eeta"] = "Rollingeeta"

local counter = 0
local firstDeath = false

function Rollingpally_OnLoad()
  this:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED");
end


function (Rollingpally)_OnEvent(self,event,...)
    local timestamp, type, sourceGUID, sourceName, sourceFlags, destGUID, destName, destFlags = ...

	if (event=="COMBAT_LOG_EVENT_UNFILTERED") then
	 if (type=="UNIT_DIED") then
       if nickname[destName] and counter <= 5 then
           SendChatMessage(..nickname[destName].." has died.", GetNumRaidMembers()>0 and "RAID" or GetNumPartyMembers()>0 and "PARTY" or "SAY")
           counter = counter + 1
       end
       if not firstDeath then
           firstDeath = GetTime()
       elseif firstDeath and firstDeath + 300 > GetTime() then --if its been 5 minutes since the first death, reset the counter and clock
           firstDeath = false
           counter = 0
       end
    end
   end
end

*.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/
..\..\FrameXML\UI.xsd">
<Frame name="(addonname)Frame">
    <Scripts>
        <OnLoad>self:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED")
        <OnEvent>(addonname)_OnEvent(self,event,...)</OnEvent>
    </Scripts>
</Frame>
</Ui>

[Xubera]

actually im wrong lol, it should be

local nicknames = {}

i guess i forgot the equals sign, okay imma look through the rest and edit the post in a bit

[Xubera]

Originally Posted by Eetabeetay First of all, thanks for the help, but sadly it's not working for me. is this supposed to be plural? this is whats called a user defined function right? local nicknames {}

alright, it was my bad,

local nicknames = {}

is the correct form, its a variable, we could name it whatever we want... we could name it "local cheezecake = {}" but that would just be silly.

also I was reading on wowwiki, trying to learn some more stuff. I came across this page - http://www.wowwiki.com/API_COMBAT_LOG_EVENT . is this unnecessary or did it just get left out? if (event=="COMBAT_LOG_EVENT_UNFILTERED") then

It isnt necessary. its really used if you register for more than one event. Like say we also registered for CHAT_MSG_SAY... we would use the if statement to say, okay if the event was from the combat log, do this, if it was from a /say, then do this.




*.lua

--local nickname {}
local nickname = {} 
  nickname["Eeta"] = "Rollingeeta"

local counter = 0
local firstDeath = false

function Rollingpally_OnLoad(self)
  --this:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED");
     self:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED");
end


--function (Rollingpally)_OnEvent(self,event,...)
function Rollingpally_OnEvent(self, event, ...)
    local timestamp, type, sourceGUID, sourceName, sourceFlags, destGUID, destName, destFlags = ...

	if (event=="COMBAT_LOG_EVENT_UNFILTERED") then
	 if (type=="UNIT_DIED") then
       if nickname[destName] and counter <= 5 then
           --SendChatMessage(..nickname[destName].." has died.", GetNumRaidMembers()>0 and "RAID" or GetNumPartyMembers()>0 and "PARTY" or "SAY")
            SendChatMessage(nickname[destName].." has died.", GetNumRaidMembers()>0 and "RAID" or GetNumPartyMembers()>0 and "PARTY" or "SAY")
           counter = counter + 1
       end
       if not firstDeath then
           firstDeath = GetTime()
       elseif firstDeath and firstDeath + 300 > GetTime() then --if its been 5 minutes since the first death, reset the counter and clock
           firstDeath = false
           counter = 0
       end
    end
   end
end

*.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/
..\..\FrameXML\UI.xsd">
<Frame name="RollingpallyFrame">
    <Scripts>
        <OnLoad>Rollingpally_OnLoad(self)</OnLoad>
        <OnEvent>Rollingpally_OnEvent(self,event,...)</OnEvent>
    </Scripts>
</Frame>
</Ui>

i didnt erase code, i just commented it out so you can see what i changed