Classic: Added an option to update threat status more frequent (ticke…

…t #1033)

Options/modules/statuses/StatusThreat.lua

@@ -15,6 +15,22 @@ Grid2Options:RegisterStatusOptions("threat", "combat", function(self, status, op
 			status:UpdateDB()
 		end,
 	}
+	if Grid2.isClassic then
+		options.frequentUpdates = {
+			type = "toggle",
+			name = L["Frequent updates"],
+			desc = L["Update threat status more frequent."],
+			width = "full",
+			order = 34,
+			get = function () return status.dbx.frequentUpdates end,
+			set = function (_, v)
+				local enabled = status.enabled
+				if enabled then status:OnDisable() end
+				status.dbx.frequentUpdates = v or nil
+				if enabled then status:OnEnable() end
+			end,
+		}
+	end
 end, {
 		color1 = L["Not Tanking"],
 		colorDesc1 = L["Higher threat than tank."],

modules/StatusThreat.lua

@@ -4,11 +4,13 @@ local Grid2 = Grid2
 local UnitExists = UnitExists
 local UnitThreatSituation = UnitThreatSituation
 
+local unit_is_valid = Grid2.roster_guids
+
 local colors
 local activeValue
 
 function Threat:UpdateUnit(_, unit)
-	if unit then -- unit can be nil which is so wtf
+	if unit_is_valid[unit or 0] then -- unit can be nil which is so wtf
 		self:UpdateIndicators(unit)
 	end
 end
@@ -17,11 +19,18 @@ function Threat:OnEnable()
 	self:UpdateDB()
 	self:RegisterEvent("UNIT_THREAT_SITUATION_UPDATE", "UpdateUnit")
 	self:RegisterEvent("ZONE_CHANGED_NEW_AREA", "UpdateAllUnits")
+	if Grid2.isClassic and self.dbx.frequentUpdates then
+		self:RegisterEvent("UNIT_HEALTH_FREQUENT", "UpdateUnit")
+	end
+
 end
 
 function Threat:OnDisable()
 	self:UnregisterEvent("UNIT_THREAT_SITUATION_UPDATE")
 	self:UnregisterEvent("ZONE_CHANGED_NEW_AREA")
+	if Grid2.isClassic and self.dbx.frequentUpdates then
+		self:UnregisterEvent("UNIT_HEALTH_FREQUENT")
+	end
 end
 
 function Threat:UpdateDB()