newbie needs help with his addon

[Sk4lli]

Hi my name is Sk4lli,
i'm completely new to Lua scripting and thought ESO is a great game so i could give it a try.

in my First addon i would gather information about the Player Character and save it to the "Saved Variables" but there is already the problem because it won't save the variables, i don't know where i have made a mistake.

user:/AddOns/CharHarvester/Charharvester.lua:62: attempt to index a nil value stack traceback
user:/AddOns/CharHarvester/Charharvester.lua:62: in function 'CH.MotifKnown'
EsoUI/Ingame/SlashCommands/SlashCommands.lua:108: in function 'DoCommand'
EsoUI/Ingame/ChatSystem/ChatSystem.lua:1849: infunction 'ChatSystem:SubmitTextEntry'
EsoUI/Ingame/ChatSystem/ChatSystem.lua:2682: in Function 'TO_ChatTextEntry_Execute'
14406795741938050902:3: infunction '(main chunk)'
(tail call):?

The Lua File:

Warning: Spoiler

CH = {}
CH.name = "CharHarvester"
CH.version = "0.0A"
 
 
 
 CH.defautls ={
 data       = {},
 motifs = {},
 }
 
 
function CH:Initialize()
  
    CH.vars = ZO_SavedVars:NewAccountWide("CharHarvesterVars", 1, nil, CH.default)
    
        
end
function CH.init(eventCode, addOnName)
    if addonName == CH.name then
        CH:Initialize() 
        d("IT WORKS")
 
    end
end
-- Locals 
 
CH.race                 = GetUnitRace("player")
CH.playername           = GetUnitName("player")
CH.level                = GetUnitLevel("player")
CH.vlevel               = GetUnitVeteranRank("player")
CH.class                = GetUnitClassId("player")
CH.gender               = GetUnitGender("player")
 
 
function CH.AssignMotifs()
CH.motifsList           = {}
CH.motifsList[1]        = 2
CH.motifsList[2]        = 3
CH.motifsList[3]        = 4
CH.motifsList[4]        = 5
CH.motifsList[5]        = 6
CH.motifsList[6]        = 7
CH.motifsList[7]        = 8
CH.motifsList[8]        = 9
CH.motifsList[9]        = 10
CH.motifsList[10]       = 16
CH.motifsList[11]       = 18
CH.motifsList[12]       = 20
CH.motifsList[13]       = 21
CH.motifsList[14]       = 35
 
end
 
 
--Locals End
 
-- Get Motif data
 
 
function CH.MotifKnown ()
    CH.vars.motifs[CH.playername] = {}
    local m, i
    for m = 1, 14 do
        i = CH.motifsList[m]
        CH.vars.motifs[CH.playername][m] = IsSmithingStyleKnown(i)
    end
end
 
 
-- End
 
local function reload()
    ReloadUI()
end
 
local function harvester(withInput)
 
        d("alea iacta est (try typing /CharHarvest Clear)")
        
    if withInput == "Clear" then    
        d("Die Würfel sind Gefallen")
    end
end 
 
 --Events
EVENT_MANAGER:RegisterForEvent("CH", EVENT_ADD_ON_LOADED, CH.init) 
 --Events END
 
 
SLASH_COMMANDS["/ch"]= harvester
SLASH_COMMANDS["/rl"]= reload
SLASH_COMMANDS["/chtest"]= CH.MotifKnown

i hope you guys can help me.

greetz Sk4lli

[katkat42]

You have a typo where you declare your "CH.defautls", er, "CH.defaults" variable.

[Sk4lli]

seriously? a Typo? ^^ thanks mate

the "savedVars" file is generating now .. but the error still appears

[Garkin]

- You are trying to get character information at the time when it is not available yet.
- It seems that you never call function CH.AssignMotifs() so the table CH.motifsList doesn't exists.
- Variable "addonName" in the CH.init function does not exist, name of argument you have defined is "addOnName".
- As katkat42 said, you have typo CH.defaults table name -> you have defined table "CH.defautls" and you are trying to use "CH.default".

Modified code:

Warning: Spoiler

CH = {}
CH.name = "CharHarvester"
CH.version = "0.0A"
 
