Add Tracking Of Lumber Nodes #110


  • Fixed
  • Enhancment
Closed
Assigned to moonfann
  • SLOKnightFall created this issue Nov 15, 2014

    What is the enhancement in mind? How should it look and feel?
    In WoD building a Lumber Mill gives you the Logging ability which allows the harvesting of trees.  The trees show up on the minimap like other harvest nodes and it would be nice to be able to record the location of them.

    Please provide any additional information below.

    I was able to the tracking to GatherMate as a treasure type with the info below.

    In Collector.lua
    add:
    local loggingSpell = (GetSpellInfo(167895))
    Add to  spells table
    [loggingSpell] = "Treasure",

    Add to function Collector:UIError(event,msg)
        elseif strfind(msg, "Lumber Mill") then
            self:addItem(loggingSpell, what)

    In Constants
    Add to node_ids["Treasure"] table
            ["Small Timber"]        = 9999,
            ["Large Timber"]        = 9998,
            ["Timber"]        = 9997,
    Add to node_textures["Treasure"]
            [9997] = icon_path.."Treasure\\tree.tga",
            [9998] = icon_path.."Treasure\\tree.tga",
            [9999] = icon_path.."Treasure\\tree.tga",

  • SLOKnightFall added the tags New Enhancment Nov 15, 2014
  • SLOKnightFall edited description Nov 15, 2014
  • SLOKnightFall edited description Nov 15, 2014
  • _ForgeUser717952 posted a comment Nov 20, 2014

    Fix works fine apart from a missing "then"

    instead of

    elseif strfind(msg, "Lumber Mill")
            self:addItem(loggingSpell, what)
    

    it should be

    elseif strfind(msg, "Lumber Mill") then
            self:addItem(loggingSpell, what)
    

    Edited Nov 20, 2014
  • nevcairiel posted a comment Nov 21, 2014

    I don't think putting it into treasure is a good idea, since you can't easily hide them then on chars which do not have the lumber mill.

    I'll probably add a new storage type for them.

  • SLOKnightFall posted a comment Nov 21, 2014

    @Nevcairiel: Go

    True. I sort of figured that would be a better solution in the long run. Adding it to treasure was just a quick fix for myself. :)

    You also may need to put the logging spells in a table. Wowhead lists spellid 167896, 167897 as logging also and it may refer to the level 2 & 3 lumbermills

  • SLOKnightFall edited description Nov 21, 2014
  • speedwayst4r posted a comment Nov 23, 2014

    thanks for this.

  • speedwayst4r posted a comment Nov 23, 2014

    for this to work with Routes, you will need to add the following (for English locales):

    to Locales/GatherMate2-enUS.lua

    NL["Small Timber"] = true
    NL["Large Timber"] = true
    NL["Timber"] = true
    
  • nevcairiel removed a tag New Nov 23, 2014
  • nevcairiel added a tag Fixed Nov 23, 2014
  • nevcairiel closed issue Nov 23, 2014

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