Cell

Cell

1M Downloads

[Bug] Deleting an indicator allows the user to create an invalid indicator

tomratford opened this issue ยท 1 comments

commented

It seems that sometimes after deleting a indicator, when you go to create a new indicator you are able to create the indicator without a name. This then breaks the entirety of the indicator panel and required me to manually change the lua table with /script to remove the broken entries.

To reproduce:

  • reload ui
  • open cell -> indicators
  • create two new indicators
  • delete one of the indicators
  • open the dialog to create a new indicator, then close it
  • delete the second indicator
  • open the dialog to create a new indicator. You should now be able to create a new indicator without a name.

Screenshot of 'create new indicator' panel with blank name but highlighted 'yes' button

Screenshot 2024-06-15 at 15 03 29


Bugsack output for broken indicator panel

8x Cell/Modules/Indicators.lua:1983: attempt to index a nil value
[string "@Cell/Modules/Indicators.lua"]:1983: in function 
[string "@Cell/Modules/Indicators.lua"]:1410: in function 
[string "@Cell/Widgets/Widgets.lua"]:1801: in function 

Locals:
last = Button {
BottomLeftCorner = Texture {
}
fs = FontString {
}
color =

{
}
OnBackdropLoaded = defined @Blizzard_SharedXML/Backdrop.lua:152
GetBackdropColor = defined @Blizzard_SharedXML/Backdrop.lua:390
ShowTooltip = defined @Cell/Modules/Indicators.lua:2000
TopLeftCorner = Texture {
}
RightEdge = Texture {
}
ApplyBackdrop = defined @Blizzard_SharedXML/Backdrop.lua:294
SetBackdrop = defined @Blizzard_SharedXML/Backdrop.lua:329
ClearBackdrop = defined @Blizzard_SharedXML/Backdrop.lua:282
GetBackdropBorderColor = defined @Blizzard_SharedXML/Backdrop.lua:409
GetBackdrop = defined @Blizzard_SharedXML/Backdrop.lua:347
height = 20
hoverColor =
{
}
SetBorderBlendMode = defined @Blizzard_SharedXML/Backdrop.lua:266
HideTooltip = defined @Cell/Modules/Indicators.lua:2009
id = 31
backdropInfo =
{
}
OnBackdropSizeChanged = defined @Blizzard_SharedXML/Backdrop.lua:182
TopRightCorner = Texture {
}
typeIcon = Texture {
}
tooltips =
{
}
0 =
UpdatePixelPerfect = defined @Cell/Widgets/Widgets.lua:674
GetEdgeSize = defined @Blizzard_SharedXML/Backdrop.lua:188
_tooltipsInited = true
SetTexture = defined @Cell/Widgets/Widgets.lua:632
TopEdge = Texture {
}
Center = Texture {
}
BottomRightCorner = Texture {
}
width = 20
HasBackdropInfo = defined @Blizzard_SharedXML/Backdrop.lua:278
BottomEdge = Texture {
}
SetTextColor = defined @Cell/Widgets/Widgets.lua:568
SetupTextureCoordinates = defined @Blizzard_SharedXML/Backdrop.lua:214
SetupPieceVisuals = defined @Blizzard_SharedXML/Backdrop.lua:246
SetBackdropColor = defined @Blizzard_SharedXML/Backdrop.lua:399
LeftEdge = Texture {
}
SetBackdropBorderColor = defined @Blizzard_SharedXML/Backdrop.lua:422
}
(for generator) = defined =[C]:-1
(for state) =
{
1 =
{
}
2 =
{
}
3 =
{
}
4 =
{
}
5 =
{
}
6 =
{
}
7 =
{
}
8 =
{
}
9 =
{
}
10 =
{
}
11 =
{
}
12 =
{
}
13 =
{
}
14 =
{
}
15 =
{
}
16 =
{
}
17 =
{
}
18 =
{
}
19 =
{
}
20 =
{
}
21 =
{
}
22 =
{
}
23 =
{
}
24 =
{
}
25 =
{
}
26 =
{
}
27 =
{
}
28 =
{
}
29 =
{
}
30 =
{
}
31 =
{
}
32 =
{
}
}
(for control) = 32
i = 32
t =
{
showDuration = false
auras =
{
}
showAnimation = true
showStack = true
spacing =
{
}
indicatorName = "indicator3"
size =
{
}
enabled = true
type = "icons"
frameLevel = 5
auraType = "buff"
castBy = "me"
font =
{
}
name = ""
position =
{
}
orientation = "right-to-left"
trackByName = false
numPerLine = 5
num = 5
}
b = Button {
BottomLeftCorner = Texture {
}
color =
{
}
OnBackdropLoaded = defined @Blizzard_SharedXML/Backdrop.lua:152
GetBackdropColor = defined @Blizzard_SharedXML/Backdrop.lua:390
SetBackdropBorderColor = defined @Blizzard_SharedXML/Backdrop.lua:422
RightEdge = Texture {
}
ApplyBackdrop = defined @Blizzard_SharedXML/Backdrop.lua:294
SetBackdrop = defined @Blizzard_SharedXML/Backdrop.lua:329
ClearBackdrop = defined @Blizzard_SharedXML/Backdrop.lua:282
GetBackdropBorderColor = defined @

commented

Fixed, thanks!