Post by jf647
I had put this in
another wowhead client error thread:
The issue (for me at least) is that the data simply isn't being scanned.
My goal is simply to see what cooking recipes I'm missing on a single character. I only have the wowhead looter addon enabled for that one character.
I delete the SV lua file, then log in, open my cooking (and first aid and smelting panels), then type /wl collect.
I then log out and open the SV lua file. It's quite clear that the recipes list has not been scanned, because this snippet appears in wlExportData:
&profs=2656,2550,3273&recipes=&mounts=75207
I'm sure the upload would work fine if the addon was actually scanning the recipe list, but it's not.
I think this may be related to 4.0.6, because another addon that I have that shades all known recipes as red on vendors (like the daily cooking vendor in SW) is no longer doing that. Presumably it would also have to scan what recipes I know to work properly.
I haven't gone through the code for either addon, but this suggests that something changed in the API for scanning which recipes you have on which tradeskill.
Having now had the chance to hack through the code a bit, I've found the problem and a fix.
The issue is that the spell name for spell id 2656 is "Smelting", but the name returned from GetTradeSkillLine is "Mining".
In the wlScanProfession function, there is a check like this:
local skillLineName = GetTradeSkillLine()
...
if not seenProfessions then return end
The first time you hit wlScanProfession and that "if not seenProfessionsskillLineName = wlProfessionSkillNameToSpellName or skillLineName
right after
local skillLineName = GetTradeSkillLine()
in the wlScanProfession() function.
I've done this and confirmed that the recipe list on the affected character is now populated.
Cheers