Core/Instances: Delete InstanceSaveMgr and replace most of its uses with new InstanceLockMgr

1 files changed, 6 insertions, 2 deletions

diff --git a/src/server/game/Scenarios/InstanceScenario.cpp b/src/server/game/Scenarios/InstanceScenario.cpp
index 1dbb6c08359..30336a6b8f9 100644
--- a/src/server/game/Scenarios/InstanceScenario.cpp
+++ b/src/server/game/Scenarios/InstanceScenario.cpp
@@ -19,11 +19,13 @@
 #include "DatabaseEnv.h"
 #include "DB2Stores.h"
 #include "GameTime.h"
-#include "InstanceSaveMgr.h"
 #include "Log.h"
 #include "Map.h"
 #include "Player.h"
 
+// TODO
+// Do not save to db except for scenario type 3 (SCENARIO_TYPE_USE_DUNGEON_DISPLAY)
+
 InstanceScenario::InstanceScenario(Map const* map, ScenarioData const* scenarioData) : Scenario(scenarioData), _map(map)
 {
     ASSERT(_map);
@@ -60,8 +62,9 @@ void InstanceScenario::SaveToDB()
         Criteria const* criteria = sCriteriaMgr->GetCriteria(iter->first);
         switch (CriteriaType(criteria->Entry->Type))
         {
-            // Blizzard only appears to store creature kills
+            // Blizzard only appears to store creature kills and dungeon encounters
             case CriteriaType::KillCreature:
+            case CriteriaType::DefeatDungeonEncounter:
                 break;
             default:
                 continue;
@@ -127,6 +130,7 @@ void InstanceScenario::LoadInstanceData(uint32 instanceId)
             {
                 // Blizzard appears to only stores creatures killed progress for unknown reasons. Either technical shortcoming or intentional
                 case CriteriaType::KillCreature:
+                case CriteriaType::DefeatDungeonEncounter:
                     break;
                 default:
                     continue;