fix(dispels): add Curse to Witch Doctor dispel set (Hexbreak 806240)

WITCHDOCTOR could only flag Disease/Poison (Cleansing Idol). Hexbreak (806240) is a single-target Curse remover, so add Curse=true. highlight.lua reads this via GetCoaDispels, so the dispellable-debuff highlight picks it up.
fix(libs): pcall AceGUI OnGamePadButtonDown (3.3.5 has no gamepad script type)
2026-05-31 14:45:28 +02:00 · 2026-05-29 20:23:33 +02:00 · 2026-05-27 22:55:04 +02:00
3 changed files with 13 additions and 13 deletions
@@ -199,7 +199,7 @@ local function Constructor()
 	button:SetScript("OnKeyDown", Keybinding_OnKeyDown)
 	button:SetScript("OnMouseDown", Keybinding_OnMouseDown)
 	button:SetScript("OnMouseWheel", Keybinding_OnMouseWheel)
-	button:SetScript("OnGamePadButtonDown", Keybinding_OnKeyDown)
+	pcall(button.SetScript, button, "OnGamePadButtonDown", Keybinding_OnKeyDown)
 	button:SetPoint("BOTTOMLEFT")
 	button:SetPoint("BOTTOMRIGHT")
 	button:SetHeight(24)
@@ -2,7 +2,7 @@
 ## Title: Shadowed Unit Frames
 ## Notes: An apple a day keeps the raptor away, or so they say
 ## Author: Shadowed
-## Version: v3.3.0
+## Version: v3.3.0-coa.2
 ## SavedVariables: ShadowedUFDB
 ## OptionalDeps: Ace3, LibSharedMedia-3.0, LibHealComm-4.0, AceGUI-3.0-SharedMediaWidgets
 ## X-Curse-Packaged-Version: v3.2.12
@@ -21,23 +21,23 @@ local COA_CLASS_DISPELS = {
 	["WITCHHUNTER"]  = { Curse = true },
 	["SUNCLERIC"]    = { Magic = true, Disease = true, Poison = true },                     -- Sanctify
 	["WILDWALKER"]   = { Disease = true, Poison = true },                                   -- Primalist (Soothing Touch — DBC says Magic, gameplay is Poison/Disease)
-	["WITCHDOCTOR"]  = { Disease = true, Poison = true },                                   -- Cleansing Idol (AoE)
+	["WITCHDOCTOR"]  = { Curse = true, Disease = true, Poison = true },                     -- Hexbreak (806240, single-target Curse) + Cleansing Idol (504840, AoE Disease/Poison)
 	["TINKER"]       = { Disease = true, Poison = true },                                   -- Nanobot Cleanser
 }
 local function getCoaDispels()
 	if playerCoaDispels ~= nil then return playerCoaDispels end
-	local cp = _G.C_Player
-	if not cp or not cp.IsCustomClass then
-		-- C_Player not yet initialised (called before PLAYER_LOGIN); don't
-		-- cache — retry on the next scan so we pick it up once in-world.
+	-- Trust the COA_CLASS_DISPELS table directly: it only contains custom-class
+	-- tokens, so any token-match implies the player IS a dispelling custom
+	-- class. Avoids C_Player.IsCustomClass timing/availability issues (the
+	-- API isn't reliable for every class on every login — Witchdoctor in
+	-- particular was getting cached as false on raid frames).
+	local _, token = UnitClass("player")
+	if not token or token == "" then
+		-- UnitClass not ready yet (very early init); don't cache — retry on
+		-- the next scan so we pick it up once in-world.
 		return nil
 	end
-	if cp:IsCustomClass() then
-		local _, token = UnitClass("player")
-		playerCoaDispels = token and COA_CLASS_DISPELS[token] or false
-	else
-		playerCoaDispels = false
-	end
+	playerCoaDispels = COA_CLASS_DISPELS[token] or false
 	return playerCoaDispels
 end
 -- Expose for other modules (highlight.lua) so the dispel set stays single-source.