-
-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Remove hack and retail profession scanning
- Loading branch information
Showing
3 changed files
with
66 additions
and
197 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,117 +1,79 @@ | ||
| local _, addon = ... | ||
|
|
||
| if addon:IsRetail() then | ||
| local professionSkillLines = {} | ||
| -- classic only | ||
| local PROFESSION_IDS = { | ||
| -- these are all the apprentice-level spells, except herbalism since there's no matching | ||
| -- apprentice skill for it, an item effect is used instead | ||
| [(GetSpellInfo(2259))] = 171, -- Alchemy | ||
| [(GetSpellInfo(2018))] = 164, -- Blacksmithing | ||
| [(GetSpellInfo(7411))] = 333, -- Enchanting | ||
| [(GetSpellInfo(4036))] = 202, -- Engineering | ||
| [(GetSpellInfo(9134))] = 182, -- Herbalism | ||
| [(GetSpellInfo(45357)) or 0] = 773, -- Inscription | ||
| [(GetSpellInfo(25229)) or 0] = 755, -- Jewelcrafting | ||
| [(GetSpellInfo(2108))] = 165, -- Leatherworking | ||
| [(GetSpellInfo(2575))] = 186, -- Mining | ||
| [(GetSpellInfo(8613))] = 393, -- Skinning | ||
| [(GetSpellInfo(3908))] = 197, -- Tailoring | ||
| } | ||
|
|
||
| function addon:PLAYER_LOGIN() | ||
| for _, skillLineID in next, C_TradeSkillUI.GetAllProfessionTradeSkillLines() do | ||
| local skillInfo = C_TradeSkillUI.GetProfessionInfoBySkillLineID(skillLineID) | ||
| if skillInfo.parentProfessionID and skillInfo.isPrimaryProfession then | ||
| if not professionSkillLines[skillInfo.parentProfessionID] then | ||
| professionSkillLines[skillInfo.parentProfessionID] = {} | ||
| end | ||
| local professionSkillLevels = {} | ||
| function addon:GetProfessionSkillLevel(professionID) | ||
| return professionSkillLevels[professionID] or 0 | ||
| end | ||
|
|
||
| professionSkillLines[skillInfo.parentProfessionID][skillLineID] = true | ||
| end | ||
| end | ||
| end | ||
| function addon:SKILL_LINES_CHANGED() | ||
| table.wipe(professionSkillLevels) -- we have to do this in case the player unlearns a profession | ||
|
|
||
| local skillLineLevels = {} | ||
| local function updateSkillLines() | ||
| for _, skillLines in next, professionSkillLines do | ||
| for skillLineID in next, skillLines do | ||
| local skillInfo = C_TradeSkillUI.GetProfessionInfoBySkillLineID(skillLineID) | ||
| if skillInfo.maxSkillLevel > 0 then | ||
| skillLineLevels[skillLineID] = skillInfo.skillLevel | ||
| else | ||
| skillLineLevels[skillLineID] = nil | ||
| end | ||
| for index = 1, GetNumSkillLines() do | ||
| local skillName, isHeader, isExpanded, skillLevel = GetSkillLineInfo(index) | ||
| if skillName == TRADE_SKILLS and isHeader and not isExpanded then | ||
| ExpandSkillHeader(index) | ||
| return | ||
| else | ||
| local professionID = PROFESSION_IDS[skillName] | ||
| if professionID then | ||
| professionSkillLevels[professionID] = skillLevel | ||
| end | ||
| end | ||
| end | ||
| end | ||
|
|
||
| addon:RegisterEvent('PLAYER_LOGIN', updateSkillLines) | ||
| addon:RegisterEvent('TRADE_SKILL_SHOW', updateSkillLines) | ||
| addon:RegisterEvent('SKILL_LINES_CHANGED', updateSkillLines) | ||
| addon:RegisterEvent('TRADE_SKILL_LIST_UPDATE', updateSkillLines) | ||
|
|
||
| function addon:GetProfessionSkillLevel(skillLineID) | ||
| return skillLineLevels[skillLineID] or 0 | ||
| end | ||
| else | ||
| local PROFESSION_IDS = { | ||
| -- these are all the apprentice-level spells, except herbalism since there's no matching | ||
| -- apprentice skill for it, an item effect is used instead | ||
| [(GetSpellInfo(2259))] = 171, -- Alchemy | ||
| [(GetSpellInfo(2018))] = 164, -- Blacksmithing | ||
| [(GetSpellInfo(7411))] = 333, -- Enchanting | ||
| [(GetSpellInfo(4036))] = 202, -- Engineering | ||
| [(GetSpellInfo(9134))] = 182, -- Herbalism | ||
| [(GetSpellInfo(45357)) or 0] = 773, -- Inscription | ||
| [(GetSpellInfo(25229)) or 0] = 755, -- Jewelcrafting | ||
| [(GetSpellInfo(2108))] = 165, -- Leatherworking | ||
| [(GetSpellInfo(2575))] = 186, -- Mining | ||
| [(GetSpellInfo(8613))] = 393, -- Skinning | ||
| [(GetSpellInfo(3908))] = 197, -- Tailoring | ||
| } | ||
|
|
||
| local professionSkillLevels = {} | ||
| function addon:GetProfessionSkillLevel(professionID) | ||
| return professionSkillLevels[professionID] or 0 | ||
| end | ||
|
|
||
| function addon:SKILL_LINES_CHANGED() | ||
| table.wipe(professionSkillLevels) -- we have to do this in case the player unlearns a profession | ||
|
|
||
| for index = 1, GetNumSkillLines() do | ||
| local skillName, isHeader, isExpanded, skillLevel = GetSkillLineInfo(index) | ||
| if skillName == TRADE_SKILLS and isHeader and not isExpanded then | ||
| ExpandSkillHeader(index) | ||
| return | ||
| else | ||
| local professionID = PROFESSION_IDS[skillName] | ||
| if professionID then | ||
| professionSkillLevels[professionID] = skillLevel | ||
| end | ||
| end | ||
| end | ||
| function addon:RequiredDisenchantingLevel(itemID) | ||
| -- TODO: this isn't working | ||
| local _, _, _, _, itemRequiredLevel = C_Item.GetItemInfo(itemID) | ||
| if not itemRequiredLevel then | ||
| return | ||
| end | ||
|
|
||
| function addon:RequiredDisenchantingLevel(itemID) | ||
| local _, _, _, _, itemRequiredLevel = C_Item.GetItemInfo(itemID) | ||
| if not itemRequiredLevel then | ||
| return | ||
| end | ||
|
|
||
| -- this is pretty much pure guesswork and probably won't be entirely accurate | ||
| if itemRequiredLevel <= 15 then | ||
| return 1 | ||
| elseif itemRequiredLevel <= 20 then | ||
| return 25 | ||
| elseif itemRequiredLevel <= 25 then | ||
| return 50 | ||
| elseif itemRequiredLevel <= 30 then | ||
| return 75 | ||
| elseif itemRequiredLevel <= 35 then | ||
| return 100 | ||
| elseif itemRequiredLevel <= 40 then | ||
| return 125 | ||
| elseif itemRequiredLevel <= 45 then | ||
| return 150 | ||
| elseif itemRequiredLevel <= 50 then | ||
| return 175 | ||
| elseif itemRequiredLevel <= 55 then | ||
| return 200 | ||
| elseif itemRequiredLevel <= 63 then | ||
| return 225 | ||
| elseif itemRequiredLevel <= 70 then | ||
| return 275 | ||
| elseif itemRequiredLevel <= 72 then | ||
| return 325 | ||
| elseif itemRequiredLevel <= 80 then | ||
| return 350 | ||
| else | ||
| return 9999 -- fallback to avoid errors without information available | ||
| end | ||
| -- this is pretty much pure guesswork and probably won't be entirely accurate | ||
| if itemRequiredLevel <= 15 then | ||
| return 1 | ||
| elseif itemRequiredLevel <= 20 then | ||
| return 25 | ||
| elseif itemRequiredLevel <= 25 then | ||
| return 50 | ||
| elseif itemRequiredLevel <= 30 then | ||
| return 75 | ||
| elseif itemRequiredLevel <= 35 then | ||
| return 100 | ||
| elseif itemRequiredLevel <= 40 then | ||
| return 125 | ||
| elseif itemRequiredLevel <= 45 then | ||
| return 150 | ||
| elseif itemRequiredLevel <= 50 then | ||
| return 175 | ||
| elseif itemRequiredLevel <= 55 then | ||
| return 200 | ||
| elseif itemRequiredLevel <= 63 then | ||
| return 225 | ||
| elseif itemRequiredLevel <= 70 then | ||
| return 275 | ||
| elseif itemRequiredLevel <= 72 then | ||
| return 325 | ||
| elseif itemRequiredLevel <= 80 then | ||
| return 350 | ||
| else | ||
| return 9999 -- fallback to avoid errors without information available | ||
| end | ||
| end |