GetSpecialization() doesn't return correctly right upon login - workaround?

[jostor]

I'm writing an addon where which frames are shown depends a bit on your talent spec, and it seems that right as you log in GetSpecialization() sometimes doesn't return a correct value, most likely because your spec and so on hasn't been loaded in yet.
I am trying to use it in the ADDON_LOADED event, and it seems to work correctly about 50% of the time when logging in, and when I just reload my UI it works more or less 100% of the time.

Is there a neat workaround for this? Is there a specific order that events are fired as you log in?

[Dridzt]

http://www.wowpedia.org/AddOn_loadin...iables_loading

Don't take this as some kind of bible, event order has changed in the past but it's relatively trustworthy.
PLAYER_LOGIN is the event you want for initializing game data most of the time.

[ravagernl]

I had the same issue with CRITERIA UPDATE firing twice when logging in, the first one was right after ADDON_LOADED, the second after PLAYER_LOGIN. GetAchievementCriteriaInfo() returned completed data on the second occurence.

[jostor]

PLAYER_LOGIN seems to work better and I wasn't able to trigger the error with using that instead, so thanks for that

Is there any way that PLAYER_LOGIN could be before ADDON_LOADED in extreme situations? My code right now relies on ADDON_LOADED going first.

[ravagernl]

Originally Posted by jostor Is there any way that PLAYER_LOGIN could be before ADDON_LOADED in extreme situations? My code right now relies on ADDON_LOADED going first.

This is how I would solve it, assuming you are using a table style event handling approach:

function addon:ADDON_LOADED(arg1)
    if arg1 ~= addon.name then return end
    -- do stuff here
    if IsLoggedIn() then
        self:PLAYER_LOGIN()
    else
        self:RegisterEvent('PLAYER_LOGIN')
    end
end
 
function addon:PLAYER_LOGIN()
    -- do stuff here
end

If you are running an if/else style event handler, just call the event handler function from the function itself, with parameters:

local OnEvent = function(frame, event, ...)
    if event == 'ADDON_LOADED' and ... == addonName then
        -- do stuff
        if IsLoggedIn() then
            OnEvent(frame, 'PLAYER_LOGIN')
        else
            frame:RegisterEvent'PLAYER_LOGIN'
        end
    elseif bla then
        -- Bla
    end
end
frame:RegisterEvent'ADDON_LOADED'

[jeruku]

Have you tried PLAYER_ALIVE or PLAYER_ENTERING_WORLD?

[Phanx]

Originally Posted by jeruku Have you tried PLAYER_ALIVE or PLAYER_ENTERING_WORLD?

If you use either of those, make sure you unregister them after seeing them, as they don't just fire on login -- they also fire every time you release your spirit, accept a resurrection, or pass through a loading screen.

[lilsparky]

for cases like this where it's ambiguous what data is available when, you can set up a timer to periodically run a simple function that checks for good return results (ie, not nil) on functions you're interested in. if it makes it thru the series of checks, then it kills the timer and fires off the your actual initialization function.

i had to do this to check when profession information is available as no events seem to work 100% of the time. also makes it future proof should blizzard re-arrange event order.