OverrideBar issues rise up again ..

[Xrystal]

I've been playing Remix like many others ( who aren't lucky enough to get into the beta yet ) and I stumbled across several override bar issues.

Now, in light with nUI, I had decided to let Blizzard manage the override bar but just have nUI reposition things it didn't want Blizzard to control. However, the way Scott set up the keybinds for nUI meant that the override bar no long has keybinds, and vice versa if I try to set it up etc.

So, thought I would return back to having nUI manage the override bar and with so many quests in the Remix using it, I had ample tests to highlight the issues that occurred.

1. Buttons don't appear straight away ( Big Boom - BfA quest)
2. Bar jumps in and out during combat ( King Dun - Panda quest)
3. Bar and Buttons appear and disappear when initiated

I have been testing 1 for the last few days ( repeatedly ) in a simple addon to help identify what's needed for it to work and for some reason I can't get the icon to show. It's there but due to item 2 on the list, I need to limit what is done to avoid those functions that can't be used in combat.

My complete code is below and in lights to item 1 on the list the bar shows ( with all buttons but no icons ) when I cause the bar to appear ( do the next step in the quest ). The keybinds work fine, and inside the override bar's setup function I can see that it gets called whenever a button is updated with its action info, but the icon isn't shown.

Now, I can see GetActionInfo(action) as one of the okay functions ( hopefully while in combat too ) to use in the secured environment but, for some reason it returns nil inside the secure environment when it's insecure equivalent.

Also, the button's SetAttribute values never seem to get triggered so can't go that route either.

If anyone can see anything that I've missed for it to work as I expect, it would be a great help.
Thanks in advance.

edit: Just to add that if you reload the UI after the bar appears the active buttons have their icons show up. But when the inactive ones become active you have to do a reload to see the icon. So the icon is set to the button, it's just not being shown.

local BUTTON_SIZE, BUTTON_SPACING = 50, 1
local NUM_ACTIONBAR_BUTTONS = 12
 
local BlizUI = CreateFrame( "Frame", "XAB_BlizUI", UIParent, "SecureFrameTemplate" )
OverrideActionBar:SetParent(BlizUI)
BlizUI:Hide()
 
 
local barFrame = CreateFrame("Frame", "XBar_OverrideBar", UIParent, "SecureHandlerStateTemplate")
barFrame:SetSize((BUTTON_SIZE + BUTTON_SPACING) * NUM_ACTIONBAR_BUTTONS, BUTTON_SIZE)
barFrame:SetPoint("CENTER")
 
local background = barFrame:CreateTexture(nil, "BACKGROUND")
background:SetColorTexture(0, 0, 0, 0.5)
background:SetAllPoints()
background:Show()
 
barFrame:Execute([[
    ActionButtons = newtable()
]])
 
barFrame:SetPoint("CENTER")
 
local buttons = { }
for i = 1, NUM_ACTIONBAR_BUTTONS do
    
    local button = CreateFrame("CheckButton","XBar_OverrideButton"..i, barFrame,"OverrideActionBarButtonTemplate")
    
    button:SetID(i)
    button:SetSize(BUTTON_SIZE, BUTTON_SIZE)
    
    if i ~= 1 then
        button:SetPoint("LEFT", buttons[i - 1], "RIGHT", BUTTON_SPACING, 0)
    else
        button:SetPoint("LEFT", 5, 0)
    end
    
    button.icon:Show()
 
    local background = button:CreateTexture(nil, "BACKGROUND")
    background:SetColorTexture(1, 1, 1, 0.5)
    background:SetAllPoints()
 
    barFrame:SetFrameRef("button", button)
    barFrame:Execute(([[
        ActionButtons[%s] = self:GetFrameRef("button")
    ]]):format(i))
    
    button:SetAttribute("_onstate-statehidden",[[
        print("statehidden: ",newstate)    
        if not newstate then return end
        if newstate == "true" then
            self:Hide()
        else
            self:Show()
        end
    ]])
 
    button:SetAttribute("_onstate-actionpage",[[
        print("_onstate-actionpage: ",newstate)    
        if not newstate then return end
    ]])
    
    buttons[i] = button
end
barFrame.Buttons = buttons    
 
barFrame:Execute([[
    self:SetAttribute("frameref-button", nil)
]])
 
