Combat log, can't get damage

[swordbeta]

So, I'm relative new to LUA and I was just messing around with the combat log. I'm unable to get the amount of damage, while there is. According to: http://www.wowpedia.org/API_COMBAT_LOG_EVENT this is the 13th parameter, but it seems like it's not working and I'm not sure what I'm doing wrong. When replacing the variable dmg with "test" it works, so it has to be "local dmg = select(12, ...)".

Can anyone point me into the right direction?

local DTPStext = CreateFrame("frame")

DTPStext:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED")
DTPStext:SetScript("OnEvent", function(this, event, ...) DTPStext[event](...) end)
DEFAULT_CHAT_FRAME:AddMessage("DTPStext: Loaded")

function DTPStext.COMBAT_LOG_EVENT_UNFILTERED(self, event, ...)
	local destGUID = select(6, ...)
	if(string.match(event, "DAMAGE")) then --this is damage!
		if(destGUID==UnitName("player")) then --this is me!
			local dmg = select(12, ...)
			DEFAULT_CHAT_FRAME:AddMessage(dmg)
		end
	end
end

EDIT: Okay, this is rather weird. Whenever taking damage from another playing, local dmg = select(11, ...) works. However when taking falling damage (environmental) ìt returns 1.

[Rilgamon]

destGuid should be 7th value.
You should pass on all args or give them the right names.
Your varargs (...) start with the timestamp followed by the event.

[tinyu]

shouldnt

DTPStext:SetScript("OnEvent", function(this, event, ...)

be

DTPStext:SetScript("OnEvent", function(self, event, ...)

?

[Ketho]

ENVIRONMENTAL_DAMAGE is a bit wierd in that the Environmental Type (arg10) moves everything else 1 position up.
So the damage amount you're looking for would be at arg11

.. Let's just say that your whole function is a mess and that everything inside should be killed with fire
The event handler you are using is good for registering multiple events. If you actually know where your parameters are

local DTPStext = CreateFrame("Frame")
DTPStext:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED")
DTPStext:SetScript("OnEvent", function(self, event, ...)
	DTPStext[event](self, ...)
end)

function DTPStext:COMBAT_LOG_EVENT_UNFILTERED(...)
	local _, subevent, _, _, _, _, _, destName = ...
	local amount

	if strfind(subevent, "DAMAGE") and destName == UnitName("player") then
		local prefix = strsub(subevent, 1, 5)
		if prefix == "SWING" then
			amount = select(10, ...)
		elseif subevent == "ENVIRONMENTAL_DAMAGE" then
			amount = select(11, ...)
		elseif prefix == "SPELL" or prefix == "RANGED" then
			amount = select(13, ...)
		end

		print("I got damaged for "..amount, subevent)
	end
end

[swordbeta]

Thanks a bunch all, especially Ketho!
While you're here anyway, is there an equivalent of 'setTimeout' of js in LUA?

[Seerah]

Originally Posted by tinyu shouldnt DTPStext:SetScript("OnEvent", function(this, event, ...) be DTPStext:SetScript("OnEvent", function(self, event, ...) ?

No. It could be

DTPStext:SetScript("OnEvent", function(myFrame, stuffHappens, ...)

if you want.

[Torhal]

To expound on what Seerah said: In that snippet, the word "this" was simply a parameter name - not the now-removed global "this".