loot roll event parsing

[hairy_palms]

im trying to write some lua that just prints the items name when a loot roll starts, yet all it seems to print is nil, am i getting the rollid the wrong way?

function printname(self, event, ...)
	if event == "START_LOOT_ROLL" then
		local texture, name, count, quality = GetLootRollItemInfo(self.rollID)		
		print(name)
        end
end
local eventFrame = CreateFrame("Frame", nil, UIParent)
eventFrame:RegisterEvent("START_LOOT_ROLL")
eventFrame:SetScript("OnEvent", function()  printname() end)

[Seerah]

You're not quite using/accessing the variables from your event handler correctly.

function printname(id)
	local texture, name, count, quality = GetLootRollItemInfo(id)		
		print(name)
        end
end


eventFrame:SetScript("OnEvent", function(self, event, id)  printname(id) end)

If you're only registered for one event, then you don't need to check on which event triggered your script. The first variable passed in any frame's script (here it's "self") is a reference to the frame that called it - eventFrame. The second variable in the function (event) is what event was called - this is the first return in an OnEvent handler. The third variable (id) is the return provided by the event (if any) - in this case, it is the id of the item being rolled for.

When calling an external function from your event handler, if you need access to any of these variables, you need to pass them through - hence printname(id).


/edit: you can also just do this:

eventFrame:SetScript("OnEvent", function(self, event, id)
     local texture, name, count, quality = GetLootRollItemInfo(id)		
          print(name)
     end
end)

[Rilgamon]

Use the arg1 of the event. The event gives the rollid, rolltime as two args.

function printname(self, event, rollId, rollTime)

[hairy_palms]

if i did need the if event, since im using more than 1 event in the finished code, they just arent implemented yet, so if i understand rightly

function printname(self, event, id)
        if event == "START_LOOT_ROLL" then
	        local texture, name, count, quality = GetLootRollItemInfo(id)		
		print(name)
        end
end

eventFrame:SetScript("OnEvent", function(self, event, id)  printname(self, event, id) end)

would be the correct code while keeping the event check?

/edit hmm guess not that seems to crash WoW

[Rilgamon]

first I'd make the function local

local function printname(self, event, id)

And since all you do is passing the info to the function you can skip the additional func.

eventFrame:SetScript("OnEvent", printname)

I encounter crashes while testing code quite often. Especially if I switch
from wow-screen to desktop fast ... just retry the same code
If it crashes 3 times in a row then you should start looking for the reason

[hairy_palms]

thanks for all the help so far guys!
ive been working at this and ive come up with this debugging function

local function printname(self, id)
	local texture, name, count, quality = GetLootRollItemInfo(id)
	print(texture.." "..name.." "..count.." "..quality)
	local color = ITEM_QUALITY_COLORS[quality]
        print(color.r)
	print(self)	
end
local eventFrame2 = CreateFrame("Frame", nil, UIParent)
eventFrame2:RegisterEvent("START_LOOT_ROLL")
eventFrame2:SetScript("OnEvent", function(self, event, id)  printname(self, id) end)

so, sort of related to this, but how would i find the frame that fired the START_LOOT_ROLL event, eg GroupLootFrame1-4?
when i print(self) above it returns a table, are any of the things in there part of what im looking for?

[Xrystal]

The id might be what you are looking for:

http://wowprogramming.com/docs/events/START_LOOT_ROLL

The self you mentioned is your event frame identity in case you have multiple frames watching events via the same function.

[hairy_palms]

hmm is there a way to get the frame that caused the event from its ID then? the ID itself is just a number that increments on its own which is no good, it doesnt reset when rolls are completed

[v6o]

noticed this code i blizzards default code

frame = _G["GroupLootFrame"..i];
if ( not frame:IsShown() ) then
frame.rollID = id;
frame.rollTime = rollTime;