barFrame:SetAttribute('_onstate-page', ([[
    if not newstate then return end
    newstate = tonumber(newstate)
    self:SetAttribute("actionpage", newstate)
    local pageOffset, hasAction = (newstate - 1) * NUM_ACTIONBAR_BUTTONS
    for id = 1, #ActionButtons do
        ActionButtons[id]:SetAttribute("actionpage",newstate)
        ActionButtons[id]:SetAttribute("action",id + pageOffset)
    end
]]):gsub('NUM_ACTIONBAR_BUTTONS', NUM_ACTIONBAR_BUTTONS))
 
RegisterStateDriver( barFrame, "visibility", "[overridebar][vehicleui][target=vehicle, exists] show; hide" )
RegisterStateDriver(barFrame, "page", ("[vehicleui] %s; [overridebar] %s; 1"):format(GetVehicleBarIndex(), GetOverrideBarIndex()))
 
MainMenuBarVehicleLeaveButton:SetParent(barFrame)
MainMenuBarVehicleLeaveButton:SetAllPoints(barFrame.Buttons[12])
RegisterStateDriver(MainMenuBarVehicleLeaveButton, "visibility", "[canexitvehicle] show; hide")
 
hooksecurefunc(OverrideActionBar,"Setup", function(self,skin, barIndex)
    print("OverrideActionBar:Setup", self:GetName(),skin,barIndex)
    for k=1,6 do
        local button = barFrame.Buttons[k]            
        local isHidden = button:GetAttribute("statehidden")
        if button.action then
            local actionType, actionID = GetActionInfo(button.action) 
            local iconVisible = button.icon:IsVisible()
            local iconShown = button.icon:IsShown()
            print(k,button.action,isHidden,actionType, actionID,iconVisible,iconShown)
            button:SetAttribute("statehidden",actionID and false or true)
            barFrame:Execute(([[
                local button = ActionButtons[%d]
                local action = button:GetAttribute("action")                            
                local actionType, actionID = action and GetActionInfo(action)           -- actionID is 0 ? 
                print(button:GetID(),action,actionType,actionID)
            ]]):format(k))
        end
    end
end)

[Fizzlemizz]

There's a PTR with an 11.0 build (actually I think it's Beta as the server has my Beta characters that I create and/or copied).

Edit: Just saw this but addons work so, when it's "released" (17th)...

[Xrystal]

Originally Posted by Fizzlemizz There's a PTR with an 11.0 build (actually I think it's Beta as the server has my Beta characters that I create and/or copied). Edit: Just saw this but addons work so, when it's "released" (17th)...

Yeah, I found that out after posting that .. but the problem still persists unfortunately And I need to figure out one way or another before the expansion goes live and people go back to playing seriously again .

Options for nUI and the overridebar situation are as follows:
1. Fix the override bar in nUI problems so that ALL quests will work without needing to reload the UI every button update
2. Change nUI's keybinding system so that it uses the main action bar keybinds rather than it's own ones which is the main issue if I don't do option 1.

Option 2 sounds easier, so I will just have to figure that part out and see if it's feasible.

[Xrystal]

Well think I have figured out nUI's issue.

For some reason Scott was using the custom action bar's name based keybinds rather than use the Blizzard action id based keybinds.

Now both the override bar ( in blizzards control ) shows and has working keybinds and so does nUI both before and after the override bar is used. And the keybind screen doesn't have a large bunch of nUI keybinds that don't need to be there.

Next step , is testing it on the combat based override bar quests ( Kung Din and I think Jade Mine Cart) quests.

[Xrystal]

And the plot thickens ..

Started a new time running character to test the bombing run which uses the override bar and that one causes an actual error.


[ADDON_ACTION_FORBIDDEN] AddOn 'nUI' tried to call the protected function 'UseAction()


Even a reload doesn't work. All nUI does is Set the anchor to over nUI's action bars and revert it's moving of the micro menu buttons.

Totally removed nUI's affect on the override bar and the error still occurs.
But if I turn off nUI it all works.

So, something in nUI is breaking it for that one quest.

The only thing I can put it down to is that quest is affected by the keybinding changes I made for nUI's action bars ..

And checking 'The Big Boom' quest in Voldun and that still works as expected.

So, sounds like a 4th type of override bar set up, but I don't have a clue how to identify the problem in this case. Hopefully it's the only one of this type and people can live with clicking for this quest.