WoWInterface - Need help creating a addon

WoWInterface (https://www.wowinterface.com/forums/index.php)

- General Authoring Discussion (https://www.wowinterface.com/forums/forumdisplay.php?f=20)

- - Need help creating a addon (https://www.wowinterface.com/forums/showthread.php?t=41415)

Levisaxos

09-13-11 08:01 AM

Need help creating a addon

Greetings,

I'm very new to creating addons in wow. However I do have some experience in programming (being a 4th year student in the programming business) but I can't get things to work yet.
Ofcourse this is because I lack the experience and knowlage about programming in LUA.

My idea is to create a very simple addon for myself which logs the mobs I kill by ID, Name and level.

Currently I have:

local frame = CreateFrame("FRAME", "KillCountFrame");
frame:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED");
local function eventHandler(self, event, ...)
local timestamp, type, sourceGUID, sourceName, sourceFlags, destGUID, destName, destFlags = select(1, ...)
if (type == "PARTY_KILL") then
print("Event: " .. type);
print("You just killed a "..UnitName("target").."("..UnitLevel("target")..")");
end
end
frame:SetScript("OnEvent", eventHandler);

The problem is however that the event is being triggered twice. Were I to put this in a WTF/Account etc file, it would register the kill as twice. How can I fix this?

I have tried by setting the timestamp in the first to a global one, and then checking wether the global timestamp matches the current. But this didn't work.

Also, I'm looking for a nice tutorial on how to write the information to a text file in the WTF/Account directory.
Hope you can help me since i've been working on it for 3 hours also and the above code is all I got.

Greetings, Levisaxos

Taryble

09-13-11 08:41 AM

Addons can't write to text files on their own - however, WoW saves the variable that has the same name as the "SavedVariable" or "SavedVariablePerCharacter" line in the TOC file to a "SavedVariable" file.

Levisaxos

09-13-11 09:46 AM

Quote:

Originally Posted by Taryble (Post 244589)

I'm sorry for the misunderstandment. I know WoW can't write to text files but it can write the savedvariables.
I think it's possible to control which data is being saved as a variable?

For instance, when I kill 2 mobs, it should read as 2 in that file?
Is this possible?

Nibelheim

09-13-11 02:30 PM

Quote:

Originally Posted by Levisaxos (Post 244591)

Let's say you've got this line in your .toc

Code:

## SavedVariables: KillCountDB

Now, in your .lua file, you can do something like so:

Code:

local CharID

local KillCountDefaults = {

        totalkills = 0,

        characters = {},

}



local frame = CreateFrame("FRAME", "KillCountFrame");

frame:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED");

frame:RegisterEvent("PLAYER_LOGIN");



local function eventHandler(self, event, ...)

        if event == "PLAYER_LOGIN" then

                CharID = UnitName("player") .. " - " .. GetRealmName()

                if not KillCountDB then

                        KillCountDB = KillCountDefaults

                end

                if not KillCountDB.characters[CharID] then

                        KillCountDB.characters[CharID] = 0,

                end

        else

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

                if (type == "PARTY_KILL") then

                        print("Event: " .. type);

                        print("You just killed a "..UnitName("target").."("..UnitLevel("target")..")");

                        KillCountDB.characters[CharID] = KillCountDB.characters[CharID] + 1,

                        KillCountDB.totalkills = KillCountDB.totalkills + 1

                end

        end

end



frame:SetScript("OnEvent", eventHandler);

Levisaxos

09-13-11 04:00 PM

Quote:

Originally Posted by Nibelheim (Post 244609)

Let's say you've got this line in your .toc

Code:

See orginal post for code

Now, in your .lua file, you can do something like so:

Code:

See orginal post for code

Thanks, this got me a working version of what I exactly want but i'm still having the problem that both events, the PLAYER_LOADED and COMBAT_LOG_EVENT_UNFILTERED are being loaded twice whenever I log in, or kill a mob.
How can I fix this?

Nibelheim

09-13-11 04:35 PM

Quote:

Originally Posted by Levisaxos (Post 244620)

Looks like your CLEU args were off.

Code:

local Char, PlayerID, Realm

local KillCountDefaults = {

        totalkills = 0,

        characters = {},

}



local frame = CreateFrame("FRAME", "KillCountFrame");

frame:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED");

frame:RegisterEvent("PLAYER_LOGIN");



local function eventHandler(self, event, ...)

        if event == "PLAYER_LOGIN" then

                Char = UnitName("player")

                Realm = GetRealmName()

                PlayerID = UnitGUID("player")

                local CharID = Char .. " - " .. Realm

                

                if not KillCountDB then

                        KillCountDB = KillCountDefaults

                end

                if not KillCountDB.characters[CharID] then

                        KillCountDB.characters[CharID] = 0,

                end

        else

                local _, type, _, sourceGUID = ...

                if ( (sourceGUID == PlayerID) and (type == "PARTY_KILL") ) then

                        print("Event: " .. type);

                        print("You just killed a "..UnitName("target").."("..UnitLevel("target")..")");

                        KillCountDB.characters[CharID] = KillCountDB.characters[CharID] + 1,

                        KillCountDB.totalkills = KillCountDB.totalkills + 1

                end

        end

end



frame:SetScript("OnEvent", eventHandler);

Levisaxos

09-14-11 03:57 AM

Quote:

Originally Posted by Nibelheim (Post 244626)

Looks like your CLEU args were off.

Code:

See orginal post for code

Still registers the events twice sadly.
also, it doesn't register party kills anymore.

[edit]
I've manage to get the UnitName from the destName parameter. However I can't seem to get the Unit level.
Is there any way, like via GUID or flags, that I can use to help me get the unit level without him being a target?

Seerah

09-14-11 12:39 PM

Are you using an XML file and/or loading your Lua file twice?

Levisaxos

09-15-11 12:13 AM

Quote:

Originally Posted by Seerah (Post 244667)

Are you using an XML file and/or loading your Lua file twice?

I'm using one xml and one lua file

The XML file consist of:

Code:

<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">

  <Script File="KillCount.lua"/>

  <Frame name="KillCountFrame">

  <Scripts>

    <OnLoad>    

    </OnLoad>

  </Scripts>

  </Frame>

</Ui>

I assume this doesn't load the lua file twice ?

Nibelheim

09-15-11 12:43 AM

Quote:

Originally Posted by Levisaxos (Post 244680)

I'm using one xml and one lua file

The XML file consist of:

Code:

<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">

  <Script File="KillCount.lua"/>

  <Frame name="KillCountFrame">

  <Scripts>

    <OnLoad>    

    </OnLoad>

  </Scripts>

  </Frame>

</Ui>

I assume this doesn't load the lua file twice ?

If that's all your XML file is, then delete it :) Make sure KillCount.lua is mentioned in your .toc file

Seerah

09-15-11 08:59 PM

If you have your Lua file listed in your TOC *and* in your XML file, then yes it would be loaded twice. But, as Nib mentioned, just delete your XML file. All you're using it for is to load the Lua file (which you can do in the TOC) and to create a frame (which you aren't using, because you create another one in the Lua file).

Levisaxos

09-16-11 01:25 AM

Thank you all for your help guys!

Now all I have to find out is how to filter the kills to only kills made by people in my group (Could use party_kills for that, but that doesn't get raid kills!) and people in my raid!

So i'm guessing I will have to check the sourceName or sourceGUID to my raidmembers.

If anyone wishes to help me with that they are welcome to do so ;)

[edit]
I noticed that PARTY_KILL event is being triggered before UNIT_DIED.. so when I kill one myself I can just check the sourceGUID with my GUID on a PARTY_KILL event.
PARTY_KILL is also triggered when someone from my group kills a mob so that one isn't hard either..

However, how can I register if a mob is killed by a raid member?
UNIT_DIED does not give a valid sourceGUID of the person that killed him?

Ketho

09-16-11 02:06 PM

Quote:

Originally Posted by Levisaxos (Post 244700)

You would maybe have to check for the overkill parameter

Here is a quick example (borrowing the code from Nibelheim ;))

Code:

local Char, PlayerID, Realm

local KillCountDefaults = {

        totalkills = 0,

        characters = {},

}



local playerName = UnitName("player")

local damageEvent = {

        ["SWING_DAMAGE"] = true,

        ["RANGE_DAMAGE"] = true,

        ["SPELL_DAMAGE"] = true,

        ["SPELL_PERIODIC_DAMAGE"] = true,

}



local frame = CreateFrame("FRAME", "KillCountFrame");

frame:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED");

frame:RegisterEvent("PLAYER_LOGIN");



local function eventHandler(self, event, ...)

        if event == "PLAYER_LOGIN" then

                Char = UnitName("player")

                Realm = GetRealmName()

                PlayerID = UnitGUID("player")

                local CharID = Char .. " - " .. Realm

                

                if not KillCountDB then

                        KillCountDB = KillCountDefaults

                end

                if not KillCountDB.characters[CharID] then

                        KillCountDB.characters[CharID] = 0,

                end

        else

                local timestamp, subevent, hideCaster, sourceGUID, sourceName, sourceFlags, sourceRaidFlags, destGUID, destName, destFlags, destRaidFlags = ...

                local amount, overkill

                if damageEvent[subevent] then

                        if subevent == "SWING_DAMAGE" then

                                amount, overkill = select(12, ...)

                        else

                                amount, overkill = select(15, ...)

                        end

                        if overkill > 1 and (UnitInRaid(sourceName) or UnitInParty(sourceName) or sourceName == playerName) then

                                --print(sourceName.." killed "..destName.." for "..amount.." damage!")

                                KillCountDB.characters[sourceName] = (KillCountDB.characters[sourceName] or 0) + 1,

                        end

                end

        end

end



frame:SetScript("OnEvent", eventHandler);

There might be a problem with multiple timed overkill events for the same unit, so you maybe have to add in a throttle for each individual player, or it would all get counted as a kill

Levisaxos

09-16-11 02:40 PM

Quote:

Originally Posted by Ketho (Post 244715)

You would maybe have to check for the overkill parameter

Here is a quick example (borrowing the code from Nibelheim ;))

Code:

See original post for code

There might be a problem with multiple timed overkill events for the same unit, so you maybe have to add in a throttle for each individual player, or it would all get counted as a kill

Thank you for your suggestion.
Currently I have it fixed by checking in which zone I am.
If i'm in a raid zone, it only reacts to the UNIT_DEAD event type.

It still has some bumbs and bruises, like party members dying are being read as a kill aswell, but slowly i'm getting there !

All times are GMT -6. The time now is 11:24 AM.