Jaliborc / Bagnon

Single window displays for you items
189 stars 107 forks source link

Color Picker unusable in any other addons because of Bagnon. #1886

Closed meyverick closed 2 months ago

meyverick commented 3 months ago

Version

Bug

Opening the color picker in any addon produce the error and prevent me from changing colors.

Reproduction Steps

  1. Open a color picker in any Addons like WeakAura or Bartender.
  2. Disabling Bagnon solve the problem in all addons.

Logs

Message: ...libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua:73: attempt to call method 'GetColorAlpha' (a nil value)
Time: Sun Mar 17 10:07:13 2024
Count: 2
Stack: ...libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua:73: attempt to call method 'GetColorAlpha' (a nil value)
[string "@Interface/AddOns/MacroToolkit/libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua"]:73: in function `func'
[string "*ColorPickerFrame.xml:157_OnColorSelect"]:3: in function <[string "*ColorPickerFrame.xml:157_OnColorSelect"]:1>
[string "=[C]"]: in function `SetColorRGB'
[string "@Interface/FrameXML/UIDropDownMenu.lua"]:1451: in function `OpenColorPicker'
[string "@Interface/AddOns/BagBrother/libs/Sushi-3.2/classes/checks/Color.lua"]:23: in function `SetupColorPickerAndShow'
[string "@Interface/AddOns/MacroToolkit/libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua"]:94: in function <...libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua:56>

Locals: r = 0.729412
g = 0
b = 0
(*temporary) = nil
(*temporary) = ColorPickerFrame {
 BottomLeftCorner = Texture {
 }
 OnBackdropLoaded = <function> defined @Interface/SharedXML/Backdrop.lua:152
 GetBackdropColor = <function> defined @Interface/SharedXML/Backdrop.lua:390
 TopLeftCorner = Texture {
 }
 RightEdge = Texture {
 }
 SetupColorPickerAndShow = <function> defined @Interface/AddOns/BagBrother/libs/Sushi-3.2/classes/checks/Color.lua:23
 ApplyBackdrop = <function> defined @Interface/SharedXML/Backdrop.lua:294
 SetBackdrop = <function> defined @Interface/SharedXML/Backdrop.lua:329
 ClearBackdrop = <function> defined @Interface/SharedXML/Backdrop.lua:282
 hasOpacity = true
 GetBackdropBorderColor = <function> defined @Interface/SharedXML/Backdrop.lua:409
 GetBackdrop = <function> defined @Interface/SharedXML/Backdrop.lua:347
 SetBorderBlendMode = <function> defined @Interface/SharedXML/Backdrop.lua:266
 opacity = 0
 func = <function> defined @Interface/AddOns/MacroToolkit/libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua:71
 backdropInfo = <table> {
 }
 OnBackdropSizeChanged = <function> defined @Interface/SharedXML/Backdrop.lua:182
 HasBackdropInfo = <function> defined @Interface/SharedXML/Backdrop.lua:278
 GetEdgeSize = <function> defined @Interface/SharedXML/Backdrop.lua:188
 cancelFunc = <function> defined @Interface/AddOns/MacroToolkit/libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua:85
 0 = <userdata>
 TopEdge = Texture {
 }
 previousValues = <table> {
 }
 TopRightCorner = Texture {
 }
 Center = Texture {
 }
 SetupTextureCoordinates = <function> defined @Interface/SharedXML/Backdrop.lua:214
 SetBackdropBorderColor = <function> defined @Interface/SharedXML/Backdrop.lua:422
 BottomEdge = Texture {
 }
 BottomRightCorner = Texture {
 }
 SetupPieceVisuals = <function> defined @Interface/SharedXML/Backdrop.lua:246
 SetBackdropColor = <function> defined @Interface/SharedXML/Backdrop.lua:399
 LeftEdge = Texture {
 }
 opacityFunc = <function> defined @Interface/AddOns/MacroToolkit/libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua:78
}
(*temporary) = "attempt to call method 'GetColorAlpha' (a nil value)"
ColorCallback = <function> defined @Interface/AddOns/MacroToolkit/libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua:21
self = <table> {
 a = 1
 b = 0
 parent = <table> {
 }
 g = 0
 OnAcquire = <function> defined @Interface/AddOns/MacroToolkit/libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua:129
 SetDisabled = <function> defined @Interface/AddOns/MacroToolkit/libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua:156
 userdata = <table> {
 }
 base = <table> {
 }
 HasAlpha = true
 type = "ColorPicker"
 SetHasAlpha = <function> defined @Interface/AddOns/MacroToolkit/libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua:152
 r = 0.729412
 text = FontString {
 }
 events = <table> {
 }
 AceGUIWidgetVersion = 28
 colorSwatch = Texture {
 }
 frame = Button {
 }
 SetLabel = <function> defined @Interface/AddOns/MacroToolkit/libs/AceGUI-3.0/widgets/AceGUIWidget-ColorPicker.lua:140
 SetColor = <fun
NayooZ commented 3 months ago

Known bug, multiple open issues on this already. Temp fix is here:

https://github.com/Jaliborc/Bagnon/issues/1884#issuecomment-1986928525