ADDON_LOADED fired multiple times?

[Yukyuk]

Am developing my first addon and today I noticed something strange.
I have a logging funtion that sends me messages to the default chat window to see what is happening within my addon.
The ADDON_LOADED event gets called several (a dozen, didn't count) times.
Is that default behavior or could there be a fault inside the coding of my addon?

Below is the part that sets up and registers the events.

--  Set up the frame for the event handler
Historia_frame = CreateFrame("Frame");

Historia_frame:SetScript("OnEvent", function(self,event, ...)
		Historia.EventManager(self,event,...); -- call event handler
			end);
			
 Historia_frame:RegisterEvent("ADDON_LOADED");
 Historia_frame:RegisterEvent("PLAYER_ENTERING_WORLD");
 Historia_frame:RegisterEvent("PLAYER_LEAVING_WORLD");
 Historia_frame:RegisterEvent("PLAYER_LEVEL_UP");
 Historia_frame:RegisterEvent("PLAYER_MONEY")

[myrroddin]

ADDON_LOADED fires for every AddOn the game loads, user developed or Blizzard in origin.

local MyAddOn, private_table = ...

private_table.eventFrame = CreateFrame("Frame")
private_table.eventFrame:RegisterEvent("ADDON_LOADED")

private_table.eventFrame:SetScript("OnEvent", function(self, event, ...)
    private_table.eventFrame[event](self, ...) -- call event handlers
end)

function private_table.eventFrame:ADDON_LOADED(AddOn)
    if not AddOn == MyAddOn then
        return -- not my AddOn
    end
    -- do stuff because this is your AddOn
    -- usually, the first thing you do here is Unregister ADDON_LOADED
    -- because you no longer need it
    -- now set up your saved variables and other "run once" stuff
end

[Yukyuk]

Thank you for your quick response.
Worked like a charm.

Made one adjustment to the code you posted.

if not AddOn == MyAddOn then

did not work for some reason.
Replaced it by

if AddOn ~= addon_name then

Now works perfect.

[SDPhantom]

It's customary practice to unregister the event when you're done with it and don't need it to fire anymore. To do this, you just run self:UnregisterEvent("ADDON_LOADED") at the end of the function.

[Yukyuk]

Thx for the advice SDPhantom.
Was also advised by myrroddin.

And who I am to ignore the advice from two experienced addon developers
ADDON_LOADED is now unregistered after it has been handled.

[Torhal]

Originally Posted by Yukyuk Thank you for your quick response. Worked like a charm. Made one adjustment to the code you posted. if not AddOn == MyAddOn then did not work for some reason. Replaced it by if AddOn ~= addon_name then Now works perfect.

That's because "not AddOn" evaluates to false if AddOn is non-nil and true otherwise. So that line is essentially saying "if false/true == MyAddOn then" which isn't at all what you want for a string comparison.