mining nodes not showing (ru locale) - fix #159


Closed
  • lazy_n0ob created this issue Apr 4, 2018

     Depending on mining skill level, there are two different skill names in Ru: "Горное дело" and "Добывание". You can check this at http://ru.wowhead.com/spell=74517/mining

     

    Seems that you only check the low-level one in Collector:SpellStarted().
    So, here's a quick little fix. 

     

    Collector.lua:

    local miningSpell = (GetSpellInfo(195122))
    local miningSpell2 = (GetSpellInfo(2575))
    
    ...
    
    local spells = { -- spellname to "database name"
    	[miningSpell] = "Mining",
    	[miningSpell2] = "Mining",
    	[herbSpell] = "Herb Gathering",
    	[fishSpell] = "Fishing",
    	[gasSpell] = "Extract Gas",
    	[openSpell] = "Treasure",
    	[openNoTextSpell] = "Treasure",
    	[pickSpell] = "Treasure",
    	[archSpell] = "Archaeology",
    	[sandStormSpell] = "Treasure",
    	[loggingSpell] = "Logging",
    }

    So far I've only checked it on a few copper (lvl 20 character, mining skill 1) and leystone nodes (lvl 110, mining skill 255) - seems ok.
    Also, there's a reference to miningSpell in Collector:UIError, but I don't know much about programming, so I'll leave fixing it to you guys =)

  • Namestaken_ posted a comment Jul 23, 2018

    I'm having the same issue with archaeology -- no minimap icons for nodes.

  • nevcairiel posted a comment Jul 30, 2018

    Added the second alternative mining spell.

  • nevcairiel closed issue Jul 30, 2018

To post a comment, please login or register a new account.