Saving hidden state

[Woeler]

Hello all,

I want my addon to be set hidden in the options. I also want this to be saved in the savedVariables. However I constantly keep getting

user:/AddOns/RaidScore/RaidScore.lua:47: attempt to index a nil value
stack traceback:
	user:/AddOns/RaidScore/RaidScore.lua:47: in function 'func'
	user:/AddOns/CombatCloud/libs/LibAddonMenu/controls/button.lua:61: in function '(anonymous)'

And I have no idea what I'm doing wrong. I've tried countless constructions. Anyone who can help me? I'm quite new to this.

--Addon: RaidScore
--Game: The Elder Scrolls Online: Tamriel Unlimited
--Author: Woeler
 
local RS = {
    Menu = LibStub:GetLibrary("LibAddonMenu-2.0"),
    hideScore = true,
    }
 
--Menu
    RS.panelData = {
        type = "panel",
        name = "RaidScore",
        author = "Woeler",
        version = "1.0.2",
        registerForRefresh = false,
        }
        
    
    RS.optionsData = {
        [1] = {
        type = "description",
        text = "Welcome to RaidScore by Woeler (EU: @Woeler)",
        width = "full",
        },
        [2] = {
              type = "checkbox",
              name = "Show/Hide RaidScore",
              tooltip = "This will reset once you zone or ReloadUI",
              getFunc = function() return hideScore end,
              setFunc = function(value) 
              if value == true then
                savedVariables.HideScores = false
                RAIDSCORE:SetHidden(false)
              elseif value == false then
                savedVariables.HideScores = true
                RAIDSCORE:SetHidden(true)
              end
              
              end,
         },
--      [3] = {
--      type = "button",
--      name = "Apply",
--      tooltip = "Apply the changes.",
--      func = function() 
--      savedVariables.Scores = RS.hideScore
--      end,
 
 
        width = "full",
        
    },
    
 
    RS.Menu:RegisterAddonPanel("RaidTimerPanel", RS.panelData)
    RS.Menu:RegisterOptionControls("RaidTimerPanel", RS.optionsData)
 
--function for getting the current score
function RaidScoreUpdate()
    RAIDSCORECounter:SetText(string.format(GetCurrentRaidScore()))
end
 
 
    local savedVariables --local reference for saved variables, actual object will be assigned later in EVENT_ADD_ON_LOADED event handler
    local defaults = { --default values for saved variables
        offsetX = 200,
        offsetY = 200,
        HideScores = false
    }
     
    --event handler for OnMoveStop
    local function OnMoveStop(self)
        savedVariables.offsetX = self:GetLeft()
        savedVariables.offsetY = self:GetTop()
    end
     
    --event handler for EVENT_ADD_ON_LOADED
    local function OnAddonLoaded(event, addonName)
        if addonName == "RaidScore" then --addonName is in general name of your addon manifext without .txt extension
     
            EVENT_MANAGER:UnregisterForEvent(MoveHandler, EVENT_ADD_ON_LOADED)
     
            --saved variables (in this case account wide)
            savedVariables = ZO_SavedVars:NewAccountWide("RaidScore", 1, "namespace", defaults)
     
            --create top level window, so we have something to work with
            RAIDSCORE:SetAnchor(TOPLEFT, GuiRoot, TOPLEFT, savedVariables.offsetX, savedVariables.offsetY) --restore window position from saved variables
            RAIDSCORE:SetHandler("OnMoveStop", OnMoveStop) --set handler to event "OnMoveStop", it means when you stop moving the window
            RAIDSCORE:SetHidden(savedVariables.HideScores) --set window invisible or visible
 
 
        end
    end
     
    EVENT_MANAGER:RegisterForEvent(MoveHandler, EVENT_ADD_ON_LOADED, OnAddonLoaded)

[Baertram]

Edit:
@Woeler
If you edit your sourcecode in the post after we are already answering it... this won't really help :-(
I was just wondering why I have a different source code that I've checked for you, then the sourc code that you have in your post NOW.
All of sudden variable names changed and source lines were commented?



My answer was:
Check your optionsData where you have set your variable "hideScore":

[2] = {
        type = "checkbox",
        name = "Show RaidScore always",
        tooltip = "This will reset once you zone or ReloadUI",
        getFunc = function() return hideScore end,
        setFunc = function(value)
            if value == true then
                hideScore = false
                RAIDSCORE:SetHidden(false)
            elseif value == false then
                hideScore = true
                RAIDSCORE:SetHidden(true)
            end
        end,
    },

1. hideScore here is a global variable because you did not define it as local
2. Or do you want to change the variable RS.hideScore and you just forgot the "RS." ?

For option 2 it should look like something like this:

[2] = {
        type = "checkbox",
        name = "Show RaidScore always",
        tooltip = "This will reset once you zone or ReloadUI",
        getFunc = function() return RS.hideScore end,
        setFunc = function(value)
            RS.hideScore = not value
            RAIDSCORE:SetHidden(RS.hideScore)
        end,
    },

In addition your saved variables table/array is using the variable Scores in your options menu, if you press the "save" button:

[3] = {
        type = "button",
        name = "Apply",
        tooltip = "Apply the changes.",
        func = function()
            savedVariables.Scores = RS.hideScore
        end,
 
        width = "full",
 
    },

But you have initialized it with the default saved variables data local defaults to be named "HideScores":

local defaults = { --default values for saved variables
    offsetX = 200,
    offsetY = 200,
    HideScores = false
}

So I assume it should look like this for your button to save the data:

[3] = {
        type = "button",
        name = "Apply",
        tooltip = "Apply the changes.",
        func = function()
            savedVariables.HideScores = RS.hideScore
        end,
        width = "full",
    },

As a better explanation:
1. You define some default data for the saved variables which will be loaded if the saved variables do not have any stored data (1st time your addon loads e.g.). You haev defined them in your table "defaults":

local defaults = { --default values for saved variables
    offsetX = 200,
    offsetY = 200,
    HideScores = false
}

2. You load your saved variables from the local file stored on your harddrive if the addon loads, in your function "OnAddonLoaded":

savedVariables = ZO_SavedVars:NewAccountWide("RaidScore", 1, "namespace", defaults)

Your addon will load all data from the stored file and if nothing is found it will use the content of your variable "defaults" as the default values.

3. In your settings you will be able to change values and then update your saved variables with the updated values. You can do this directly by changing "savedVariables.HideScores" e.g., of like you do by changing a helper variable table "RS" witht he entry "hideScore", and afterwards assign the content of your helper variable to your "savedVariables" table.

4. If your game client is logging out, doing a /reloadui or exits the variable "savedVariables" of your addon will be written to the .LUA file on your local harddrive.
The harddrive's filename will have the same name like your addons does and will be stored inside the SavedVariables folder in "c:\users\<your windows username>\documents\Elder Scrolls Online\live (or liveeu)\SavedVariables\". Inside this .LUA file you will find the stored settings in the array structured that you have specified in your addon's .TXT manifest file in this statement:

## SavedVariables: <your saved variables name>

Below this structured table you will find the settings saved account wide, or for each of your characters (depends on what method you use to save the savedvariables) and below you will find your content of the table "savedVariables" ->
You'll find "offsetX", "offsetY" and "HideScores" in there with their values.

[votan]

savedVariables is declared after the options. In LUA the order is important. In line 47 savedVariables is still unknown.

[Woeler]

Your solutions combined did it! Thanks for the help guys!