Pillar towers sync their defeation upon death

Vanilla does update their downed states in the WorldData sync. However, it does not trigger when the Pillars are killed, which is what we want, but are instead updated when all are defeated.

Author: SheepishShepherd

NPCAssist.cs

@@ -94,34 +94,22 @@ public override void OnKill(NPC npc) {
 		/// <summary>
 		/// Handles all of BossChecklist's custom downed variables, makring them as defeated and updating all clients when needed.
 		/// </summary>
-		public void HandleDownedNPCs(int npcType) {
+		/// <returns>If the corresponding flag was flipped.</returns>
+		internal static bool HandleDownedNPCs(int npcType) {
 			if (!WorldAssist.TrackingDowns)
-				return;
+				return false;
 
-			if ((npcType == NPCID.DD2DarkMageT1 || npcType == NPCID.DD2DarkMageT3) && !WorldAssist.downedDarkMage) {
-				WorldAssist.downedDarkMage = true;
-				if (Main.netMode == NetmodeID.Server) {
-					NetMessage.SendData(MessageID.WorldData);
-				}
-			}
-			else if ((npcType == NPCID.DD2OgreT2 || npcType == NPCID.DD2OgreT3) && !WorldAssist.downedOgre) {
-				WorldAssist.downedOgre = true;
-				if (Main.netMode == NetmodeID.Server) {
-					NetMessage.SendData(MessageID.WorldData);
-				}
-			}
-			else if (npcType == NPCID.PirateShip && !WorldAssist.downedFlyingDutchman) {
-				WorldAssist.downedFlyingDutchman = true;
-				if (Main.netMode == NetmodeID.Server) {
-					NetMessage.SendData(MessageID.WorldData);
-				}
-			}
-			else if (npcType == NPCID.MartianSaucerCore && !WorldAssist.downedMartianSaucer) {
-				WorldAssist.downedMartianSaucer = true;
-				if (Main.netMode == NetmodeID.Server) {
-					NetMessage.SendData(MessageID.WorldData);
-				}
-			}
+			return npcType switch {
+				NPCID.DD2DarkMageT1 | NPCID.DD2DarkMageT1 => Networking.DownedEntryCheck(ref WorldAssist.downedDarkMage),
+				NPCID.DD2OgreT2 | NPCID.DD2OgreT3 => Networking.DownedEntryCheck(ref WorldAssist.downedOgre),
+				NPCID.PirateShip => Networking.DownedEntryCheck(ref WorldAssist.downedFlyingDutchman),
+				NPCID.MartianSaucerCore => Networking.DownedEntryCheck(ref WorldAssist.downedMartianSaucer),
+				NPCID.LunarTowerVortex => Networking.DownedEntryCheck(ref NPC.downedTowerVortex),
+				NPCID.LunarTowerStardust => Networking.DownedEntryCheck(ref NPC.downedTowerStardust),
+				NPCID.LunarTowerNebula => Networking.DownedEntryCheck(ref NPC.downedTowerNebula),
+				NPCID.LunarTowerSolar => Networking.DownedEntryCheck(ref NPC.downedTowerSolar),
+				_ => false
+			};
 		}
 	}
 }

Networking.cs

@@ -31,6 +31,16 @@ internal class Networking {
 		public static bool NewWorldRecord(NetRecordID netRecord) => netRecord.HasFlag(NetRecordID.WorldRecord_Duration) || netRecord.HasFlag(NetRecordID.WorldRecord_HitsTaken);
 		public static bool ResettingRecords(NetRecordID netRecord) => netRecord.HasFlag(NetRecordID.PersonalBest_Reset) || netRecord.HasFlag(NetRecordID.FirstVictory_Reset);
 
+		public static bool DownedEntryCheck(ref bool downed) {
+			bool temp = downed;
+			downed = true;
+			if (temp == false && Main.netMode == NetmodeID.Server) {
+				NetMessage.SendData(MessageID.WorldData);
+				return true;
+			}
+			return false;
+		}
+
 		/// <summary>
 		/// Send a packet to the server to add, remove, or clear entries from the hidden entries list.
 		/// </summary>

WorldAssist.cs

@@ -185,14 +185,11 @@ public override void NetSend(BinaryWriter writer) {
 			};
 			writer.Write(flags);
 
-			//8 flags added, do not add more, move onto next BitsByte
+			/* 8 flags added, do not add more, move onto next BitsByte
 			flags = new BitsByte {
-				[0] = NPC.downedTowerSolar,
-				[1] = NPC.downedTowerVortex,
-				[2] = NPC.downedTowerNebula,
-				[3] = NPC.downedTowerStardust,
+
 			};
-			writer.Write(flags);
+			writer.Write(flags); */
 
 			writer.Write(HiddenEntries.Count);
 			foreach (var bossKey in HiddenEntries) {
@@ -216,11 +213,7 @@ public override void NetReceive(BinaryReader reader) {
 			downedFlyingDutchman = flags[6];
 			downedMartianSaucer = flags[7];
 
-			flags = reader.ReadByte();
-			NPC.downedTowerSolar = flags[0];
-			NPC.downedTowerVortex = flags[1];
-			NPC.downedTowerNebula = flags[2];
-			NPC.downedTowerStardust = flags[3];
+			//flags = reader.ReadByte();
 
 			HiddenEntries.Clear();
 			int count = reader.ReadInt32();