• 1 page(s)
Help fixing my addon

Hi. Back when I played WoW I created an addon by the name of Skills (http://wow.curse.com/downloads/wow-addons/details/skills.aspx).

What it does is it has a list containing all the skills which a player can train.
To check if a skill has been trained the spellbook is checked. This worked fine until Death Knight came along.

Apparently some skills are still beeing listed as untrained even though they aren't. Could this be because of the skills below not being the same pages that the normal skills?

 

The skills:
[12:08:09] : Listing all your untrained skills: 
[12:08:09] Level 40: [Rune of Cinderglacier] 
[12:08:09] Level 40: [Rune of Lichbane] 
[12:08:09] Level 40: [Rune of Razorice] 
[12:08:09] Level 40: [Rune of Spellbreaking] 
[12:08:09] Level 40: [Rune of Spellshattering] 
[12:08:09] Level 40: [Rune of Swordbreaking] 
[12:08:09] Level 40: [Rune of Swordshattering] 
[12:08:09] Level 40: [Rune of the Fallen Crusader] 
[12:08:09] Level 55: [Acherus Deathcharger, Summon] 
[12:08:09] Level 55: [Death Coil] 
[12:08:09] Level 55: [Death Coil] 
[12:08:09] Level 55: [Frozen Rune Weapon] 

Report this thread post

I believe that runesmithing is treated as a tradeskill and not a class skill. Not entirely sure :)

See my "Thread to end all threads thread" for the answer to your question!

Report this thread post

  Quote:
Originally Posted by teek5449 Go to post by >teek5449

I believe that runesmithing is treated as a tradeskill and not a class skill. Not entirely sure :)

I'm sure it is, any ideas on how to fix it.

ATM I'm using the method which checks if spell XXXXX (where XXXXX is the spells ID) is in the spellbook

Report this thread post

You will have to treat the runeforging differently than a skill. I think you will have to predefine a table with all of the possible runeforging skills like the ones that you already have. Then you will have to scan the known tradeskills and compare them against your table. That will give you the missing skills.

See my "Thread to end all threads thread" for the answer to your question!

Report this thread post
  • 1 page(s)
Subscribe to this thread: (you will receive emails when new posts are made)