b-morgan / Skillet-Classic

World of Warcraft Classic addon
GNU General Public License v3.0
6 stars 6 forks source link

Update ScrapeWowhead.js #163

Closed matthewhively closed 9 months ago

matthewhively commented 9 months ago

Updated to account for pure spells within professions other than enchanting. Example: Northrend Alchemy Research

I tested this with Alchemy, Enchanting and Inscription (new in wotlk)... but it should work for everything.

I'm making an assumption that "pure spells" like Northrend Alchemy Research should also be handled by skillet by a - prefix before the ID in the table... if this is wrong you'll need to add special handling within the print_skill_levels() function to differentiate between enchanting and other pure spells.

This adds some error checking incase other weirdness with the 2 parsable links shows up in the future. It'll just skip a recipe in that case, but it'll report it so you can take care of it manually (hopefully)

It is currently unknown whether there are additional weird cases that would need special handling