From 28921f191de2269dd90b50ac99dd0d29ba90d0c4 Mon Sep 17 00:00:00 2001 From: jackpoz Date: Sat, 5 Sep 2020 12:46:09 +0200 Subject: Core/SAI: Do not allow SMART_EVENT_RESET to call SMART_ACTION_CALL_SCRIPT_RESET There might be cases where this should be allowed (i.e. if the SMART_TARGET_* is another object) but for now we'll just never allow it. --- src/server/game/AI/SmartScripts/SmartScriptMgr.cpp | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) (limited to 'src/server/game/AI/SmartScripts') diff --git a/src/server/game/AI/SmartScripts/SmartScriptMgr.cpp b/src/server/game/AI/SmartScripts/SmartScriptMgr.cpp index 6ee459d2ad1..eafb6253b9e 100644 --- a/src/server/game/AI/SmartScripts/SmartScriptMgr.cpp +++ b/src/server/game/AI/SmartScripts/SmartScriptMgr.cpp @@ -1018,6 +1018,14 @@ bool SmartAIMgr::IsEventValid(SmartScriptHolder& e) return false; } break; + case SMART_EVENT_RESET: + if (e.action.type == SMART_ACTION_CALL_SCRIPT_RESET) + { + // There might be SMART_TARGET_* cases where this should be allowed, they will be handled if needed + TC_LOG_ERROR("sql.sql", "SmartAIMgr: Entry %d SourceType %u Event %u uses event SMART_EVENT_RESET and action SMART_ACTION_CALL_SCRIPT_RESET, skipped.", e.entryOrGuid, e.GetScriptType(), e.event_id); + return false; + } + break; case SMART_EVENT_LINK: case SMART_EVENT_GO_LOOT_STATE_CHANGED: case SMART_EVENT_GO_EVENT_INFORM: @@ -1034,7 +1042,6 @@ bool SmartAIMgr::IsEventValid(SmartScriptHolder& e) case SMART_EVENT_DEATH: case SMART_EVENT_EVADE: case SMART_EVENT_REACHED_HOME: - case SMART_EVENT_RESET: case SMART_EVENT_QUEST_ACCEPTED: case SMART_EVENT_QUEST_OBJ_COPLETETION: case SMART_EVENT_QUEST_COMPLETION: -- cgit v1.2.3