AlexFolland/AutoGear

Errors when WoW returns "HEXENMEISTER" for class name

croshy opened this issue · 6 comments

Using the burning crusade version

82x AutoGear\AutoGear.lua:1416: attempt to index field '?' (a nil value)
[string "@AutoGear\AutoGear.lua"]:1416: in function AutoGearSetStatWeights' [string "@AutoGear\AutoGear.lua"]:2255: in function AutoGearConsiderAllItems'
[string "@AutoGear\AutoGear.lua"]:1833: in function <AutoGear\AutoGear.lua:1610>

Locals:
class = "HEXENMEISTER"
spec = "Destruction"
(*temporary) = nil
(*temporary) = "HEXENMEISTER"
(*temporary) = "Destruction"
(*temporary) = 0
(*temporary) = "HEXENMEISTER"
(*temporary) = ""
(*temporary) = "Hexenmeister"
(*temporary) = "attempt to index field '?' (a nil value)"
weapons = nil

How strange. Blizzard localized the class constants? "Destruction" is right, but "HEXENMEISTER" isn't. This will take some research.

That being said, the stat scanning only works in English anyway, so any fix for this is unlikely to allow the expected usage of the addon.

That is sad. Your ad on has so much potential to make things a lot easier for a blind player.

Do you expect the problems not to appear when I switched my client to English?

I expect it'll work well with the game language set to English.

OK I tried that out with language said to English. But then our blind assistance add-ons we use won't work they are in German.

What would you need for a translation of your add-on? The guy that it's programming hour blind add-ons may have lists or other stuff for you you could get us assistance.
He made audio files of every singer item. Quest titles and all used words off the game 

I think a huge refactor that involves changing language-specific WoW API calls like parts that scan the tooltip to language-agnostic calls like GetItemStats would be necessary. For the class list, I'm really not sure why the class constant is different ("HEXENMEISTER"), since I thought those were meant to be language-agnostic, but I guess they're not. More research would be needed for changing almost every part of the addon.

It's all possible, and I intend to do it some day, but it hasn't been a priority because both Synthetikaryote and I play the game in English. I do want to support this, but I don't have time in my life these days to do such an extensive overhaul, with a full-time job and a family.

 I can absolutely understand that. I'm a father myself and the word of Warcraft shall be fun not work.

I still think you very much