Revert "small tweaks to scanning"

This reverts commit 820de13.

This doesn't work: we still need to scan target and mouseover in case we learn the GUID but don't change the target/mouseover.

The vb stuff was just unnecessary, the time between learning the GUID and setting the icon is realistically at worst 5 seconds long, more like 3. Further, if you can get that synced that implies someone else is running DBM and found it. They will have already set the icon and you no longer need to.

The target/mouseover mix-up is already fixed in another commit

DBM-Party-Vanilla/Classic/SOD_KarazhanCrypts/DarkRider.lua

@@ -59,7 +59,7 @@ end
 
 function mod:SPELL_AURA_REMOVED(args)
 	if args:IsSpell(1220939) then
-		self.vb.phaseTarget = args.destGUID
+		phaseTarget = args.destGUID
 		DBM:Debug("Found real horse GUID: " .. tostring(args.destGUID))
 		self:ScanLoop(150)
 	end
@@ -76,20 +76,20 @@ function mod:ScanMirrorTarget(uId)
 end
 
 function mod:ScanPhaseTarget(uId)
-	if not self.vb.phaseTarget or not UnitGUID(uId) then
+	if not phaseTarget or not UnitGUID(uId) then
 		return
 	end
-	if UnitGUID(uId) == self.vb.phaseTarget then
+	if UnitGUID(uId) == phaseTarget then
 		DBM:Debug("Found real horse uid: " .. tostring(uId))
 		if GetRaidTargetIndex(uId) ~= 8 then
 			SetRaidTarget(uId, 8) -- Everyone can (and should in this case) set icons
 		end
-		self.vb.phaseTarget = nil
+		phaseTarget = nil
 		self:UnscheduleMethod("ScanLoop")
 	end
 end
 
-local scanIds = {"party1target", "party2target", "party3target", "party4target"}
+local scanIds = {"target", "mouseover", "party1target", "party2target", "party3target", "party4target"}
 
 function mod:ScanLoop(maxCount)
 	maxCount = maxCount - 1