ARKINV_ButtonOverrideBag not working

[tflo]

When AI is disabled it provides a button to the Blizz bag frame (ARKINV_ButtonOverrideBag1, or ARKINV_ButtonOverrideBag2 for non-split bags) that should allow to reenable AI. This button does not work for me. I get the tooltip and highlight when hovering over, but clicks do nothing.

AI 31009.Alpha.1, WoW 10.0.2 (46741)

[arkayenro]

no errors? mines working. it should close the arkinventory window at a minimum, does it do that?

[tflo]

No errors, it just does nothing.

it should close the arkinventory window at a minimum, does it do that?

Maybe we are speaking of different buttons, I mean this one:

[arkayenro]

yep thats the one. do you have errors enabled, or a bug catcher mod installed?

[tflo]

Sure, I have always BugSack and BugGrabber loaded. No errors.

Just double-checked it again with really every addon unloaded, except AI and BugGrabber/Sack. Button gets highlighted and shows the tooltip, but does nothing, and no errors.

In a second test I even moved the ContainerFrameCombinedBags away from the MinimapCluster, to rule out any interference with that, but it changes nothing.

In your previous answer, I do not understand what you mean with "it should close the arkinventory window" (that's why I thought we are not speaking of the same button): When this button is on screen, there is no AI window that could be closed, as AI is disabled.

What I expect this button to do is activating[^1] AI again, i.e. running ArkInventory:Enable() or a similar function, which also corresponds to what the tooltip is saying.

[^1]: Not that I'm activating/deactivating AI all day long, I just stumbled on this while trying to narrow down the mail retrieval lag issues.

[tflo]

Just had a look at the code and found out that if I comment out the first conditional of the OnClick function ArkInventory.LocationOverrideSet, at line 12633 in ArkInventory.lua…

--  if codex.profile.location[loc_id].override == value then
--      return
--  end

…then the button closes the Blizz bag (which it does not otherwise). It still does not reenable AI, and i haven't tried to find out where loc_id comes from, but something seems to be wrong with the logic of the OnClick function, or the value of loc_id.

[arkayenro]

wait, have you disabled arkinventory, not just the bank/bank override?

[tflo]

Yes, with this checkbox:

I was assuming the button that appears then on the Blizz bag frame is for re-enabling it. (See also my first post.)

What is "bank/bank override"?

[arkayenro]

the override leaves ai running, but it doesnt override the default blizzard frames. so yo get teh default bag or bank or vault (whatever you dont set for override)

its also in the config ? general page, under controls for each location

eg, open the ai bag window, click on the location icon, go down to bag, then click on disable in the override menu that will close the ai window and give you back the blizzard bags clicking the ai bag icon on that will revert it.

i guess i hadnt expected anyone to disable ai and go back in but its not hard to add some more code to cater for that.

[tflo]

overrride […] its also in the config ? general page, under controls for each location

Thanks, found it now. Indeed, for that, your button on the Blizz bag does work.

i hadnt expected anyone to disable ai and go back in but its not hard to add some more code to cater for that.

Yeah I already made a button that runs ArkInventory:Enable(), but if the above-mentioned AI button on the Blizz bag could do that, it would be more elegant of course.

[arkayenro]

will be fixed next release

[tflo]

Works fine in 31010 now. Thank you very much!