mifan

This doesn't works for me. Still bugged

In combat log when i click on once at Divine Storm spell (3 targets):

Divine Storm for 111 physical

Seal of Command for 92 holy (crit)

Divine Storm for 111 physical

Seal of Command for 46 holy

Divine Storm for 112 physical

Seal of Command for 47 holy

Should be:

Divine Storm for 111 physical

Seal of Command for 92 holy (crit)

Divine Storm for 111 physical

Divine Storm for 112 physical

the upper one of the two logs is correct, i tested and verified on official servers (3.2.2a).

Divine Storm should proc seal exactly once on EVERY target it hits.

so there should be no problem with my patch.

2) do not allow to attack other 2 targets for Seal of Command when using Divine Storm

first of all, thank you for your incredible works!

ok, then, i find a way to solve this problem you addressed.

here goes my patch:

--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -2573,6 +2573,14 @@ void Spell::cast(bool skipCheck)

    FillTargetMap();

+    // let not attack nearby enemies when Seal of Command is triggered by Divine Storm
+    if (m_spellInfo->Id == 20424) {
+        if (m_caster->FindCurrentSpellBySpellId(53385)) {
+            for (int numTargets = m_UniqueTargetInfo.size(); numTargets > 1; numTargets--)
+                m_UniqueTargetInfo.pop_back();
+        }
+    }
+
    if(m_spellState == SPELL_STATE_FINISHED)                // stop cast if spell marked as finish somewhere in FillTargetMap
    {
        SetExecutedCurrently(false);