refaim / AutoDB2

Vanilla WoW 1.12.1 addon. Automatically runs arbitrary /db commands for pfQuest users.
2 stars 0 forks source link

/mines auto does not work #2

Closed refaim closed 1 year ago

refaim commented 1 year ago

xxx: hey hope you don't mind me messaging you xxx: it looks like AutoDB 2.0 doesn't work with "mines auto" xxx: just "mines"works but not with auto xxx: I tried disabling any other addon than pfQuest and AutoDB, but it gives the same issue xxx: could it have something to do with not knowing the skill level yet at load? Or something in pfQuest not being ready yet? do you think it can be delayed somehow?

jendryk commented 1 year ago

It worked for me, did you use "/autodb mines" ?

refaim commented 1 year ago

Worked for me too, I used /autodb add mines auto

blehz-be commented 1 year ago

I retested it with a clean Addons and empty WTF folder. It kind of works (now), but only shows copper nodes while it should show up to iron nodes. When running it manually it does work.

refaim commented 1 year ago

I retested it with a clean Addons and empty WTF folder. It kind of works (now), but only shows copper nodes while it should show up to iron nodes. When running it manually it does work.

That's weird. I tested and saw copper and tin. Did you try to re-enter the game world or reload interface?

blehz-be commented 1 year ago

I login to my character, add the command with "/autodb add mines auto" -> this also runs it, which works --> this actually opens the map on a zone different than the zone I am in? Which is also weird to me? Does it do that for you too? It used to be Ashenvale, now it's Arathi Highlands. Then I logout and login again on the same character -> only shows Copper nodes as far as I can see. Run it manually -> shows only Iron nodes (I'm at mining skill 146).

blehz-be commented 1 year ago

I added this small debug function to the RunAutoexec function (inside the if) and the HandlePrintCommand function at the top. It prints 0 in the function, but works in the list function.

So the skill info is not yet loaded it seems. Can this hook in or wait for pfQuest or whatever is needed in the WoW API to get initialized?

local function PrintSkills()
    AutoDB.info("Skills: " .. GetNumSkillLines())
    for i=0,GetNumSkillLines() do
        local skillName, _, _, skillRank = GetSkillLineInfo(i)
        if skillName ~= nil then
            AutoDB.info("Skill " .. skillName .. "rank: " .. skillRank)
        end
    end
end
refaim commented 1 year ago

@blehz-be thanks a lot! Your research has been very helpful. I fixed the issue, please try the new version.

blehz-be commented 1 year ago

That's fixed, thank you!