From 240d5c2a66b7f4d482ec7164192f177624789b85 Mon Sep 17 00:00:00 2001
From: Akkadius <akkadius1@gmail.com>
Date: Wed, 11 Mar 2020 01:39:08 -0500
Subject: [PATCH] Add LOS check with NPC::AICheckCloseBeneficialSpells [skip
 ci]

---
 zone/npc.cpp | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/zone/npc.cpp b/zone/npc.cpp
index 6df237d5d..0cc2580fe 100644
--- a/zone/npc.cpp
+++ b/zone/npc.cpp
@@ -3093,6 +3093,14 @@ bool NPC::AICheckCloseBeneficialSpells(
 			continue;
 		}
 
+		if (!mob->CheckLosFN(caster)) {
+			continue;
+		}
+
+		if (mob->GetReverseFactionCon(caster) >= FACTION_KINDLY) {
+			continue;
+		}
+
 		LogAICastBeneficialClose(
 			"NPC [{}] Distance [{}] Cast Range [{}] Caster [{}]",
 			mob->GetCleanName(),
@@ -3101,10 +3109,6 @@ bool NPC::AICheckCloseBeneficialSpells(
 			caster->GetCleanName()
 		);
 
-		if (mob->GetReverseFactionCon(caster) >= FACTION_KINDLY) {
-			continue;
-		}
-
 		if ((spell_types & SpellType_Buff) && !RuleB(NPC, BuffFriends)) {
 			if (mob != caster) {
 				spell_types = SpellType_Heal;