Originally Posted by Terrillyn
I tried what you said before votan, and its actually just corrupting the table, even a fresh one.
first we call
Lua Code:
SimpleXPBar.AWSV = ZO_SavedVars:New("SimpleXPBar_Settings", "1", nil, SimpleXPBar.default_settings, nil, nil, '$' .. SimpleXPBar.name)
|
Use :NewAccountWide. This is just obscuring the meaning.
Also, SV version should be number. Not that it's very useful, but if you ever need to go beyond "9", be prepared for an unpleasant surprise.
Originally Posted by Terrillyn
and this is AWSV
Lua Code:
{ __index = { mysaveddata1, mysavedata2, version = "1", }, __newindex = function, default = table..., GetINterfaceForCharacter = function, }
|
That looks like a metatable, not AWSV contents.
Originally Posted by Terrillyn
then we call
Lua Code:
SimpleXPBar.CharSV = ZO_SavedVars:New("SimpleXPBar_Settings", "1", nil, SimpleXPBar.AWSV)
|
Defaults are magic.
You never, ever, want to pass something you pulled from SV as defaults for another. Pass raw defaults table that's in addon source.
I'd solve the whole thing in a completely different way, but technically the only thing that's wrong with your original code is that ZO_DeepTableCopy copies metatables and you probably don't want that in this case, as it can mess with ZO_SavedVars magic. So you'll be better off writing your own RawDeepTableCopy. Other than that, you should've focused on the color picker:
Lua Code:
getFunc = function()
return SimpleXPBar.CurSV.textbar.color.r,
SimpleXPBar.CurSV.textbar.color.b--[[!!!]],
SimpleXPBar.CurSV.textbar.color.g--[[!!!]],
SimpleXPBar.CurSV.textbar.color.a
end,