Fixing an issue tracking debuffs by spellID with multiple raid-debuffs statuses (GH issue #289)

Added a clean database option in raiddebuffs configuration.

Author: michaelnpsp

RaidDebuffs/Grid2RaidDebuffs.lua

@@ -339,7 +339,7 @@ function class:LoadZoneSpells()
 		end
 		spells_count = spells_count + self.spells_count
 		if GSRD.debugging then
-			GSRD:Debug("Zone[%s] C_MapID[%d] EjID[%d] mapID[%d] Status [%s]: %d raid debuffs loaded from [%d]", instance_map_name, instance_bmap_id, instance_ej_id, instance_map_id, self.name, spells_count, (debuffs[instance_map_id] and instance_map_id) or (debuffs[instance_ej_id] and instance_ej_id) )
+			GSRD:Debug("Zone[%s] C_MapID[%d] EjID[%d] mapID[%d] Status [%s]: %d raid debuffs loaded from [%d]", instance_map_name, instance_bmap_id, instance_ej_id, instance_map_id, self.name, self.spells_count, (debuffs[instance_map_id] and instance_map_id) or (debuffs[instance_ej_id] and instance_ej_id) )
 		end
 	end
 end

RaidDebuffsOptions/OptionsAdvanced.lua

@@ -342,7 +342,8 @@ local function StatusDisableDebuff(spellId)
 	local status = debuffsStatuses[spellId]
 	if status then
 		local index = debuffsIndexes[spellId]
-		DbDelTableValue( spellId, status.dbx.debuffs, visibleInstance)
+		DbDelTableValue(spellId, status.dbx.debuffs, visibleInstance)
+		DbDelTableValue(-spellId, status.dbx.debuffs, visibleInstance)
 		debuffsStatuses[spellId] = nil
 		debuffsIndexes[spellId] = nil
 		for k,v in pairs(debuffsStatuses) do

RaidDebuffsOptions/OptionsGeneral.lua

@@ -279,6 +279,57 @@ do
 	} }
 end
 
+-- db maintenance
+do
+	local function IsZoneEnabled(zoneID) -- check if the zone belongs to any enabled module.
+		for module in pairs(RDO.db.profile.enabledModules) do
+			local zones = RDO.RDDB[module]
+			if zones and zones[zoneID] then
+				return true
+			end
+		end
+		return RDO.RDDB['[Custom Debuffs]'][zoneID] ~= nil
+	end
+
+	local function RemoveDisabledZones()
+		local delete, zcount, dcount = {}, 0, 0
+		for _,status in ipairs(RDO.statuses) do
+			local debuffs = status.dbx.debuffs
+			for zone in pairs(debuffs) do
+				if not IsZoneEnabled(zone) then
+					delete[zone] = debuffs
+					zcount = zcount + 1
+					dcount = dcount + #debuffs[zone]
+				end
+			end
+		end
+		if zcount>0 or dcount>0 then
+			Grid2Options:ConfirmDialog(string.format(L["Raid Debuffs Warning:\n %d zones and %d orphan raid debuffs has been detectetd. Do you want to remove this information ?"], zcount, dcount), function()
+				for zone, debuffs in pairs(delete) do
+					debuffs[zone] = nil
+				end
+				RDO:UpdateZoneSpells()
+				RDO:RefreshAdvancedOptions()
+				print("Grid2RaidDebuffs database maintenance finished.")
+			end)
+		else
+			Grid2Options:MessageDialog(L["Raid debuffs database is OK, nothing to clean."])
+		end
+	end
+
+	options.maintenance = { type = "group", order = 20,	name = L["Maintenance"], inline= true, args = {
+		clean = {
+			type = "execute",
+			order = 10,
+			name = L["Clean Database"],
+			desc = L["Clean the database removing orphan raid debuffs."],
+			func = function(info)
+				RemoveDisabledZones()
+			end,
+		}
+	} }
+end
+
 ----------------------------------------------------------------------
 --
 ----------------------------------------------------------------------