coa-addons/ShadowedUnitFrames
3
0
Fork 0
Code Issues Pull requests Projects Releases 7 Packages Wiki Activity Actions

fix(necromancer): ship bar layout in defaults so existing profiles don't crash
All checks were successful
release / release (push) Successful in 2s
Details

Browse source 
The necromancerBar height/order/background only reached a profile via the
defaultlayout merge, which runs on profile reset. Enabling the bar on a
pre-existing profile left height nil, crashing Layout:PositionWidgets
("attempt to compare number with nil" at layout.lua:441). Carry the layout
in the AceDB defaults so it resolves via the defaults metatable for old and
new profiles alike.
This commit is contained in:
Florian Andrew George Berthold 2026-06-08 20:42:10 +02:00
parent 2d9e9329db
commit 319bd05930
1 changed files with 5 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

6
ShadowedUnitFrames/ShadowedUnitFrames.lua
View file

@ -243,7 +243,11 @@ function ShadowUF:LoadUnitDefaults()
self.defaults.profile.units.player.runeBar = {enabled = false}
self.defaults.profile.units.player.totemBar = {enabled = false}
self.defaults.profile.units.player.druidBar = {enabled = false}
self.defaults.profile.units.player.necromancerBar = {enabled = false}
-- Carry the layout (height/order/background) in the defaults, not only in the
-- defaultlayout merge: that merge only runs on profile reset, so a bar added to
-- an existing profile would otherwise have a nil height and crash
-- Layout:PositionWidgets ("attempt to compare number with nil" in layout.lua).
self.defaults.profile.units.player.necromancerBar = {enabled = false, height = 0.40, order = 26, background = true}
self.defaults.profile.units.player.xpBar = {enabled = false}
self.defaults.profile.units.player.fader = {enabled = false, combatAlpha = 1.0, inactiveAlpha = 0.60}
self.defaults.profile.units.player.indicators.lfdRole = {enabled = true, size = 0, x = 0, y = 0}