Stop mapping command from unsetting it's own settings if used repeatedly (#7875)

Content.Client/Commands/MappingClientSideSetupCommand.cs

@@ -0,0 +1,34 @@
+using JetBrains.Annotations;
+using System;
+using Content.Client.Markers;
+using Robust.Client.Graphics;
+using Robust.Shared.Console;
+using Robust.Shared.GameObjects;
+
+namespace Content.Client.Commands;
+
+/// <summary>
+/// Sent by mapping command to client.
+/// This is because the debug commands for some of these options are on toggles.
+/// </summary>
+[UsedImplicitly]
+internal sealed class MappingClientSideSetupCommand : IConsoleCommand
+{
+    // ReSharper disable once StringLiteralTypo
+    public string Command => "mappingclientsidesetup";
+    public string Description => "Sets up the lighting control and such settings client-side. Sent by 'mapping' to client.";
+    public string Help => "";
+
+    public void Execute(IConsoleShell shell, string argStr, string[] args)
+    {
+        var mgr = IoCManager.Resolve<ILightManager>();
+        if (!mgr.LockConsoleAccess)
+        {
+            EntitySystem.Get<MarkerSystem>().MarkersVisible = true;
+            mgr.Enabled = false;
+            shell.ExecuteCommand("showsubfloorforever");
+            shell.ExecuteCommand("loadmapacts");
+        }
+    }
+}
+

Content.Server/GameTicking/Commands/MappingCommand.cs

@@ -85,10 +85,7 @@ namespace Content.Server.GameTicking.Commands
 
             shell.ExecuteCommand("sudo cvar events.enabled false");
             shell.ExecuteCommand($"tp 0 0 {mapId}");
-            shell.RemoteExecuteCommand("showmarkers");
+            shell.RemoteExecuteCommand("mappingclientsidesetup");
-            shell.RemoteExecuteCommand("togglelight");
-            shell.RemoteExecuteCommand("showsubfloorforever");
-            shell.RemoteExecuteCommand("loadmapacts");
             mapManager.SetMapPaused(mapId, true);
 
             if (args.Length == 2)