From d4f285479672dcf5c7b833b704967fa6d266f2bc Mon Sep 17 00:00:00 2001
From: Ovahlord <dreadkiller@gmx.de>
Date: Sat, 11 May 2019 06:47:01 +0200
Subject: [PATCH] Core/Spells: fixed an issue in Pyromaniac's spell script that
 was allowing players to trigger the buff when having three dots instead of
 dots on three targets active

---
 src/server/scripts/Spells/spell_mage.cpp | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/src/server/scripts/Spells/spell_mage.cpp b/src/server/scripts/Spells/spell_mage.cpp
index eaecef7d088..d1cb37fef06 100644
--- a/src/server/scripts/Spells/spell_mage.cpp
+++ b/src/server/scripts/Spells/spell_mage.cpp
@@ -2107,8 +2107,13 @@ class spell_mage_pyromaniac : public AuraScript
             {
                 std::list<AuraEffect*> _dotAuraEffects = dotTarget->GetAuraEffectsByType(SPELL_AURA_PERIODIC_DAMAGE);
                 for (AuraEffect const* dotEff : _dotAuraEffects)
+                {
                     if (dotEff->GetCasterGUID() == target->GetGUID() && dotEff->GetSpellInfo()->SpellFamilyFlags[2] & 0x00000008)
+                    {
                         dotTargetCount++;
+                        break;
+                    }
+                }
             }
             else
                 _dotTargetGuids.erase(guid);
@@ -2137,8 +2142,13 @@ class spell_mage_pyromaniac : public AuraScript
                 uint8 dotTargetCount = 0;
                 std::list<AuraEffect*> _dotAuraEffects = dotTarget->GetAuraEffectsByType(SPELL_AURA_PERIODIC_DAMAGE);
                 for (AuraEffect const* dotEff : _dotAuraEffects)
+                {
                     if (dotEff->GetCasterGUID() == target->GetGUID() && dotEff->GetSpellInfo()->SpellFamilyFlags[2] & 0x00000008)
+                    {
                         dotTargetCount++;
+                        break;
+                    }
+                }
 
                 if (!dotTargetCount)
                     _dotTargetGuids.erase(guid);