Eliminate duplicate rooms

ProjectLighthouse.Servers.Website/Controllers/Debug/RoomVisualizerController.cs

@@ -46,4 +46,17 @@ public class RoomVisualizerController : ControllerBase
         return this.Redirect("/debug/roomVisualizer");
         #endif
     }
+
+    [HttpGet("createRoomsWithDuplicatePlayers")]
+    public async Task<IActionResult> CreateRoomsWithDuplicatePlayers()
+    {
+        #if !DEBUG
+        return this.NotFound();
+        #else
+        List<int> users = await this.database.Users.OrderByDescending(_ => EF.Functions.Random()).Take(1).Select(u => u.UserId).ToListAsync();
+        RoomHelper.CreateRoom(users, GameVersion.LittleBigPlanet2, Platform.PS3);
+        RoomHelper.CreateRoom(users, GameVersion.LittleBigPlanet2, Platform.PS3);
+        return this.Redirect("/debug/roomVisualizer");
+        #endif
+    }
 }

ProjectLighthouse.Servers.Website/Pages/Debug/RoomVisualizerPage.cshtml

@@ -44,6 +44,10 @@
     <div class="ui blue button">Create Fake Room</div>
 </a>
 
+<a href="/debug/roomVisualizer/createRoomsWithDuplicatePlayers">
+    <div class="ui blue button">Create Rooms With Duplicate Players</div>
+</a>
+
 <a href="/debug/roomVisualizer/deleteRooms">
     <div class="ui red button">Nuke all rooms</div>
 </a>

ProjectLighthouse/Match/Rooms/RoomHelper.cs

@@ -225,7 +225,7 @@ public class RoomHelper
             {
                 try
                 {
-                    rooms.RemoveAll(r => r.HostId == hostId);
+                    rooms.RemoveAll(r => r.PlayerIds.Contains((int)hostId));
                 }
                 catch
                 {
@@ -233,15 +233,18 @@ public class RoomHelper
                 }
             }
 
-            // Remove players in this new room from other rooms
+            // Remove rooms containing players in this new room
             if (newRoom != null)
-                foreach (Room room in rooms)
+            {
+                foreach (Room room in rooms.Where(room => room != newRoom))
                 {
-                    if (room == newRoom) continue;
+                    foreach (int newRoomPlayer in newRoom.PlayerIds)
-
+                    {
-                    foreach (int newRoomPlayer in newRoom.PlayerIds) room.PlayerIds.RemoveAll(p => p == newRoomPlayer);
+                        if (room.PlayerIds.Contains(newRoomPlayer)) 
-                    roomsToUpdate.Add(room);
+                            rooms.Remove(room);
+                    }
                 }
+            }
 
             foreach (Room room in roomsToUpdate)
             {