Ace3 convertion

[Kygo]

Greetings.
I wrote a small addon to assist me with my crappy memory and I got the brilliant idea to try to convert it into a Ace3 addon (Yay!). But as soon as I started I did encounter problems that I have absolutley no idea how to solve.

Source code for the whole addon can be found here > https://github.com/Kygos/kNameHider

It works, sort of. I can see the "kNameHider" tab in the interface addons panel but it does not show anything at all. Currently the "get" and "set" for the toggle are blanks because I havent found the correct function to call (yet!).

It does generate an error tho when I click "kNameHider" in the options panel about it expected a options table but got "kNameHider".

Message: ...nfig-3.0\AceConfigDialog-3.0\AceConfigDialog-3.0.lua:1803: AceConfigRegistry-3.0:ValidateOptionsTable(): kNameHider.handler: expected a table, got 'kNameHider'
Time: 05/31/15 12:49:10
Count: 1
Stack: (tail call): ?
[C]: ?
...-3.0\AceConfigRegistry-3.0\AceConfigRegistry-3.0.lua:50: in function <...-3.0\AceConfigRegistry-3.0\AceConfigRegistry-3.0.lua:45>
...-3.0\AceConfigRegistry-3.0\AceConfigRegistry-3.0.lua:187: in function <...-3.0\AceConfigRegistry-3.0\AceConfigRegistry-3.0.lua:182>
...-3.0\AceConfigRegistry-3.0\AceConfigRegistry-3.0.lua:221: in function <...-3.0\AceConfigRegistry-3.0\AceConfigRegistry-3.0.lua:196>
...-3.0\AceConfigRegistry-3.0\AceConfigRegistry-3.0.lua:259: in function `ValidateOptionsTable'
...-3.0\AceConfigRegistry-3.0\AceConfigRegistry-3.0.lua:301: in function `app'
...nfig-3.0\AceConfigDialog-3.0\AceConfigDialog-3.0.lua:1803: in function `Open'
...nfig-3.0\AceConfigDialog-3.0\AceConfigDialog-3.0.lua:1896: in function <...nfig-3.0\AceConfigDialog-3.0\AceConfigDialog-3.0.lua:1894>
(tail call): ?
[C]: ?
[string "safecall Dispatcher[2]"]:9: in function <[string "safecall Dispatcher[2]"]:5>
(tail call): ?
...ace\AddOns\AddOnSkins\Libs\AceGUI-3.0\AceGUI-3.0.lua:314: in function `Fire'
...GUI-3.0\widgets\AceGUIContainer-BlizOptionsGroup.lua:20: in function <...GUI-3.0\widgets\AceGUIContainer-BlizOptionsGroup.lua:19>
[C]: in function `Show'
Interface\FrameXML\InterfaceOptionsFrame.lua:28: in function `InterfaceOptionsList_DisplayPanel'
Interface\FrameXML\InterfaceOptionsFrame.lua:46: in function `InterfaceOptionsListButton_OnClick'
[string "*:OnClick"]:2: in function <[string "*:OnClick"]:1>
 
Locals: <none>

Cheers folks!

[Gethe]

That error occured because you gave it a string instead of a table. You already have a table variable named kNameHider (created by AceAddon), so just remove the quotes from that field to use it.

local options = {
    name = "kNameHider",
    handler = kNameHider,
    type = "group",
    args = {
        ...
    }
}

If you haven't already, I recommend looking at the Ace3 docs.

[Kygo]

Originally Posted by Gethe That error occured because you gave it a string instead of a table. You already have a table variable named kNameHider (created by AceAddon), so just remove the quotes from that field to use it. local options = { name = "kNameHider", handler = kNameHider, type = "group", args = { ... } } If you haven't already, I recommend looking at the Ace3 docs.

A simple solution to my pretty dumb problem. Thanks Gethe!

[Seerah]

Remember that Ace3 isn't "magic-sauce" that will make an addon better.

[Kygo]

Originally Posted by Seerah Remember that Ace3 isn't "magic-sauce" that will make an addon better.

I know Just thought it would be good practice to add it into a small non-complex addon

[Fizzlemizz]

Originally Posted by Kygo I know Just thought it would be good practice to add it into a small non-complex addon

That pretty much defeats the purpose of a small non-complex addon if it's not needed

[Kygo]

Originally Posted by Fizzlemizz That pretty much defeats the purpose of a small non-complex addon if it's not needed

It's not needed at all to add Ace3 components to an addon such as this, just doing it because it's rather good practice for a person like me to learn even more on how stuff works ^^

[Seerah]

I only ever use AceConfig (and AceGUI, LibStub, and CallbackHandler as its dependencies) if my addon has more than just a few options. You don't really need anything else in most projects.