Assistance needed for registering events

[jtsmith1287]

So, I'm learning lua and the ESO API. I had an idea for a super simple addon, but getting just the basics down is proving to be a nightmare for me. I'm going to (embarrassingly) paste my code and ask that you guys tell me where I'm going wrong. What I'm looking to accomplish is getting the information of the items I'm looting, but I can't even get the dang loot event registered. And excuse the weird variable names--lots of poking and testing has been going on, as well as changing my addon idea a few times after learning of API restrictions. That and some of this is pasted examples from the esoui wiki.

When I gain loot, I want to see these items printed into chat via d() or something so I can start figuring this out. Thanks in advance for the help.

PileOMobs.lua

function lootReceived(receivedBy, itemName, quantity, itemSound, lootType, self)

  Loot:SetText("You looted stuff!!!")
  d(itemName)

  end
  
function RegisterLootEvent(eventcode, name)

  if (name == "PileOMobs") then
    PileOMobs:RegisterForEvent(EVENT_LOOT_RECEIVED, lootReceived)
    Loot:SetText("Loot event registered.")
    end

  end

function PileOMobsInit()

  EVENT_MANAGER:RegisterForEvent("PileOMobs", EVENT_ADD_ON_LOADED, RegisterLootEvent)

  end

PileOMobs.xml

<GuiXml>
    <Controls>
        <TopLevelControl name="Whatever">
            <Dimensions x="250" y="80" />
            <Anchor point="RIGHT" />

            <OnInitialized>
                PileOMobsInit()
            </OnInitialized>
 
            <Controls>
                <Label name="Loot" font="ZoFontGame" color="CFDCBD" wrapMode="ELLIPSIS" verticalAlignment="TOP" text="Got yerself a errawr">
                    <AnchorFill />
                </Label>
            </Controls>
        </TopLevelControl>
    </Controls>
</GuiXml>

[Tar000un]

A good practice is to prefix your public function with the addon name. You will avoid few kinds of conflict. As example :

PileOMobs.lootReceived

I also saw with a :, instead the dot.


And i think you missed a name on the second, and got a wrong controller.

PileOMobs:RegisterForEvent(EVENT_LOOT_RECEIVED, lootReceived)

Try to type it like the first, what will be like :

EVENT_MANAGER:RegisterForEvent("PileOMobs", EVENT_LOOT_RECEIVED, lootReceived)

[Xrystal]

You're almost there but but not quite got it right.

Give this slight change a go

local function lootReceived(eventCode,receivedBy, itemName, quantity, itemSound, lootType, self)
  Loot:SetText("You looted stuff!!!")
  d(itemName)
end
  
local function RegisterLootEvent(eventcode, name)
  if (name == "PileOMobs") then
    EVENT_MANAGER:RegisterForEvent("PileOMobs",EVENT_LOOT_RECEIVED, lootReceived)
    Loot:SetText("Loot event registered.")   --- Note: Message probably won't appear at this stage
  end
end
 
function PileOMobsInit(self)
  self:RegisterForEvent("PileOMobs", EVENT_ADD_ON_LOADED, RegisterLootEvent)
end

[Sasky]

Originally Posted by Tar000un I also saw with a :, instead the dot.

In Lua, using the colon ( : ) instead of the dot makes a variable "self" available in the function.

For example:

local foo = {}
foo.val = 1
 
function foo:bar()
    d(self.val) --Prints 1
end

When you fully define the tables, it's not as critical, since you can simply use the table name instead:

function foo.bar()
    d(foo.val) --Prints 1
end

Where it's most useful is object-oriented programming style, where you have the same function with different objects and need self to differentiate between them.

[Tar000un]

Thanks for the enlightenment !