Not signed in (Sign In)
    • CommentAuthorzvelena
    • CommentTimeNov 18th 2006
     
    Hello,

    I have an issue with Barker included with the version 3.8.0 of Enchantrix where Crusader is displayed as +75 str in the trade message. Does anyone have a suggestion about what and where I need to make changes to display it correctly ?

    Thank you
    Elena
    •  
      CommentAuthorNemelis
    • CommentTimeNov 19th 2006
     
    Crusader + Barker... Hmm sounds familiar... Search on the forum please, since there is already is a post about it, if I remember correct.
    • CommentAuthorzvelena
    • CommentTimeNov 19th 2006
     
    Yes, you're right, there is an entire thread on the site. However i can only manage to display it as 75 Crusader, but its good enough for me.... Thank you for this great addon.
    • CommentAuthorYouri
    • CommentTimeDec 7th 2006
     
    I managed to fix this for 3.8 and posted how to it in the other thread which you could have searched for. Note: there has been some discussion about this because it makes the code a bit messy (instead of getting a correct the detection, it does a dirty trick to get rid of the +75). It is not the desired way to fix it, but for the time being, it works just fine.

    Considering you already know how to change it into +75 Crusader, I can skip that explanation.

    The function EnchantrixBarker_GetCraftDescription (index) needed to be changed a bit. In front of the line:
    return "+"..statvalue' '..short_attributes[index]

    I added 3 other lines:

    if (short_attributes[index] == "Crusader") then
    return "Crusader"
    end
World of Warcraft™ and Blizzard Entertainment™ are trademarks or registered trademarks of Blizzard Entertainment, Inc. in the U.S. and/or other countries.