GovtGeek / Outfitter

Outfitter - Classic
MIT License
7 stars 2 forks source link

Lua Errors in Classic Era #34

Closed Gogo1951 closed 2 weeks ago

Gogo1951 commented 1 month ago

I feel like you've done a great job helping this add-on limp along. And I'd be lost without it.

But looking at how many XML and Lua errors just popped up, I have a feeling this may be the end of the line without a bit of a re-write.

Thanks for maintaining this!

GovtGeek commented 1 month ago

Blizzard has been doing a lot of movement underneath. They're changing some very structural pieces and finally removing some deprecated code. It's just not obvious when they're going to remove some of those pieces.

I still depend heavily on the bug reports, especially in Classic (includes SOD) and Retail.

Gogo1951 commented 1 month ago

Well man, if you're keen on working through this still. In Classic Era here's all the bugs I get on load.

Another

30x Outfitter/Outfitter.xml:13 Couldn't find inherited node: OptionsCheckButtonTemplate

Another

7x [string "*Outfitter.xml:1011_OnLoad"]:1: attempt to index a nil value

Locals: self = OutfitterQuickslotFlyouts { } (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = "attempt to index a nil value"

Another

7x [string "*Outfitter.xml:984_OnLoad"]:1: attempt to index a nil value

Locals: self = OutfitterItemComparisons { } (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = "attempt to index a nil value"

Another

7x [string "*Outfitter.xml:957_OnLoad"]:1: attempt to index a nil value

Locals: self = OutfitterTooltipInfo { } (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = "attempt to index a nil value"

Another

7x [string "*Outfitter.xml:930_OnLoad"]:1: attempt to index a nil value

Locals: self = OutfitterShowHotkeyMessages { } (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = "attempt to index a nil value"

Another

7x [string "*Outfitter.xml:903_OnLoad"]:1: attempt to index a nil value

Locals: self = OutfitterShowMinimapButton { } (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = "attempt to index a nil value"

Another

7x [string "*Outfitter.xml:878_OnLoad"]:1: attempt to index a nil value

Locals: self = OutfitterShowOutfitBar { } (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = "attempt to index a nil value"

Another

7x [string "*Outfitter.xml:851_OnLoad"]:1: attempt to index a nil value

Locals: self = OutfitterAutoSwitch { } (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = "attempt to index a nil value"

Gogo1951 commented 1 month ago
image

And when you go to use the menu.

17x ...rfaceOutfitter/Libraries/LibDropdown-1.0.lua:215: attempt to call upvalue 'GetMouseFocus' (a nil value) [string "@Outfitter/Libraries/LibDropdown-1.0.lua"]:215: in function <...rfaceOutfitter/Libraries/LibDropdown-1.0.lua:214> [string "@Outfitter/Libraries/LibDropdown-1.0.lua"]:280: in function <...rfaceOutfitter/Libraries/LibDropdown-1.0.lua:278>

Locals: frame = Button { handled = true released = false data =

{ } dataname = 3 PixelSnapDisabled = true expand = Texture { } enabled = true clickable = false group =
{ } option =
{ } text = FontString { } groupFrame = LibDropdownFrame1 { } swatch = Button { } parentTree =
{ } check = Texture { } } (temporary) = nil (temporary) = "attempt to call upvalue 'GetMouseFocus' (a nil value)" GetMouseFocus = nil UIParent = UIParent { firstTimeLoaded = 1 variablesLoaded = true }

GovtGeek commented 1 month ago

Check out the beta versions I have available.

Gogo1951 commented 1 month ago

Seems to work great, apologies I should have checked the beta builds!