Need help with a hard trigger

[GreyFox777]

Need your help guys, trying to get this trigger to work...

local localeicon = CreateFrame("Frame", "LocaleIcon", self) localeicon:RegisterEvent("PLAYER_ENTERING_WORLD") localeicon:RegisterEvent("PLAYER_TARGET_CHANGED") localeicon:SetFrameLevel(self.Health:GetFrameLevel() + 6) local localeicontexture = localeicon:CreateTexture(nil, "OVERLAY") localeicon:SetScript("OnEvent", function(self, event, ...) local servers = { ["Aggra(Português)"] = "Portuguese", ["AeriePeak"] = "English", ["Agamaggan"] = "English", ["Aggramar"] = "English", ["Ahn'Qiraj"] = "English", ["Al'Akir"] = "English" -- > ALL OTHER SERVERS -- ... -- ... } if select(2, UnitName("target")) and select(2, UnitName("target")) ~= '' then local serverName = select(2, UnitName("target")) if servers[serverName] and m.locale[servers[serverName]] then locale_texture:SetTexture(m.locale[servers[serverName]]) return true end else if (GetLocale() == "deDE") then localeicontexture:SetTexture(m.locale.ruRU) localeicontexture:Show() return true end localeicontexture:Hide() end end) localeicontexture:SetPoint('CENTER', self, 46.5, 136) localeicontexture:SetSize(19,19) localeicontexture:SetTexCoord(0, 1, 0, 1)

how do i insert the tables correctly?

My table file

