From af3445b40a08c067bc41d1eee7d080a8bda57f55 Mon Sep 17 00:00:00 2001
From: hondacrx <prelude_20032002@yahoo.com>
Date: Thu, 14 Oct 2021 19:37:44 -0400
Subject: [PATCH] Core/Spells: Shadow of Death & Spiritual Vengeance should be
 negative spells (BlackTemple/Teron Gorefiend)

Port From (https://github.com/TrinityCore/TrinityCore/commit/af26380d9b1fc86f9b70e744bacdb947770d47c9)
---
 Source/Game/Spells/SpellInfo.cs | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/Source/Game/Spells/SpellInfo.cs b/Source/Game/Spells/SpellInfo.cs
index 0846eff4b..fb3991550 100644
--- a/Source/Game/Spells/SpellInfo.cs
+++ b/Source/Game/Spells/SpellInfo.cs
@@ -3275,6 +3275,7 @@ namespace Game.Spells
                 case SpellFamilyNames.Generic:
                     switch (spellInfo.Id)
                     {
+                        case 40268: // Spiritual Vengeance, Teron Gorefiend, Black Temple
                         case 61987: // Avenging Wrath Marker
                         case 61988: // Divine Shield exclude aura
                         case 72410: // Rune of Blood, Saurfang, Icecrown Citadel
@@ -3292,21 +3293,23 @@ namespace Game.Spells
                             break;
                     }
                     break;
-                case SpellFamilyNames.Warrior:
-                    // Slam, Execute
-                    if ((spellInfo.SpellFamilyFlags[0] & 0x20200000) != 0)
-                        return false;
-                    break;
                 case SpellFamilyNames.Rogue:
                     switch (spellInfo.Id)
                     {
                         // Envenom must be considered as a positive effect even though it deals damage
                         case 32645: // Envenom
                             return true;
+                        case 40251: // Shadow of Death, Teron Gorefiend, Black Temple
+                            return false;
                         default:
                             break;
                     }
                     break;
+                case SpellFamilyNames.Warrior:
+                    // Slam, Execute
+                    if ((spellInfo.SpellFamilyFlags[0] & 0x20200000) != 0)
+                        return false;
+                    break;
                 default:
                     break;
             }