Minor cleanup and logic update to delete map events when maps are removed from the db (#1849)

Author: jcsnider

Intersect.Server/Database/DbInterface.cs

@@ -1049,6 +1049,15 @@ public static void DeleteGameObject(IDatabaseObject gameObject)
 
                             break;
                         case GameObjectType.Map:
+                            //Delete all map events first
+                            foreach (var evtId in ((MapController)gameObject).EventIds)
+                            {
+                                var evt = EventBase.Get(evtId);
+                                if (evt != null)
+                                {
+                                    DeleteGameObject(evt);
+                                }
+                            }
                             context.Maps.Remove((MapController)gameObject);
                             MapController.Lookup.Delete(gameObject);
 

Intersect.Server/Networking/PacketHandler.cs

@@ -3349,11 +3349,12 @@ public void HandlePacket(Client client, Network.Packets.Editor.MapListUpdatePack
 
                         lock (ServerContext.Instance.LogicService.LogicLock)
                         {
+                            var map = MapController.Get(mapId);
                             ServerContext.Instance.LogicService.LogicPool.WaitForIdle();
                             mapId = packet.TargetId;
-                            var players = MapController.Get(mapId).GetPlayersOnAllInstances();
+                            var players = map.GetPlayersOnAllInstances();
                             MapList.List.DeleteMap(mapId);
-                            DbInterface.DeleteGameObject(MapController.Get(mapId));
+                            DbInterface.DeleteGameObject(map);
                             DbInterface.GenerateMapGrids();
                             PacketSender.SendMapListToAll();
                             foreach (var plyr in players)