From 06b983bee245c2a9487cf36f5e899b1a595371ef Mon Sep 17 00:00:00 2001
From: Ovahlord <dreadkiller@gmx.de>
Date: Thu, 23 Apr 2020 12:52:04 +0200
Subject: [PATCH] Core/Spells: fixed Mobile Banking spawning a different chest
 for Alliance players

---
 src/server/scripts/Spells/spell_generic.cpp | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/server/scripts/Spells/spell_generic.cpp b/src/server/scripts/Spells/spell_generic.cpp
index 4e34c9a989e..0af9355f90e 100644
--- a/src/server/scripts/Spells/spell_generic.cpp
+++ b/src/server/scripts/Spells/spell_generic.cpp
@@ -5263,7 +5263,8 @@ class spell_gen_guild_battle_standard_buff : public SpellScript
 
 enum MobileBanking
 {
-    SPELL_GUILD_CHEST = 88306
+    SPELL_GUILD_CHEST_HORDE     = 88306,
+    SPELL_GUILD_CHEST_ALLIANCE  = 88304
 };
 
 class spell_gen_mobile_banking : public SpellScript
@@ -5277,7 +5278,11 @@ class spell_gen_mobile_banking : public SpellScript
 
     bool Validate(SpellInfo const* /*spellInfo*/) override
     {
-        return ValidateSpellInfo({ SPELL_GUILD_CHEST });
+        return ValidateSpellInfo(
+            {
+                SPELL_GUILD_CHEST_HORDE,
+                SPELL_GUILD_CHEST_ALLIANCE
+            });
     }
 
     SpellCastResult CheckRequirement()
@@ -5291,7 +5296,8 @@ class spell_gen_mobile_banking : public SpellScript
 
     void HandleScriptEffect(SpellEffIndex /*effIndex*/)
     {
-        GetHitUnit()->CastSpell(GetHitUnit(), SPELL_GUILD_CHEST);
+        Unit* target = GetHitUnit();
+        target->CastSpell(target, target->ToPlayer()->GetTeamId() == TEAM_HORDE ? SPELL_GUILD_CHEST_HORDE : SPELL_GUILD_CHEST_ALLIANCE);
     }
 
     void Register() override