Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot expand Border or Glow dropdowns when a Tick sub element exists #5635

Closed
nostradumazz opened this issue Jan 27, 2025 · 1 comment
Closed
Labels
⏱ Awaiting Response This ticket hasn't been triaged yet.

Comments

@nostradumazz
Copy link

Message: ...s/AceGUI-3.0/widgets/AceGUIWidget-DropDown-Items.lua:101: bad argument #1 to 'SetText' (Usage: self:SetText([text]))
Time: Mon Jan 27 01:07:27 2025
Count: 3
Stack: ...s/AceGUI-3.0/widgets/AceGUIWidget-DropDown-Items.lua:101: bad argument #1 to 'SetText' (Usage: self:SetText([text]))
[string "@Interface/AddOns/Bartender4/libs/AceGUI-3.0/AceGUI-3.0.lua"]:66: in function <...ace/AddOns/Bartender4/libs/AceGUI-3.0/AceGUI-3.0.lua:64>
[string "@Interface/AddOns/Bartender4/libs/AceGUI-3.0/AceGUI-3.0.lua"]:300: in function `Fire'
[string "@Interface/AddOns/WeakAurasOptions/AceGUI-Widgets/AceGUIWidget-WeakAurasExpand.lua"]:29: in function <...ions/AceGUI-Widgets/AceGUIWidget-WeakAurasExpand.lua:28>

Reproduce: Reproducible with a progress bar aura
Create a progress bar aura
Create Tick sub element
Create Border or Glow sub element
Attempt to expend either sub element

@github-actions github-actions bot added the ⏱ Awaiting Response This ticket hasn't been triaged yet. label Jan 27, 2025
@nostradumazz
Copy link
Author

nostradumazz commented Jan 27, 2025

function ItemBase.SetText(self, text)
	self.text:SetText(text or "")
end

not familiar enough to find where it's coming from but it's erroring because the above function is being passed a table:
ex:
1 Tick 3
2 Tick Area 1

ex:
1 = "Tick 1"
2 = "Tick Area 1"

full stack:
2x ...s/AceGUI-3.0-41/widgets/AceGUIWidget-DropDown-Items.lua:101: bad argument #1 to 'SetText' (Usage: self:SetText([text]))
[string "=[C]"]: in function SetText' [string "@Bartender4/libs/AceGUI-3.0-41/widgets/AceGUIWidget-DropDown-Items.lua"]:101: in function SetText'
[string "@Bartender4/libs/AceGUI-3.0-41/widgets/AceGUIWidget-DropDown.lua"]:566: in function <...r4/libs/AceGUI-3.0/widgets/AceGUIWidget-DropDown.lua:560>
[string "@Bartender4/libs/AceGUI-3.0-41/widgets/AceGUIWidget-DropDown.lua"]:605: in function SetList' [string "@Bartender4/libs/AceConfig-3.0-3/AceConfigDialog-3.0-87/AceConfigDialog-3.0.lua"]:1282: in function <...nfig-3.0/AceConfigDialog-3.0/AceConfigDialog-3.0.lua:1113> [string "@Bartender4/libs/AceConfig-3.0-3/AceConfigDialog-3.0-87/AceConfigDialog-3.0.lua"]:1642: in function FeedGroup'
[string "@Bartender4/libs/AceConfig-3.0-3/AceConfigDialog-3.0-87/AceConfigDialog-3.0.lua"]:1922: in function Open' [string "@Bartender4/libs/AceConfig-3.0-3/AceConfigDialog-3.0-87/AceConfigDialog-3.0.lua"]:869: in function <...nfig-3.0/AceConfigDialog-3.0/AceConfigDialog-3.0.lua:661> [string "=[C]"]: ? [string "@Bartender4/libs/AceGUI-3.0-41/AceGUI-3.0.lua"]:65: in function <...aceBartender4/libs/AceGUI-3.0/AceGUI-3.0.lua:63> [string "@Bartender4/libs/AceGUI-3.0-41/AceGUI-3.0.lua"]:299: in function Fire'
[string "@WeakAurasOptions/AceGUI-Widgets/AceGUIWidget-WeakAurasExpand.lua"]:29: in function <...ions/AceGUI-Widgets/AceGUIWidget-WeakAurasExpand.lua:28>

Locals:
(*temporary) = FontString {
}
(*temporary) =

{
1 = "Tick 1"
2 = "Tick Area 1"
}

adding a check for a table seems to have stopped the errors with no negative consequences. i can expand the dropdowns again:

function ItemBase.SetText(self, text)
	if type(text) == "table" then
		return
	end
	self.text:SetText(text or "")
end

InfusOnWoW added a commit to InfusOnWoW/WeakAuras2 that referenced this issue Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⏱ Awaiting Response This ticket hasn't been triaged yet.
Projects
None yet
Development

No branches or pull requests

1 participant