Add workaround for UNIT_HEALTH firing with nil unit (fixes CF-859)

phanx-wow · Oct 26, 2016 · 132c9ed · 132c9ed
1 parent b46f729
commit 132c9ed
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,7 @@
+### Version 7.1.0.0
+
+- Added a workaround for a Blizzard bug causing errors in the Health status module ([ticket #859](https://wow.curseforge.com/addons/grid/tickets/859))
+
 ### Version 7.0.3.3
 
 - Really fixed the frames sometimes appearing behind the backdrop ([ticket #840](https://wow.curseforge.com/addons/grid/tickets/840))

diff --git a/Grid.toc b/Grid.toc
@@ -1,4 +1,4 @@
-## Interface: 70000
+## Interface: 70100
 ## Version: @project-version@
 
 ## Title: Grid

diff --git a/Statuses/Health.lua b/Statuses/Health.lua
@@ -193,6 +193,12 @@ end
 local UnitGUID, UnitHealth, UnitHealthMax, UnitIsConnected, UnitIsDeadOrGhost, UnitIsFeignDeath = UnitGUID, UnitHealth, UnitHealthMax, UnitIsConnected, UnitIsDeadOrGhost, UnitIsFeignDeath
 
 function GridStatusHealth:UpdateUnit(event, unitid, ignoreRange)
+	if not unitid then
+		-- 7.1: UNIT_HEALTH and UNIT_MAXHEALTH sometimes fire with no unit token
+		-- https://wow.curseforge.com/addons/grid/tickets/859
+		return
+	end
+
 	local guid = UnitGUID(unitid)
 
 	if not GridRoster:IsGUIDInRaid(guid) then