Problem adding options

Crober · 03-15-12, 09:34 PM

I am attempting to add options to my addon Holy Power Indicator.

I followed the tutorial found at http://forums.wowace.com/showthread.php?t=12050

The options show up fine but when I attempt to gain a charge of holy power i get the error "..\AddOns\HPIDev\Core.lua line 151: attempt to index field 'db' (a nil value)"

I can't seem to figure out what the problem is. Here is the code pastebin.

Any ideas/help is appreciated.

Also asked on CurseForge http://forums.curseforge.com/showthr...858#post320858

Phanx · 03-16-12, 02:32 AM

See my reply on your CurseForge thread.

kaels · 03-16-12, 02:41 AM

Basically, you have 2 problems with how you're referring to your saved variables. See here:

lua Code:

function events:UNIT_POWER(source,type)
    if source == "player" and type == "HOLY_POWER" then
        local power = UnitPower("player",9)
 
        if power == 1 then
            texture:SetTexture(self.db.textureSelect.."1.tga")
            frame:Show()
 
        elseif power == 2 then
            texture:SetTexture(self.db.textureSelect.."2.tga")
            frame:Show()
 
        elseif power == 3 then
            texture:SetTexture(self.db.textureSelect.."3.tga")
            frame:Show()
 
        elseif power == 0 then
            frame:Hide()
        end
    end
end

"self" in this function refers to "events", so it's trying to access events.db instead of HolyPowerIndicator.db.

The other problem is that HolyPowerIndicator.db is defined here:

lua Code:

self.db = LibStub("AceDB-3.0"):New("HolyPowerIndicatorDB", defaults, "Default")

to refer to the table "defaults", but the data you're actually interested in is in defaults.profile.

So basically you need to replace all instances of "self.db" (ouside of your OnInitialize function) with "HolyPowerIndicator.db.profile".

Phanx · 03-16-12, 02:58 AM

I already explained that in great detail in the other thread, which is why I posted here that I already replied there, so that others would not waste their time typing the same thing over again.

kaels · 03-16-12, 03:04 AM

Sorry, I was typing before you posted

Crober · 03-16-12, 10:04 PM

From CurseForge:

Thank you, that helped a lot. I think as I looked through the code and your comments I was able to pretty much understand what was going on.

I did run into an issue when trying to change the alpha (it would just snap back to the default value), I noticed the set function had

Code:

set = function(_, value) HolyPowerIndicator.db.profile.Alpha = value end,

I changed it to db.profile.alpha and that seemed to fix it.

Sometimes on /reload ui it unlocks the frame and shows the 3 charge texture, I removed "not" from

Code:

function HolyPowerIndicator:ToggleFrameLock(value)
	if value ~= nil then
		value = not self.db.profile.locked
	end

which fixes that, but I don't know if that would cause another issue.

Also is it possible to have the frame update as the user changes the options without a ui reload?

Phanx · 03-17-12, 02:12 AM

Also replied again on CurseForge.