KG Panels/LDB Plugin help

wolftech · 08-30-12, 04:29 PM

I have kg panels that I am setting up for the addon Broker_StatsNow. The addon is broken down into sub addons (Spells, Ranged, Melee, Tank, PVP) and each of those has a LDB display for each stat. (I am using StatBlockCore as my LDB display). I have it set up so the panel shows only in combat and it hides the ldb plugins when the panel isn't visible. This works great, as long as the ldb plugins are active. When they are not, I get this error.(I only have 2 set up so far... one for spells and one for ranged stats)

Code:

1x <string>:"ranged_OnEvent":13: attempt to index global "StatBlockCore_RangeAP" (a nil value)
<string>:"ranged_OnEvent":13: in function <string>:"ranged_OnEvent":1

Locals:

I get one for StatBlockCore_SpellPower if I am on my hunter and the spell stats are inactive.

The panels are not parented to the addons because when I try, I get a warning about a loop and it breaks.

Here is how they are scripted:

On Load:

Lua Code:

local _,class = UnitClass("player")
 
self:SetBackdropBorderColor(RAID_CLASS_COLORS[class].r, RAID_CLASS_COLORS[class].g, RAID_CLASS_COLORS[class].b)
 
self:RegisterEvent("PET_BATTLE_OPENING_START")
self:RegisterEvent("PET_BATTLE_CLOSE")
self:RegisterEvent("PLAYER_REGEN_DISABLED")
self:RegisterEvent("PLAYER_REGEN_ENABLED")
self:RegisterEvent("PLAYER_ENTERING_WORLD")
self:Hide()

On Event:

Lua Code:

if event == "PET_BATTLE_OPENING_START" then
  self:Hide()
elseif event == "PET_BATTLE_CLOSE" then
  self:Show()
end
 
if event == "PLAYER_REGEN_ENABLED" then
  self:Hide()
elseif event == "PLAYER_REGEN_DISABLED" then
  self:Show()
end
 
StatBlockCore_RangeAP:SetParent(kgPanels:FetchFrame("ranged"))
StatBlockCore_RangeCrit:SetParent(kgPanels:FetchFrame("ranged"))
StatBlockCore_RangeHaste:SetParent(kgPanels:FetchFrame("ranged"))
StatBlockCore_RangeMastery:SetParent(kgPanels:FetchFrame("ranged"))

It is scripted the exact same way for the spells (except using the LDB plugin frame names for spell stats). I also tried making the script dependent on the addon but that didn't work either.

Phanx · 08-30-12, 07:35 PM

As the error message indicates, you need to check that StatBlockCore_RangeAP exists before trying to do something with it.

Change your code from:

Code:

StatBlockCore_RangeAP:SetParent(kgPanels:FetchFrame("ranged"))
StatBlockCore_RangeCrit:SetParent(kgPanels:FetchFrame("ranged"))
StatBlockCore_RangeHaste:SetParent(kgPanels:FetchFrame("ranged"))
StatBlockCore_RangeMastery:SetParent(kgPanels:FetchFrame("ranged"))

to:

Code:

if StatBlockCore_RangeAP then
     StatBlockCore_RangeAP:SetParent(kgPanels:FetchFrame("ranged"))
end
if StatBlockCore_RangeCrit then
    StatBlockCore_RangeCrit:SetParent(kgPanels:FetchFrame("ranged"))
end
if StatBlockCore_RangeHaste then
    StatBlockCore_RangeHaste:SetParent(kgPanels:FetchFrame("ranged"))
end
if StatBlockCore_RangeMastery then
    StatBlockCore_RangeMastery:SetParent(kgPanels:FetchFrame("ranged"))
end

or this less repetitive and more easily extensible version:

Code:

local frameNames = {
    "StatBlockCore_RangeAP",
    "StatBlockCore_RangeCrit",
    "StatBlockCore_RangeHaste",
    "StatBlockCore_RangeMastery",
}

local panelFrame = kgPanels:FetchFrame("ranged")

for i = 1, #frameNames do
    local frameName = frameNames[i]
    if _G[frameName] then
        _G[frameName]:SetParent(panelFrame)
    end
end

wolftech · 08-31-12, 11:45 AM

Thanks Phanx. That first code change fixed the errors (for some reason, it didn't like the more compact version, but I don't mind the extra code as long as it works).

Now I just need to change my show on combat scripting so that it doesn't spawn the panel at all if the addons for it are not present. I will try making it dependent on the addon first, if that doesn't work, then I simple if then should do the trick. I might be back though, if I can't get it sorted

Phanx · 08-31-12, 07:44 PM

There was an error in the code in my last post; it's fixed now.