View Single Post
09-01-12, 10:54 AM   #1
zork
A Pyroguard Emberseer
 
zork's Avatar
AddOn Author - Click to view addons
Join Date: Jul 2008
Posts: 1,740
Does [vehicleui] not reflect all states where a Override bar skin is used?

I just had someone report a bug on rActionBarStyler telling that on the Nefarian HC encounter in Blackwing Descent there is a spell called Dominon. If you are affected by that spell you will get the override bar.

My problem is:
I'm having a state driver on the override bar to only show on [vehicleui] but I think that does not reflect this condition in the ActionButtonController:

Lua Code:
  1. -- If we have a skinned vehicle bar or skinned override bar, display the OverrideActionBar
  2. if ((HasVehicleActionBar() and UnitVehicleSkin("player") and UnitVehicleSkin("player") ~= "")
  3. or (HasOverrideActionBar() and GetOverrideBarSkin() and GetOverrideBarSkin() ~= "")) then
  4.     -- For now, a vehicle has precedence over override bars (hopefully designers make it so these never conflict)
  5.     if (HasVehicleActionBar()) then
  6.         OverrideActionBar_Setup(UnitVehicleSkin("player"), GetVehicleBarIndex());
  7.     else
  8.         OverrideActionBar_Setup(GetOverrideBarSkin(), GetOverrideBarIndex());
  9.     end
  10.     ...
  11. end


Actually I thought [vehicleui] would catch all of those events but since Dominon is just using an OverrideBarSkin and is no VehicleBar itself it seems not to get triggered.

Now I'm curious. What would be the macro condition to spawn the OverrideBar if you have just an OverrideBarSkin.

I know we have [possessbar] aswell. But I cannot show the overridebar everytime someone is possessed because that would be wrong aswell. Because normally the possessbuttons come up in the default actionbar not in the OverrideBar.

*EDIT*

Solution posted below.
__________________
| Simple is beautiful.
| WoWI AddOns | GitHub | Zork (WoW)

"I wonder what the non-pathetic people are doing tonight?" - Rajesh Koothrappali (The Big Bang Theory)

Last edited by zork : 02-13-13 at 07:50 AM.