[zaifon]

Hey,

Basically this is my first attempt at an addon, it's pretty basic and is supposed to show the amount of Chain Heal bounces for each cast, with multistrikes separated. I've got some previous programming experience, but this is the first time dabbling with Lua. I'll start with explaining the issue and then pasting in the code below.

The basic idea is two event listeners, one on UNIT_SPELLCAST_SUCCEEDED and then one on 'COMBAT_LOG_EVENT'. I think I've managed these ones pretty well, but the succeeded event fires before all the combat log collecting has been done. So I figured that the data gathering is taking too long and with no connection between the two events, they'll work asynchronously. Solution seems to be either to speed up the combat log events or make them fire off one another (somehow).

Here's the code:

-- Register Frame
function CHC:registerFrame()
    local CHCatcher_EventFrame = CreateFrame("Frame")
    CHC:setEventHandlers(CHCatcher_EventFrame)
end
 
-- Register Events
function CHC:setEventHandlers(Catcher)
    print("EventListener activated")
    Catcher:RegisterEvent("UNIT_SPELLCAST_SUCCEEDED")
    Catcher:RegisterEvent("COMBAT_LOG_EVENT")
    CHC:handleEvents(Catcher)
end
 
function CHC:handleEvents(Catcher)
    Catcher:SetScript("OnEvent", function(self, event, ...)
        if event == "UNIT_SPELLCAST_SUCCEEDED" then
            CHC:spellCast_Succeeded(...)
        elseif event == "COMBAT_LOG_EVENT" then
            CHC:filterTheCombatLog(...)
        end
    end)
end

With the successful cast function..

function CHC:spellCast_Succeeded(...)
    local unitID, spell, rank, lineID, spellID = select(1, ...)
 
    -- Only do stuff on chain heal casts that succeed
    if spellId == 1064 or spell == "Chain Heal" then 
        print("Finished casting " .. spell)
    end
end

And the combat log filtering.

function CHC:filterTheCombatLog(...)
    local type = select(2, ...)
 
    -- Check if combat log event is a heal
    if type == "SPELL_HEAL" then
        local spellId, spellName, spellSchool = select(12, ...)
 
        -- If the heal event matches chain heal and all that.
        if spellId == 1064 or spellName == "Chain Heal" then
 
            -- Check if the heal was a multistrike
            -- Save those separately
            if select(19, ...) == true then
                rawChainHealData.history.multistrikes = rawChainHealData.history.multistrikes + 1
                print("Multistrike #" .. rawChainHealData.history.multistrikes)
            else
                rawChainHealData.history.targets = rawChainHealData.history.targets + 1
                print("Hit #" .. rawChainHealData.history.targets .. " at " .. select(1, ...))
            end
 
            -- Save some of the data for later use.
            local spellData = {
                timestamp = select(1, ...),
                target = select(9, ...),
                source = select(5, ...),
                amount = select(15, ...),
                overhealing = select(16, ...),
                multistrike = select(19, ...)
            }
            table.insert(rawChainHealData.history.data, spellData)
        end
    end
end

Here's some sample output:


And full code can be viewed here (pastebin)

Thanks for your time!