From 3e669a7b7a0f022fe0c5a78754793535b94e3703 Mon Sep 17 00:00:00 2001 From: Claude Date: Sun, 16 Nov 2025 18:20:51 +0000 Subject: [PATCH] Fix pfUI integration - use unit names and spell names, not GUIDs/IDs CRITICAL FIX: pfUI's libdebuff uses completely different data structures: - Uses unit NAME (not GUID) - Uses unit LEVEL as a key - Uses spell NAME (effect, not ID) - Storage: pflib.objects[unitName][unitLevel][effectName] Fixed SyncComboDurationToPfUI: - Convert GUID to unit name (check target, fallback to guidToName map) - Get unit level from target or default to 0 - Convert spell ID to spell name using SpellInfo() - Remove rank from spell name to match pfUI's format - Search both specific level and level 0 (pfUI's fallback) Fixed AddEffect hook: - Updated parameters to match pfUI signature: (unit, unitlevel, effect, duration, caster) - Check spell by name instead of ID - Map spell name back to ID to find tracking data This ensures pfUI's cooldown displays show correct combo durations. --- Compatibility/pfUI.lua | 107 ++++++++++++++++++++++++++++++++--------- 1 file changed, 84 insertions(+), 23 deletions(-) diff --git a/Compatibility/pfUI.lua b/Compatibility/pfUI.lua index 0aa8833..587e810 100644 --- a/Compatibility/pfUI.lua +++ b/Compatibility/pfUI.lua @@ -148,26 +148,37 @@ function Extension.HookPfUILibdebuff() if pflib.AddEffect and not Extension.pfLibAddEffectHooked then local originalAddEffect = pflib.AddEffect - pflib.AddEffect = function(self, guid, unitName, spellID, duration, stacks, caster) - -- For combo spells, override duration with our tracked value - if CleveRoids.IsComboScalingSpellID and CleveRoids.IsComboScalingSpellID(spellID) then - -- Check if we have tracking data for this spell - if CleveRoids.ComboPointTracking and CleveRoids.ComboPointTracking.byID and - CleveRoids.ComboPointTracking.byID[spellID] then - local tracking = CleveRoids.ComboPointTracking.byID[spellID] - if tracking.target_guid == guid and tracking.duration then - duration = tracking.duration - if CleveRoids.debug then - DEFAULT_CHAT_FRAME:AddMessage( - string.format("|cff00ff00[pfUI AddEffect Hook]|r Overriding duration to %ds for spell %d", - duration, spellID) - ) + pflib.AddEffect = function(self, unit, unitlevel, effect, duration, caster) + -- effect is a spell name, not ID + -- Check if this is a combo scaling spell by name + if CleveRoids.IsComboScalingSpell and CleveRoids.IsComboScalingSpell(effect) then + -- Try to get learned duration for this combo spell + local spellData = CleveRoids.GetComboScalingData(effect) + if spellData then + -- Find the spell ID for this effect + for spellID, data in pairs(CleveRoids.ComboScalingSpellsByID) do + if data.name == effect then + -- Check if we have tracking data + if CleveRoids.ComboPointTracking and CleveRoids.ComboPointTracking.byID and + CleveRoids.ComboPointTracking.byID[spellID] then + local tracking = CleveRoids.ComboPointTracking.byID[spellID] + if tracking.duration then + duration = tracking.duration + if CleveRoids.debug then + DEFAULT_CHAT_FRAME:AddMessage( + string.format("|cff00ff00[pfUI AddEffect Hook]|r Overriding %s duration to %ds", + effect, duration) + ) + end + break + end + end end end end end - return originalAddEffect(self, guid, unitName, spellID, duration, stacks, caster) + return originalAddEffect(self, unit, unitlevel, effect, duration, caster) end Extension.pfLibAddEffectHooked = true @@ -183,19 +194,69 @@ function Extension.SyncComboDurationToPfUI(guid, spellID, duration) return end + -- Get unit name from GUID + local unitName = nil + local unitLevel = 0 + + -- Check if this is the current target + local _, targetGUID = UnitExists("target") + if targetGUID == guid then + unitName = UnitName("target") + unitLevel = UnitLevel("target") or 0 + end + + -- If we couldn't find the unit, use GUID to name mapping from libdebuff + if not unitName and CleveRoids.libdebuff and CleveRoids.libdebuff.guidToName then + unitName = CleveRoids.libdebuff.guidToName[guid] + -- Default to level 0 if we don't have the unit targeted + unitLevel = 0 + end + + if not unitName then + if CleveRoids.debug then + DEFAULT_CHAT_FRAME:AddMessage("|cffff0000[pfUI Sync]|r Could not find unit name for GUID") + end + return + end + + -- Get spell name from spell ID + local spellName = SpellInfo(spellID) + if not spellName then + if CleveRoids.debug then + DEFAULT_CHAT_FRAME:AddMessage("|cffff0000[pfUI Sync]|r Could not find spell name for ID " .. spellID) + end + return + end + + -- Remove rank from spell name to match pfUI's format + local effectName = string.gsub(spellName, "%s*%(Rank %d+%)", "") + -- Update pfUI's stored debuff duration local pflib = pfUI.api.libdebuff - if pflib.objects and pflib.objects[guid] and pflib.objects[guid][spellID] then - local old_duration = pflib.objects[guid][spellID].duration - pflib.objects[guid][spellID].duration = duration + if pflib.objects and pflib.objects[unitName] then + -- Try both the specific level and level 0 (fallback) + for _, level in ipairs({unitLevel, 0}) do + if pflib.objects[unitName][level] and pflib.objects[unitName][level][effectName] then + local old_duration = pflib.objects[unitName][level][effectName].duration + pflib.objects[unitName][level][effectName].duration = duration - if CleveRoids.debug then - DEFAULT_CHAT_FRAME:AddMessage( - string.format("|cff00ffaa[pfUI Sync]|r Updated spell %d duration: %ds -> %ds", - spellID, old_duration or 0, duration) - ) + if CleveRoids.debug then + DEFAULT_CHAT_FRAME:AddMessage( + string.format("|cff00ffaa[pfUI Sync]|r Updated %s on %s (L%d): %ds -> %ds", + effectName, unitName, level, old_duration or 0, duration) + ) + end + return + end end end + + if CleveRoids.debug then + DEFAULT_CHAT_FRAME:AddMessage( + string.format("|cffaaaa00[pfUI Sync]|r Effect not found in pfUI storage: %s on %s", + effectName, unitName) + ) + end end -- Main compatibility check and setup