basic function help

[suipsyco]

I have a similar code that functions, so I copied the frame, setscript and registerevent codes from the working addons, yet for somereason the very first print statement wont run on this code, any ideas? The end goal of this addon will be to track the Spirit Shift auras on all members of the raid and show on my screen, the damage build up for each aura.

local f = CreateFrame("Frame")

f:SetScript("OnEvent", function(...)
	print("Start")
	local group = UnitInRaid("player") --if nil party , if >0 raid
	print("Group")
	local size = GetNumGroupMembers() --nil solo, else party or raid
	print("size")
	local groupType = "none"
	if size == nil then groupType = "solo" 
		elseif (group>0 and size<=5) then groupType = "party" 
		elseif (group>0 and size>5) then groupType = "raid" 
		else print("You done screwed up the groupType")
	end
	print("Check1: "..groupType) --Check
	capList = {}
	if (groupType == "party" or groupType == "raid") then CapList() end
	function CapList()
		for i=1, size do 
			local member = groupType..i local m, _ = GetInventoryItemID(member, INVSLOT_TRINKET1); local n,_ = GetInventoryItemID(member, INVSLOT_TRINKET2)
			if m or n =="124225" then
				table.insert(capList, member)
			end
		end 
	end
	for i in capList do print("Check2: "..i) end
	for i in capList do local n, _, _, _, _, _, _, _, _, _, _, _, _,_,t = UnitBuff(i, "Spirit Shift") print n, t end
end)

f:RegisterEvent("UNIT_AURA");

[myrroddin]

It might be working, but that checking for group is very ugly.

local f = f or CreateFrame("Frame)
f:RegisterEvent("GROUP_ROSTER_UPDATE")
 
f:SetScript("OnEvent", function(...)
    if ... ~= "GROUP_ROSTER_UPDATE" then
        return
    end
    local grouptype = nil -- reset
    if IsInRaid() then
       groupType = "raid"
    elseif IsInGroup()
        groupType = "party"
   else
       groupType = nil
    end
end)

[myrroddin]

http://wow.gamepedia.com/Handling_events#Lua-specific

[suipsyco]

Thanks, I knew there had to be a better way, but after searching for half an hour I decided to try my own way lol.

[MunkDev]

local f = CreateFrame("Frame")
f:RegisterEvent("GROUP_ROSTER_UPDATE")
 
f:SetScript("OnEvent", function(self, ...)
    if ... ~= "GROUP_ROSTER_UPDATE" then
        return
    end
    local grouptype = nil -- reset
    if IsInRaid() then
       groupType = "raid"
    elseif IsInGroup()
        groupType = "party"
   else
       groupType = nil
    end
end)

Fixed your code. First argument to the function is the frame itself, if more events are needed down the road.

[myrroddin]

D'oh, thanks Munk. That's what I get for drycoding when half (okay, 90%) asleep.