m.locale = { ptBR = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ptBR", enGB = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_enGB", deDE = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_deDE", itIT = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_itIT", esES = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_esES", ruRU = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ruRU", frFR = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_frFR", }

[jukx]

You can just pass a function as a second argument to RegisterEvent:

local doStuff = function(self, event, ...)
    local servers = { 
        ["Aggra(Português)"] = "Portuguese", 
    }
    ...
end
 
localeicon:RegisterEvent("PLAYER_ENTERING_WORLD", doStuff)
localeicon:RegisterEvent("PLAYER_TARGET_CHANGED", doStuff)

---

edit for verbosity:

Right now you're passing function that's supposed to be executed as a callback to SetScript:

localeicon:RegisterEvent("PLAYER_ENTERING_WORLD")
localeicon:RegisterEvent("PLAYER_TARGET_CHANGED")
localeicon:SetScript("OnEvent", function(self, event, ...) 
 
    local servers = { 
        -- ...
    }
 
    if select(2, UnitName("target")) and select(2, UnitName("target")) ~= '' then
        -- ...
    end
end)

Since oUF rewrote the RegisterEvent function, this might not work out as planned. Instead, you can just define that function locally and pass it as a second argument to RegisterEvent() like I showed above:

local setRealmIcon = function(self, event, ...)
    local servers = { 
        -- ...
    }
 
    if select(2, UnitName("target")) and select(2, UnitName("target")) ~= '' then
        -- ...
    end
end
 
localeicon:RegisterEvent("PLAYER_ENTERING_WORLD", setRealmIcon)
localeicon:RegisterEvent("PLAYER_TARGET_CHANGED", setRealmIcon)

[GreyFox777]

local setRealmIcon = CreateFrame("Frame", "RealmIcon", self) local setRealmIcon = function(self, event, ...) local servers = { ["Aggra(Português)"] = "Portuguese", ["AeriePeak"] = "English", ["Agamaggan"] = "English", ["Aggramar"] = "English", ["Ahn'Qiraj"] = "English", ["Al'Akir"] = "English", ["Alonsus"] = "English", ... ... ... ... ["Ysondre"] = "French" } if select(2, UnitName("target")) and select(2, UnitName("target")) ~= '' then local serverName = select(2, UnitName("player")) if servers[serverName] and m.locale[servers[serverName]] then setRealmIcon:SetTexture(m.locale[servers[serverName]]) return true end end end setRealmIcon:RegisterEvent("PLAYER_ENTERING_WORLD", setRealmIcon) setRealmIcon:RegisterEvent("PLAYER_TARGET_CHANGED", setRealmIcon)

Don't seems to be working

the error:

Interface\AddOns\oUF_lumen\units\target.lua:849: attempt to index local 'setRealmIcon' (a function value)

[jukx]

For debugging (=making stuff work that doesn't work right now), I'd recommend installing BugSack and BugGrabber so you can see where the error is.

---

Also, you're calling m.locale[servers[serverName]] which won't work.
servers[serverName] returns something like "English" which is not a valid key in your table m.locale.
To make it work you'd have to match the keys to the content of the servers table:

m.locale = {
Portuguese = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ptBR",
English = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_enGB",
-- ...
}

---

Furthermore, if your question is how to insert the code for the table into the code above, just rename m.locale to locale and insert the table definition at the top, like this:

local locale = {
    ptBR = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ptBR",
    enGB = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_enGB",
    deDE = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_deDE",
    itIT = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_itIT",
    esES = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_esES",
    ruRU = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ruRU",
    frFR = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_frFR",
}
 
local localeicon = CreateFrame("Frame", "LocaleIcon", self)
localeicon:RegisterEvent("PLAYER_ENTERING_WORLD")
localeicon:RegisterEvent("PLAYER_TARGET_CHANGED")
localeicon:SetFrameLevel(self.Health:GetFrameLevel() + 6)
local localeicontexture = localeicon:CreateTexture(nil, "OVERLAY")
localeicon:SetScript("OnEvent", function(self, event, ...)
 
    local servers = { 
        -- ...
    }
 
    if select(2, UnitName("target")) and select(2, UnitName("target")) ~= '' then
        local serverName = select(2, UnitName("target"))
        if servers[serverName] 
            and m.locale[servers[serverName]] then
            locale_texture:SetTexture(locale[servers[serverName]])
            return true
        end
    else
        -- ...

[GreyFox777]

yes, it was already

m.locale = { ptBR = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ptBR", enGB = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_enGB", deDE = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_deDE", itIT = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_itIT", esES = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_esES", ruRU = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ruRU", frFR = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_frFR", Portuguese = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ptBR", English = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_enGB", German = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_deDE", Italian = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_itIT", Spanish = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_esES", Russian = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ruRU", French = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_frFR", }

[GreyFox777]

Now, it has zero errors, but still not working...

local locale = { ptBR = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ptBR", enGB = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_enGB", deDE = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_deDE", itIT = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_itIT", esES = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_esES", ruRU = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ruRU", frFR = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_frFR", } local localeicon = CreateFrame("Frame", "LocaleIcon", self) localeicon:RegisterEvent("PLAYER_ENTERING_WORLD") localeicon:RegisterEvent("PLAYER_TARGET_CHANGED") localeicon:SetFrameLevel(self.Health:GetFrameLevel() + 6) local localeicontexture = localeicon:CreateTexture(nil, "OVERLAY") localeicontexture:SetPoint('CENTER', self, 46.5, 136) localeicontexture:SetSize(19,19) localeicontexture:SetTexCoord(0, 1, 0, 1) localeicon:SetScript("OnEvent", function(self, event, ...) local servers = { ["Aggra(Português)"] = "Portuguese", ["AeriePeak"] = "English", ["Agamaggan"] = "English", ["Aggramar"] = "English", ... ... ... ["Varimathras"] = "French", ["Vol'jin"] = "French", ["Ysondre"] = "French" } if select(2, UnitName("target")) and select(2, UnitName("target")) ~= '' then local serverName = select(2, UnitName("target")) if servers[serverName] and locale[servers[serverName]] then localeicontexture:SetTexture(locale[servers[serverName]]) return true end end end) localeicontexture:Show() localeicontexture:SetPoint('CENTER', self, 46.5, 136) localeicontexture:SetSize(19,19) localeicontexture:SetTexCoord(0, 1, 0, 1)

Uiiii it works now!

I forget to add

Portuguese = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ptBR", English = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_enGB", German = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_deDE", Italian = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_itIT", Spanish = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_esES", Russian = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_ruRU", French = "Interface\\AddOns\\oUF_lumen\\media\\Locale_flag_frFR",

[Ammako]

Originally Posted by GreyFox777 Don't seems to be working the error:

For the record, you created a frame under that var name there, and then overwrote it with a function right after. Hence the error, because your frame is gone and :SetTexture is not valid for functions.

You've changed that later and have it working now, but you need to be careful with variable names not to overwrite other important stuff.


Also, you can do localicon.texture:CreateTexture(name, layer) to create a texture for your frame, without having to declare a second local variable. What you did would work too, but this way is cleaner and more straightforward.

You can add pretty much anything you want to frames that way. It doesn't specifically have to be called ".texture" for it to work as a texture (lua variables are typeless, you put anything you want in them), but it is recommended to use descriptive variable names so that's why I use that as the example.