CH.defaults = {
   data  = {},
   motifs = {},
}
 
CH.motifsList = {
   [1]  = ITEMSTYLE_RACIAL_HIGH_ELF,   --Racial Motifs 1: The High Elves
   [2]  = ITEMSTYLE_RACIAL_DARK_ELF,   --Racial Motifs 2: The Dark Elves
   [3]  = ITEMSTYLE_RACIAL_WOOD_ELF,   --Racial Motifs 3: The Wood Elves
   [4]  = ITEMSTYLE_RACIAL_NORD,       --Racial Motifs 4: The Nords
   [5]  = ITEMSTYLE_RACIAL_BRETON,     --Racial Motifs 5: The Bretons
   [6]  = ITEMSTYLE_RACIAL_REDGUARD,   --Racial Motifs 6: The Redguards
   [7]  = ITEMSTYLE_RACIAL_KHAJIIT,    --Racial Motifs 7: The Khajiit
   [8]  = ITEMSTYLE_RACIAL_ORC,        --Racial Motifs 8: The Orcs
   [9]  = ITEMSTYLE_RACIAL_ARGONIAN,   --Racial Motifs 9: The Argonians
   [10] = ITEMSTYLE_RACIAL_IMPERIAL,   --Racial Motifs 10: Imperial Cyrods
   [11] = ITEMSTYLE_AREA_ANCIENT_ELF,  --Racial Motifs 11: Ancient Elves
   [12] = ITEMSTYLE_AREA_REACH,        --Racial Motifs 12: Barbaric
   [13] = ITEMSTYLE_ENEMY_PRIMITIVE,   --Racial Motifs 13: Primal
   [14] = ITEMSTYLE_ENEMY_DAEDRIC,     --Racial Motifs 14: Daedric
}
 
function CH.Initialize()
   CH.vars = ZO_SavedVars:NewAccountWide("CharHarvesterVars", 1, nil, CH.defaults)
 
   CH.race                 = GetUnitRace("player")
   CH.playername           = GetUnitName("player")
   CH.level                = GetUnitLevel("player")
   CH.vlevel               = GetUnitVeteranRank("player")
   CH.class                = GetUnitClassId("player")
   CH.gender               = GetUnitGender("player")
end
 
function CH.init(eventCode, addOnName)
   if addOnName == CH.name then
      CH.Initialize()
      EVENT_MANAGER:UnregisterForEvent(CH.name, eventCode)
   end
end
 
-- Get Motif data
function CH.MotifKnown()
   CH.vars.motifs[CH.playername] = {}
   for m, i in ipairs(CH.motifsList) do
      --IsSmithingStyleKnown(styleIndex), styleIndex = itemStyle + 1
      CH.vars.motifs[CH.playername][m] = IsSmithingStyleKnown(i+1)
      d(GetString("SI_ITEMSTYLE", i) .. (IsSmithingStyleKnown(i+1) and " |c00FF00known|r" or " |cFF0000unknown|r"))
   end
end
 
local function reload()
   ReloadUI("ingame")
end
 
local function harvester(withInput)
   d("alea iacta est (try typing /CharHarvest Clear)")
       
   if withInput == "Clear" then   
      d("Die Würfel sind Gefallen")
   end
end
 
EVENT_MANAGER:RegisterForEvent(CH.name, EVENT_ADD_ON_LOADED, CH.init)
 
SLASH_COMMANDS["/ch"]= harvester
SLASH_COMMANDS["/rl"]= reload
SLASH_COMMANDS["/chtest"]= CH.MotifKnown

[merlight]

Originally Posted by Sk4lli --     local m, i     for m = 1, 14 do         i = CH.motifsList[m]

Lua language corner for loop control variable is implicitly local to the loop, you don't have to declare it. Actually your 'local m' is a different variable, shadowed by the 'for m' inside the loop. And to further simplify iterating an array-like table, use ipairs:

-- used your names, but had to think twice, since conventionally 'i' is the index variable ;)
for m, i in ipairs(CH.motifsList) do ...

[Sk4lli]

@Garkin and merlight

thank you both, now I'm a big step further with my plan.

But it shows me I still have much to learn.


greetings Sk4lli