ESOUI - Multiple Event Filters?

ESOUI (https://www.esoui.com/forums/index.php)

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

- - Multiple Event Filters? (https://www.esoui.com/forums/showthread.php?t=7910)

Multiple Event Filters?

So I'm following the Wiki on how to add multiple filters for the same event and I'm stumped. According to the Wiki:

Quote:

Multiple filters can be added in one call by repeatedly supplying a filter type followed by a parameter or in multiple calls by calling AddFilterForEvent for the same event. Filters cannot be removed once they were added without unregistering the event.

But when I try to do this (either method mentioned), the only filter that applies is the last one to run in the code.

My code for method 1:

Code:

EVENT_MANAGER:RegisterForEvent(SFR.addonName, EVENT_EFFECT_CHANGED, SFR.OnEffectChanged)



EVENT_MANAGER:AddFilterForEvent(SFR.addonName, EVENT_EFFECT_CHANGED, REGISTER_FILTER_ABILITY_ID, 89957, REGISTER_FILTER_ABILITY_ID, 61259)

My code for method 2:

Code:

EVENT_MANAGER:RegisterForEvent(SFR.addonName, EVENT_EFFECT_CHANGED, SFR.OnEffectChanged)



EVENT_MANAGER:AddFilterForEvent(SFR.addonName, EVENT_EFFECT_CHANGED, REGISTER_FILTER_ABILITY_ID, 89957)



EVENT_MANAGER:AddFilterForEvent(SFR.addonName, EVENT_EFFECT_CHANGED, REGISTER_FILTER_ABILITY_ID, 61259)

In both cases only the last filter to be executed works. I verified this by switching the abilityId's and still only the last one to be applied works. Any ideas or suggestions for what I'm doing wrong? I'd like to add a whole bunch of filters so that my code only fires on the events I want, but I can't even get these 2 to work.

Thanks in advance for any help!

Pretty obvious question: why not filter effects inside your function, rather than try to break your brain? :D

Don't get me wrong it had crossed my mind, I just wanted to see if there was an advantage to using the built-in filter or not. If it doesn't work that is exactly what I will be doing.

you need something like this:

lua Code:

local DETONATIONS =
{
    [61500] = true,
    [63296] = true,
    [63299] = true,
    [63302] = true,
}
 
local eventCounter = 0
for abilityId in pairs(DETONATIONS) do
    eventCounter = eventCounter + 1
    local eventName = ADDON_NAME..eventCounter
    EVENT_MANAGER:RegisterForEvent(eventName, EVENT_EFFECT_CHANGED, function(eventCode, changeType, _, _, unitTag, beginTime, endTime, _, _, _, _, _, _, _, _, abilityId)
        self:OnEffectChanged(changeType, unitTag, beginTime, endTime, abilityId)
    end)
    EVENT_MANAGER:AddFilterForEvent(eventName, EVENT_EFFECT_CHANGED, REGISTER_FILTER_ABILITY_ID, abilityId)
end

Thank you for the reply, I wil experiment with it :D

Quote:

Originally Posted by SilverWF (Post 35249)

Pretty obvious question: why not filter effects inside your function, rather than try to break your brain? :D

Pretty obvious answer: Because it's better.

Using the filters on the Event Manger is better for performance, since the game does the filtering in the C code rather than Lua. For stuff with abilities which can fire a lot, this means you have less lag in places where it matters the most: Combat.

Short, what Scootworks want to say:
The eventFilters need unique names as 1st parameter

Quote:

Originally Posted by Baertram (Post 35254)

Short, what Scootworks want to say:
The eventFilters need unique names as 1st parameter

That's not correct. The problem OP has is that he tries to register the same filter type multiple times, which is not possible. You have to register a separate event + filter for each ability id you want to filter.

Quote:

Originally Posted by sirinsidiator (Post 35257)

Ok, I was not clear enough. The loop in scoot works example showed the registering + filter for each ability id already.
But afai remember I even had to change the event name in order to get the multiple ability id's to work at the same time.

Quote:

Originally Posted by Baertram (Post 35259)

That's right. You can also register a eventNamespace/eventId pair only once, so you need to use a different namespace when you want to listen for the same event more than once.