Re-sync after coa-ace3 9583952 backported WoWUIDev/Ace3 PR #10 which fixes
the AceDB-3.0 simple-value defaults metatable: previously falsy defaults
like ["*"] = false read back as nil because of `k2~=nil and v or nil`
short-circuiting. Now they round-trip correctly.
ProfessionMenu.lua:
- profList: add Name="Woodcutting" (spell 13977860) and Name="Woodworking"
(spells 1005008-1005011) so getProfessionRanks can find the matching
GetSkillLineInfo entry. Without Name, both fell through GetSpellInfo+match
with no result and returned nil,nil.
- getProfessionRanks: replace compName:match(name) (regex) with compName == name
(plain compare); GetSkillLineInfo names are exact, regex was a bug magnet.
- Guard rank concat at lines ~394/397/400: skip the " (rank)" / " (max)" /
" (rank/max)" append when getProfessionRanks returns nil so unknown
skill lines don't error.
- InterfaceOptionsFrame:HookScript: wrap in 'if InterfaceOptionsFrame then';
CoA's reworked FrameXML has no InterfaceOptionsFrame.
ProfessionMenuOptions.lua:
- Guard InterfaceOptionsFrame / InterfaceOptionsFrame_OpenToCategory /
InterfaceOptions_AddCategory call sites (Options_Toggle, ProfessionMenu_
OpenOptions, CreateOptionsUI). All nil on CoA's FrameXML.
- TxtSize dropdown func: drop deprecated WoW-2.x global 'this'; use
UIDropDownMenu_GetSelectedID(ProfessionMenuOptions_TxtSizeMenu) instead.
Woodworking is a new CoA crafting profession sitting alongside the
existing Woodcutting gathering line. It progresses through the
standard Apprentice/Journeyman/Expert/Artisan tier spells (no Master
or Grand Master yet), per db.exil.es:
1005008 Apprentice Woodworking ( 75)
1005009 Journeyman Woodworking (150)
1005010 Expert Woodworking (225)
1005011 Artisan Woodworking (300)
GetSpellInfo on each returns the rank-prefixed name; the addon's
getProfessionRanks() then matches against the 'Woodworking' skill
line, same shape as Apprentice/Alchemy etc. No transmute cooldowns
known yet, so no profCooldowns entry.
- updated to delete sanguine workbench after dropping with the item delete setting
- added open menu on mouse over of the button
- Woodcutting now works
- added variable text size for the menus
- added disenchanting and prospecting to the menu
- changed the text on the option to hide the main button to standalone button
florian.berthold
andrew6180
Anch