Clean up NetModulePacketHandler a little

TShockAPI/Handlers/NetModules/CreativeUnlocksHandler.cs

@@ -1,5 +1,6 @@
 using System.IO;
 using System.IO.Streams;
+using Terraria;
 using Terraria.GameContent.NetModules;
 using Terraria.Net;
 
@@ -45,6 +46,17 @@ namespace TShockAPI.Handlers.NetModules
 		/// <param name="rejectPacket"></param>
 		public void HandlePacket(TSPlayer player, out bool rejectPacket)
 		{
+			if (!Main.GameModeInfo.IsJourneyMode)
+			{
+				TShock.Log.ConsoleDebug(
+					"NetModuleHandler received attempt to unlock sacrifice while not in journey mode from",
+					player.Name
+				);
+
+				rejectPacket = true;
+				return;
+			}
+
 			if (UnknownField != 0)
 			{
 				TShock.Log.ConsoleDebug(

TShockAPI/Handlers/NetModules/NetModulePacketHandler.cs

@@ -1,4 +1,6 @@
-using Terraria;
+using System;
+using System.Collections.Generic;
+using Terraria;
 using static TShockAPI.GetDataHandlers;
 
 namespace TShockAPI.Handlers.NetModules
@@ -8,6 +10,19 @@ namespace TShockAPI.Handlers.NetModules
 	/// </summary>
 	public class NetModulePacketHandler : IPacketHandler<LoadNetModuleEventArgs>
 	{
+		/// <summary>
+		/// Maps net module types to handlers for the net module type. Add to or edit this dictionary to customise handling
+		/// </summary>
+		public static Dictionary<NetModulesTypes, Type> NetModulesToHandlersMap = new Dictionary<NetModulesTypes, Type>
+		{
+			{ NetModulesTypes.CreativePowers,               typeof(CreativePowerHandler)    },
+			{ NetModulesTypes.CreativeUnlocksPlayerReport,  typeof(CreativeUnlocksHandler)  },
+			{ NetModulesTypes.TeleportPylon,                typeof(PylonHandler)            },
+			{ NetModulesTypes.Liquid,                       typeof(LiquidHandler)           },
+			{ NetModulesTypes.Bestiary,                     typeof(BestiaryHandler)         },
+			{ NetModulesTypes.Ambience,                     typeof(AmbienceHandler)         }
+		};
+
 		/// <summary>
 		/// Invoked when a load net module packet is received. This method picks a <see cref="INetModuleHandler"/> based on the
 		/// net module type being loaded, then forwards the data to the chosen handler to process
@@ -18,61 +33,25 @@ namespace TShockAPI.Handlers.NetModules
 		{
 			INetModuleHandler handler;
 
-			switch (args.ModuleType)
+			if (NetModulesToHandlersMap.ContainsKey(args.ModuleType))
 			{
-				case NetModulesTypes.CreativePowers:
+				handler = (INetModuleHandler)Activator.CreateInstance(NetModulesToHandlersMap[args.ModuleType]);
-					{
+			}
-						handler = new CreativePowerHandler();
+			else
-						break;
+			{
-					}
+				// As of 1.4.x.x, this is now used for more things:
-
+				//	NetCreativePowersModule
-				case NetModulesTypes.CreativeUnlocksPlayerReport:
+				//	NetCreativePowerPermissionsModule
-					{
+				//	NetLiquidModule
-						if (!Main.GameModeInfo.IsJourneyMode)
+				//	NetParticlesModule
-						{
+				//	NetPingModule
-							TShock.Log.ConsoleDebug(
+				//	NetTeleportPylonModule
-								"NetModuleHandler received attempt to unlock sacrifice while not in journey mode from",
+				//	NetTextModule
-								args.Player.Name
+				// I (particles) have disabled the original return here, which means that we need to
-							);
+				// handle this more. In the interm, this unbreaks parts of vanilla. Originally
-
+				// we just blocked this because it was a liquid exploit.
-							args.Handled = true;
+				args.Handled = false;
-							return;
+				return;
-						}
-
-						handler = new CreativeUnlocksHandler();
-						break;
-					}
-				case NetModulesTypes.TeleportPylon:
-					{
-						handler = new PylonHandler();
-						break;
-					}
-				case NetModulesTypes.Liquid:
-					{
-						handler = new LiquidHandler();
-						break;
-					}
-				case NetModulesTypes.Bestiary:
-					{
-						handler = new BestiaryHandler();
-						break;
-					}
-				default:
-					{
-						// As of 1.4.x.x, this is now used for more things:
-						//	NetCreativePowersModule
-						//	NetCreativePowerPermissionsModule
-						//	NetLiquidModule
-						//	NetParticlesModule
-						//	NetPingModule
-						//	NetTeleportPylonModule
-						//	NetTextModule
-						// I (particles) have disabled the original return here, which means that we need to
-						// handle this more. In the interm, this unbreaks parts of vanilla. Originally
-						// we just blocked this because it was a liquid exploit.
-						args.Handled = false;
-						return;
-					}
 			}
 
 			handler.Deserialize(args.Data);

TShockAPI/TShockAPI.csproj

@@ -89,6 +89,7 @@
     <Compile Include="DB\TileManager.cs" />
     <Compile Include="Extensions\ExceptionExt.cs" />
     <Compile Include="Handlers\IPacketHandler.cs" />
+    <Compile Include="Handlers\NetModules\AmbienceHandler.cs" />
     <Compile Include="Handlers\NetModules\BestiaryHandler.cs" />
     <Compile Include="Handlers\NetModules\CreativePowerHandler.cs" />
     <Compile Include="Handlers\NetModules\CreativeUnlocksHandler.cs" />