WoWInterface - Defaults "These Settings" handle

WoWInterface (https://www.wowinterface.com/forums/index.php)

- General Authoring Discussion (https://www.wowinterface.com/forums/forumdisplay.php?f=20)

- - Defaults "These Settings" handle (https://www.wowinterface.com/forums/showthread.php?t=54065)

Defaults "These Settings" handle

Hey, I made an addon that uses Ace3:

It stores options, adds it to Blizzard Addon options page, saves beetween sessions etc.
However, when I click Interface → Options → AddOns → Defaults → These Settings, nothing happens.

Lua Code:

-- libs
-- external /libs as ADDON_NAME.libs objects
ADDON_NAME.libs = {};
ADDON_NAME.libs.Config = LibStub("AceConfig-3.0");
ADDON_NAME.libs.ConfigDialog = LibStub("AceConfigDialog-3.0");
ADDON_NAME.libs.DB = LibStub("AceDB-3.0");
ADDON_NAME.libs.Timer = LibStub("AceTimer-3.0");

Lua Code:

-- options
-- ADDON_NAME.default.global.MODULE_NAME
-- ADDON_NAME.options.args.MODULE_NAME
ADDON_NAME.default = {global = {}};
ADDON_NAME.options = {
    name = "ADDON_FULL_NAME",
    type = "group",
    args = {},
};

Lua Code:

ADDON_NAME.db = ADDON_NAME.libs.DB:New("ADDON_NAMEDB", ADDON_NAME.default, true);
ADDON_NAME.libs.Config:RegisterOptionsTable("ADDON_NAME", ADDON_NAME.options);
ADDON_NAME.frames = {options = ADDON_NAME.libs.ConfigDialog:AddToBlizOptions("ADDON_NAME")};

I tried to implement it according to some comments I saw, using:

Lua Code:

ADDON_NAME.libs.ConfigRegistry = LibStub("AceConfigRegistry-3.0");

Lua Code:

ADDON_NAME.frames.options.default = function(...)
    ADDON_NAME.db = ADDON_NAME.default;
    ADDON_NAME.libs.ConfigRegistry:NotifyChange("ADDON_NAME");
end;

It changes ADDON_NAME.db, and corresponding GUI in options panel at will.
However, it works only first time after (re)loading GUI and changes are not saved.

Problem: How to handle Defaults → "These Settings" to default it's settings and save them correctly?
Any manual page, tutorial, examples or direct solution would be appreciated.

Quote:

Originally Posted by Benio (Post 317186)

Lua Code:

ADDON_NAME.frames.options.default = function(...)
    ADDON_NAME.db = ADDON_NAME.default;
    ADDON_NAME.libs.ConfigRegistry:NotifyChange("ADDON_NAME");
end;

Tables are passed by reference, what you're doing is just making ADDON_NAME.db point to ADDON_NAME.default instead of copying the table

So anything you do afterwards is with the table referenced in ADDON_NAME.default instead of the dereferenced one from ADDON_NAME.db

If you're lazy you can use Blizzard's CopyTable for deep copying

Lua Code:

function CopyTable(settings)
    local copy = {};
    for k, v in pairs(settings) do
        if ( type(v) == "table" ) then
            copy[k] = CopyTable(v);
        else
            copy[k] = v;
        end
    end
    return copy;
end

Code:

ADDON_NAME.db = CopyTable(ADDON_NAME.default)

But there also was something about AceDB tables being "magic" tables
So in that specific case I'm probably wrong, and I don't really understand that part anymore :(

http://forums.wowace.com/showthread.php?t=25654#3
http://forums.wowace.com/showthread.php?t=19243
http://www.wowace.com/addons/ace3/pa...-3-0-tutorial/

(Edit) would DBObjectLib:ResetProfile() work for you?

Code:

ADDON_NAME.db:ResetProfile()

Thanks for a reply! Actually, ResetDB with CopyTable makes change permanent.
Unfortunatelly, it looks like I still need to reload UI in order to make GUI active again (clicking checkbox changes it's visuals, and related ADDON_NAME.db.global setting, but those are not saved on reloading UI).

Lua Code:

ADDON_NAME.db:ResetDB("Default");
ADDON_NAME.db.global = CopyTable(ADDON_NAME.default.global);
ADDON_NAME.libs.ConfigRegistry:NotifyChange("ADDON_NAME");

edit: ResetProfile makes nothing, even with previously RegisterDefaults.

edit2: ResetDB defaults .db already thanks to second parameter of New, so no CopyTable is needed.
Instead, we need to just reinitialize database since it's reset:

Lua Code:

ADDON_NAME.db:ResetDB("Default"); -- reset .db to default settings (passed as second parameter of ADDON_NAME.libs.DB:New)
ADDON_NAME.db = ADDON_NAME.libs.DB:New("ADDON_NAMEDB", ADDON_NAME.default, true); -- reinitialize .db (same as first init above)
ADDON_NAME.libs.ConfigRegistry:NotifyChange("ADDON_NAME"); -- finally, let GUI know to handle changes and work correctly after changing config

Works like a charm.

Final Note: Thanks, Ketho, for pointing out that passing table by reference won't work.
Thanks that, and the linked threads, connected with the "magic" issue, I found out how to handle it.
For futher reference, I am pasting the corrected code:

Lua Code:

-- libs
-- external /libs as ADDON_NAME.libs objects
ADDON_NAME.libs = {};
ADDON_NAME.libs.Config = LibStub("AceConfig-3.0");
ADDON_NAME.libs.ConfigDialog = LibStub("AceConfigDialog-3.0");
ADDON_NAME.libs.ConfigRegistry = LibStub("AceConfigRegistry-3.0");
ADDON_NAME.libs.DB = LibStub("AceDB-3.0");
ADDON_NAME.libs.Timer = LibStub("AceTimer-3.0");

Lua Code:

-- options
-- ADDON_NAME.default.global.MODULE_NAME
-- ADDON_NAME.options.args.MODULE_NAME
ADDON_NAME.default = {global = {}};
ADDON_NAME.options = {
    name = "ADDON_FULL_NAME",
    type = "group",
    args = {},
};

Lua Code:

-- add options
ADDON_NAME.db = ADDON_NAME.libs.DB:New("ADDON_NAMEDB", ADDON_NAME.default, true);
ADDON_NAME.libs.Config:RegisterOptionsTable("ADDON_NAME", ADDON_NAME.options);
ADDON_NAME.frames = {options = ADDON_NAME.libs.ConfigDialog:AddToBlizOptions("ADDON_NAME")};
ADDON_NAME.frames.options.default = function()
    ADDON_NAME.db:ResetDB("Default");
    ADDON_NAME.db = ADDON_NAME.libs.DB:New("ADDON_NAMEDB", ADDON_NAME.default, true);
    ADDON_NAME.libs.ConfigRegistry:NotifyChange("ADDON_NAME");
end;

I was working on the same thing last night- ha!

I did a little example addon for others in case they get stuck too. Not as detailed as your stuff, but might help others get off the ground. It is @ https://github.com/ChrisNolan/TestAd...hor=ChrisNolan