From 66f4492fa755e59a20b64e5ec98c225aab9525de Mon Sep 17 00:00:00 2001 From: Parnic Date: Sat, 27 Jul 2024 15:42:31 -0500 Subject: [PATCH] Fix spell cooldown bars in 11.x GetSpellCooldown used to return 1 for "has duration", but the new native version returns true. Fun. Additionally, it looks like the events triggering UpdateCustomBarEvent() are passing a nil argument that the mod thought was supposed to be the affected unit. So I also fixed that, though I don't know if that's an 11.x thing or not. https://www.wowace.com/projects/ice-hud/issues/363 --- changelog.md | 4 ++++ modules/CustomCDBar.lua | 6 +++--- this_version.md | 4 ++++ 3 files changed, 11 insertions(+), 3 deletions(-) diff --git a/changelog.md b/changelog.md index 0512ff1..0cc4181 100644 --- a/changelog.md +++ b/changelog.md @@ -1,5 +1,9 @@ # Changelog +v1.15.3: + +- Fix custom cooldown bars not working in 11.x (wowace issue #363) + v1.15.2: - Re-enable right-click context menus on 11.x. diff --git a/modules/CustomCDBar.lua b/modules/CustomCDBar.lua index 7f9412b..9191b20 100644 --- a/modules/CustomCDBar.lua +++ b/modules/CustomCDBar.lua @@ -637,9 +637,9 @@ function IceCustomCDBar.prototype:GetCooldownDuration(buffName) local localRemaining = nil local localStart, localDuration, hasCooldown = GetSpellCooldown(buffName) - if (hasCooldown == 1) then + if hasCooldown then -- the item has a potential cooldown - if (localDuration <= 1.5) then + if localDuration <= 1.5 then return nil, nil end @@ -734,7 +734,7 @@ function IceCustomCDBar.prototype:UpdateItemUnitInventoryChanged(event, unit) end function IceCustomCDBar.prototype:UpdateCustomBarEvent(event, unit) - if unit ~= self.unit then + if unit and unit ~= self.unit then return end if not self.moduleSettings.cooldownType or self.moduleSettings.cooldownType == COOLDOWN_TYPE_SPELL then diff --git a/this_version.md b/this_version.md index f148855..baea79d 100644 --- a/this_version.md +++ b/this_version.md @@ -1,5 +1,9 @@ # Changelog +v1.15.3: + +- Fix custom cooldown bars not working in 11.x (wowace issue #363) + v1.15.2: - Re-enable right-click context menus on 11.x.