Event Chaining

[Wukar]

Hi,

i try to attach actions to already defined ui event, for example inventory tab clicks, without killing the already assigned actions. I saw a solution inside the InventoryMod (by Cr4x) but it looks cumbersome and may fail once another addon uses its own event storage.

The PreHookHandler, however, is not suitable in this scenario.

-- that kills the default behavior
ZO_PlayerInventoryTabsButton7:SetHandler("OnMouseUp", function() {}))

[Nogaruk]

To your example, I could be wrong, but I think the problem is you're not giving it anything to do.

-- SetHandler(string handlerName, function functionRef)

When I've used handlers before I haven't had any issues, though, it is possible the ones I've needed didn't already use the event I did. I would think the way you would want to write it though, would be:

ZO_PlayerInventoryTabsButton7:SetHandler("OnMouseUp", function(self) addonName:SomeFunction() end)

If that still breaks it, you could try seeing if registering into the main mouse event will work. You'll of course have to check against what the mouse was interacting with.

EVENT_GLOBAL_MOUSE_UP (integer button, bool ctrl, bool alt, bool shift, bool command)
I believe it would be:

EVENT_MANAGER:RegisterForEvent(addonName .. "_OnMouseUp", EVENT_GLOBAL_MOUSE_UP , function(...) self:OnMouseUpHandler(...) end)

function addonName:OnMouseUpHandler(event, button, ctrl, alt, shift, command)
end

Another choice would be to change what event you want to handle. For example you could try:

ZO_PlayerInventoryTabsButton7:SetHandler("OnClicked", function(self) addonName:SomeFunction() end)

edit: forgot an example

[Wukar]

Thanks for the examples. The OnMouseUp, kills the default operation, OnClicked is ignored. Maybe there's just something wrong with my event handler assignment:

local function BBS_GetActiveTabs(tabControl)
    local children = {}
    local counter = tabControl:GetNumChildren()
    for i=1, counter do
        local child = tabControl:GetChild(i)
        
        if (child:GetType() == CT_CONTROL) then
            -- add a handler for each control
            -- TODO: add one handler to the parent and handle tab changes
            child:SetHandler("OnClicked", function(self) 
                BBS_SearchBoxOnTextChanged(bbs.editbox) 
            end)
            children[i] = child
        end
    end
    return children
end

[Nogaruk]

I believe the "OnClicked" will only work if it is a CT_BUTTON. Beyond that, you'll probably have to wait for someone more experienced to see this. I have not needed to use handlers that required parameters to be passed along yet, so not sure if there's anything else wrong with that. Otherwise the code looks right.

Edit: also, I'm not sure if the syntax is correct unless you're calling a global function, but it might need addonName or self for

child:SetHandler("OnClicked", function(self) addonName:BBS_SearchBoxOnTextChanged(bbs.editbox) end)

[Wukar]

The EVENT_GLOBAL_MOUSE_UP did the trick, thanks

Anyway, the event chaining problem still ain't solved (at least for me )

[Kith]

You could try using the ZO built in prehook. As far as I know there is no posthook though so it'll run before the default code does

ZO_PreHookHandler(ZO_PlayerInventoryTabsButton7, 'OnMouseUp', function(...)
    -- do your stuff here, the ... is the variables passed
end)

[Cr4x]

Originally Posted by Wukar Hi, ... and may fail once another addon uses its own event storage.

Each addon should make sure not to override any existing handlers, wether it is using a event storage or not.