Quantcast

Don't emit for warrior Execute if not on the main target.

Johnny C. Lam [12-11-14 - 20:03]
Don't emit for warrior Execute if not on the main target.

Ovale does not really do multi-target spell suggestions in a single icon.
Filename
SimulationCraft.lua
diff --git a/SimulationCraft.lua b/SimulationCraft.lua
index 60a38d5..e563186 100644
--- a/SimulationCraft.lua
+++ b/SimulationCraft.lua
@@ -1366,6 +1366,15 @@ EmitAction = function(parseNode, nodeList, annotation)
 			conditionCode = "pet.Present() and pet.CreatureFamily(Wrathguard)"
 		elseif class == "WARRIOR" and action == "charge" then
 			conditionCode = "target.InRange(charge)"
+		elseif class == "WARRIOR" and strsub(action, 1, 7) == "execute" then
+			if modifier.target then
+				local target = Unparse(modifier.target)
+				local target = tonumber(target)
+				if target then
+					-- Skip "execute" actions if they are not on the main target.
+					isSpellAction = false
+				end
+			end
 		elseif class == "WARRIOR" and action == "heroic_leap" then
 			-- Use Charge as a range-finder for Heroic Leap.
 			local spellName = "charge"