Update content repo and RobustToolbox

Content.Client/GameObjects/Components/Actor/CharacterInterface.cs

@@ -78,10 +78,9 @@ namespace Content.Client.GameObjects.Components.Actor
             inputMgr.SetInputCommand(ContentKeyFunctions.OpenCharacterMenu, null);
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleMessage(ComponentMessage message, IComponent component)
-            IComponent component = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
 
             switch (message)
             {

Content.Client/GameObjects/Components/Construction/ConstructorComponent.cs

@@ -12,6 +12,7 @@ using Robust.Shared.Interfaces.Network;
 using Robust.Shared.IoC;
 using Robust.Shared.Map;
 using Robust.Shared.Maths;
+using Robust.Shared.Players;
 
 namespace Content.Client.GameObjects.Components.Construction
 {
@@ -33,9 +34,9 @@ namespace Content.Client.GameObjects.Components.Construction
             Owner.GetComponent<ITransformComponent>();
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleNetworkMessage(message, channel, session);
 
             switch (message)
             {

Content.Client/GameObjects/Components/HUD/Inventory/ClientInventoryComponent.cs

@@ -181,10 +181,9 @@ namespace Content.Client.GameObjects
             SendNetworkMessage(new OpenSlotStorageUIMessage(slot));
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleMessage(ComponentMessage message, IComponent component)
-            IComponent component = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
 
             switch (message)
             {

Content.Client/GameObjects/Components/Instruments/InstrumentComponent.cs

@@ -9,6 +9,7 @@ using Robust.Shared.Interfaces.GameObjects;
 using Robust.Shared.Interfaces.Network;
 using Robust.Shared.Interfaces.Timing;
 using Robust.Shared.IoC;
+using Robust.Shared.Players;
 using Robust.Shared.Serialization;
 using Robust.Shared.ViewVariables;
 using Timer = Robust.Shared.Timers.Timer;
@@ -109,10 +110,9 @@ namespace Content.Client.GameObjects.Components.Instruments
             serializer.DataField(ref _instrumentProgram, "program", 1);
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
-            IComponent component = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleNetworkMessage(message, channel, session);
 
             if (_renderer == null)
             {

Content.Client/GameObjects/Components/Items/ClientHandsComponent.cs

@@ -12,6 +12,7 @@ using Robust.Shared.GameObjects;
 using Robust.Shared.Interfaces.GameObjects;
 using Robust.Shared.Interfaces.Network;
 using Robust.Shared.IoC;
+using Robust.Shared.Players;
 using Robust.Shared.Serialization;
 using Robust.Shared.ViewVariables;
 
@@ -144,10 +145,35 @@ namespace Content.Client.GameObjects
             }
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleMessage(ComponentMessage message, IComponent component)
-            IComponent component = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
+
+            switch (message)
+            {
+                case PlayerAttachedMsg _:
+                    if (_gui == null)
+                    {
+                        _gui = new HandsGui();
+                    }
+                    else
+                    {
+                        _gui.Parent?.RemoveChild(_gui);
+                    }
+
+                    _gameHud.HandsContainer.AddChild(_gui);
+                    _gui.UpdateHandIcons();
+                    break;
+
+                case PlayerDetachedMsg _:
+                    _gui.Parent?.RemoveChild(_gui);
+                    break;
+            }
+        }
+
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
+        {
+            base.HandleNetworkMessage(message, channel, session);
 
             switch (message)
             {

Content.Client/GameObjects/Components/Mobs/CameraRecoilComponent.cs

@@ -6,6 +6,7 @@ using Robust.Shared.Interfaces.GameObjects;
 using Robust.Shared.Interfaces.Network;
 using Robust.Shared.Log;
 using Robust.Shared.Maths;
+using Robust.Shared.Players;
 
 namespace Content.Client.GameObjects.Components.Mobs
 {
@@ -62,9 +63,21 @@ namespace Content.Client.GameObjects.Components.Mobs
             _updateEye();
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleMessage(ComponentMessage message, IComponent component)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
+
+            switch (message)
+            {
+                case RecoilKickMessage msg:
+                    Kick(msg.Recoil);
+                    break;
+            }
+        }
+
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
+        {
+            base.HandleNetworkMessage(message, channel, session);
 
             switch (message)
             {

Content.Client/GameObjects/Components/Mobs/ClientOverlayEffectsComponent.cs

@@ -10,6 +10,7 @@ using Robust.Shared.Interfaces.GameObjects;
 using Robust.Shared.Interfaces.Network;
 using Robust.Shared.IoC;
 using Robust.Shared.Log;
+using Robust.Shared.Players;
 
 namespace Content.Client.GameObjects
 {
@@ -53,7 +54,20 @@ namespace Content.Client.GameObjects
             };
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleMessage(ComponentMessage message, IComponent component)
+        {
+            switch (message)
+            {
+                case PlayerAttachedMsg _:
+                    SetOverlay(_currentEffect);
+                    break;
+                case PlayerDetachedMsg _:
+                    RemoveOverlay();
+                    break;
+            }
+        }
+
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
         {
             switch (message)
             {

Content.Client/GameObjects/Components/Mobs/ClientStatusEffectsComponent.cs

@@ -13,6 +13,7 @@ using Robust.Shared.Interfaces.GameObjects;
 using Robust.Shared.Interfaces.Network;
 using Robust.Shared.IoC;
 using Robust.Shared.Log;
+using Robust.Shared.Players;
 
 namespace Content.Client.GameObjects.Components.Mobs
 {
@@ -40,10 +41,9 @@ namespace Content.Client.GameObjects.Components.Mobs
             PlayerDetached();
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleMessage(ComponentMessage message, IComponent component)
-            IComponent component = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
             switch (message)
             {
                 case PlayerAttachedMsg _:

Content.Client/GameObjects/Components/Mobs/CombatModeComponent.cs

@@ -7,6 +7,7 @@ using Robust.Shared.GameObjects;
 using Robust.Shared.Interfaces.GameObjects;
 using Robust.Shared.Interfaces.Network;
 using Robust.Shared.IoC;
+using Robust.Shared.Players;
 
 namespace Content.Client.GameObjects.Components.Mobs
 {
@@ -39,9 +40,9 @@ namespace Content.Client.GameObjects.Components.Mobs
             }
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleMessage(ComponentMessage message, IComponent component)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
 
             switch (message)
             {

Content.Client/GameObjects/Components/Observer/GhostComponent.cs

@@ -57,10 +57,9 @@ namespace Content.Client.GameObjects.Components.Observer
                 component.Visible = _playerManager.LocalPlayer.ControlledEntity?.HasComponent<GhostComponent>() ?? false;
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleMessage(ComponentMessage message, IComponent component)
-            IComponent component = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
 
             switch (message)
             {

Content.Client/GameObjects/Components/Power/PowerDebugTool.cs

@@ -4,6 +4,7 @@ using Robust.Client.UserInterface.CustomControls;
 using Robust.Shared.GameObjects;
 using Robust.Shared.Interfaces.GameObjects;
 using Robust.Shared.Interfaces.Network;
+using Robust.Shared.Players;
 
 namespace Content.Client.GameObjects.Components.Power
 {
@@ -11,9 +12,9 @@ namespace Content.Client.GameObjects.Components.Power
     public class PowerDebugTool : SharedPowerDebugTool
     {
         SS14Window LastWindow;
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleNetworkMessage(message, channel, session);
 
             switch (message)
             {

Content.Client/GameObjects/Components/Sound/SoundComponent.cs

@@ -6,6 +6,7 @@ using Robust.Shared.Interfaces.GameObjects;
 using Robust.Shared.Interfaces.Network;
 using Robust.Shared.Interfaces.Random;
 using Robust.Shared.IoC;
+using Robust.Shared.Players;
 using Robust.Shared.Serialization;
 using Robust.Shared.Timers;
 
@@ -68,9 +69,9 @@ namespace Content.Client.GameObjects.Components.Sound
                 });
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleNetworkMessage(message, channel, session);
             switch (message)
             {
                 case ScheduledSoundMessage msg:

Content.Client/GameObjects/Components/Storage/ClientStorageComponent.cs

@@ -12,6 +12,7 @@ using Robust.Shared.Interfaces.GameObjects;
 using Robust.Shared.Interfaces.Network;
 using Robust.Shared.IoC;
 using Robust.Shared.Maths;
+using Robust.Shared.Players;
 
 namespace Content.Client.GameObjects.Components.Storage
 {
@@ -40,9 +41,10 @@ namespace Content.Client.GameObjects.Components.Storage
             base.OnRemove();
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
-            IComponent component = null)
         {
+            base.HandleNetworkMessage(message, channel, session);
+
             switch (message)
             {
                 //Updates what we are storing for the UI

Content.Client/GameObjects/Components/Weapons/Ranged/BallisticMagazineWeaponComponent.cs

@@ -14,6 +14,7 @@ using Robust.Shared.GameObjects;
 using Robust.Shared.Interfaces.GameObjects;
 using Robust.Shared.Interfaces.Network;
 using Robust.Shared.Maths;
+using Robust.Shared.Players;
 using Robust.Shared.Serialization;
 using Robust.Shared.ViewVariables;
 using static Content.Client.StaticIoC;
@@ -108,17 +109,16 @@ namespace Content.Client.GameObjects.Components.Weapons.Ranged
             _statusControl?.Update();
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
-            IComponent component = null)
         {
+            base.HandleNetworkMessage(message, channel, session);
+
             switch (message)
             {
                 case BmwComponentAutoEjectedMessage _:
                     _statusControl?.PlayAlarmAnimation();
                     return;
             }
-
-            base.HandleMessage(message, netChannel, component);
         }
 
         public Control MakeControl()

Content.Server/GameObjects/Components/Construction/ConstructorComponent.cs

@@ -14,6 +14,7 @@ using Robust.Shared.IoC;
 using Robust.Shared.Localization;
 using Robust.Shared.Map;
 using Robust.Shared.Maths;
+using Robust.Shared.Players;
 using Robust.Shared.Prototypes;
 
 namespace Content.Server.GameObjects.Components.Construction
@@ -30,9 +31,10 @@ namespace Content.Server.GameObjects.Components.Construction
         [Dependency] private readonly ILocalizationManager _localizationManager;
 #pragma warning restore 649
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleNetworkMessage(message, channel, session);
+
             switch (message)
             {
                 case TryStartStructureConstructionMessage tryStart:

Content.Server/GameObjects/Components/Doors/ServerDoorComponent.cs

@@ -85,9 +85,9 @@ namespace Content.Server.GameObjects
             ActivateImpl(eventArgs);
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleMessage(ComponentMessage message, IComponent component)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
 
             switch (message)
             {

Content.Server/GameObjects/Components/GUI/HumanInventoryControllerComponent.cs

@@ -51,9 +51,9 @@ namespace Content.Server.GameObjects
             return flagsCheck;
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleMessage(ComponentMessage message, IComponent component)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
 
             switch (message)
             {

Content.Server/GameObjects/Components/GUI/InventoryComponent.cs

@@ -11,6 +11,7 @@ using Robust.Shared.Interfaces.GameObjects.Components;
 using Robust.Shared.Interfaces.Network;
 using Robust.Shared.IoC;
 using Robust.Shared.Map;
+using Robust.Shared.Players;
 using Robust.Shared.ViewVariables;
 using static Content.Shared.GameObjects.Components.Inventory.EquipmentSlotDefines;
 using static Content.Shared.GameObjects.SharedInventoryComponent.ClientInventoryMessage;
@@ -304,16 +305,35 @@ namespace Content.Server.GameObjects
         }
 
         /// <inheritdoc />
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleMessage(ComponentMessage message, IComponent component)
-            IComponent component = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
+
+            switch (message)
+            {
+                case ContainerContentsModifiedMessage msg:
+                    if (msg.Removed)
+                        ForceUnequip(msg.Container, msg.Entity);
+                    break;
+
+                default:
+                    break;
+            }
+        }
+
+        /// <inheritdoc />
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel netChannel, ICommonSession session = null)
+        {
+            base.HandleNetworkMessage(message, netChannel, session);
+
+            if (session == null)
+            {
+                throw new ArgumentNullException(nameof(session));
+            }
 
             switch (message)
             {
                 case ClientInventoryMessage msg:
-                    var playerMan = IoCManager.Resolve<IPlayerManager>();
-                    var session = playerMan.GetSessionByChannel(netChannel);
                     var playerentity = session.AttachedEntity;
 
                     if (playerentity == Owner)
@@ -327,11 +347,6 @@ namespace Content.Server.GameObjects
                     if (item != null && item.Owner.TryGetComponent(out ServerStorageComponent storage))
                         storage.OpenStorageUI(Owner);
                     break;
-
-                case ContainerContentsModifiedMessage msg:
-                    if (msg.Removed)
-                        ForceUnequip(msg.Container, msg.Entity);
-                    break;
             }
         }
 

Content.Server/GameObjects/Components/GUI/ServerHandsComponent.cs

@@ -17,6 +17,7 @@ using Robust.Shared.IoC;
 using Robust.Shared.Log;
 using Robust.Shared.Map;
 using Robust.Shared.Maths;
+using Robust.Shared.Players;
 using Robust.Shared.Serialization;
 using Robust.Shared.ViewVariables;
 
@@ -448,17 +449,19 @@ namespace Content.Server.GameObjects
             return false;
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
-            IComponent component = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleNetworkMessage(message, channel, session);
+
+            if (session == null)
+            {
+                throw new ArgumentNullException(nameof(session));
+            }
 
             switch (message)
             {
                 case ClientChangedHandMsg msg:
                 {
-                    var playerMan = IoCManager.Resolve<IPlayerManager>();
-                    var session = playerMan.GetSessionByChannel(netChannel);
                     var playerEntity = session.AttachedEntity;
 
                     if (playerEntity == Owner && HasHand(msg.Index))
@@ -475,8 +478,6 @@ namespace Content.Server.GameObjects
                         return;
                     }
 
-                    var playerMan = IoCManager.Resolve<IPlayerManager>();
-                    var session = playerMan.GetSessionByChannel(netChannel);
                     var playerEntity = session.AttachedEntity;
                     var used = GetActiveHand?.Owner;
 
@@ -502,8 +503,6 @@ namespace Content.Server.GameObjects
 
                 case UseInHandMsg msg:
                 {
-                    var playerMan = IoCManager.Resolve<IPlayerManager>();
-                    var session = playerMan.GetSessionByChannel(netChannel);
                     var playerEntity = session.AttachedEntity;
                     var used = GetActiveHand?.Owner;
 
@@ -518,8 +517,6 @@ namespace Content.Server.GameObjects
 
                 case ActivateInHandMsg msg:
                 {
-                    var playerMan = IoCManager.Resolve<IPlayerManager>();
-                    var session = playerMan.GetSessionByChannel(netChannel);
                     var playerEntity = session.AttachedEntity;
                     var used = GetHand(msg.Index)?.Owner;
 

Content.Server/GameObjects/Components/Instruments/InstrumentComponent.cs

@@ -7,6 +7,7 @@ using Robust.Server.Interfaces.Player;
 using Robust.Shared.GameObjects;
 using Robust.Shared.Interfaces.GameObjects;
 using Robust.Shared.Interfaces.Network;
+using Robust.Shared.Players;
 using Robust.Shared.Serialization;
 using Robust.Shared.ViewVariables;
 
@@ -45,11 +46,11 @@ namespace Content.Server.GameObjects.Components.Instruments
             serializer.DataField(ref _handheld, "handheld", false);
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleNetworkMessage(message, channel, session);
             // If the client that sent the message isn't the client playing this instrument, we ignore it.
-            if (netChannel != _instrumentPlayer) return;
+            if (channel != _instrumentPlayer) return;
             switch (message)
             {
                 case InstrumentMidiEventMessage midiEventMsg:

Content.Server/GameObjects/Components/Items/Storage/EntityStorageComponent.cs

@@ -272,9 +272,9 @@ namespace Content.Server.GameObjects.Components
         }
 
         /// <inheritdoc />
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleMessage(ComponentMessage message, IComponent component)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
 
             switch (message)
             {

Content.Server/GameObjects/Components/Items/Storage/ServerStorageComponent.cs

@@ -1,4 +1,5 @@
-using System.Collections.Generic;
+using System;
+using System.Collections.Generic;
 using System.Linq;
 using Content.Server.GameObjects.Components;
 using Content.Server.GameObjects.Components.Items.Storage;
@@ -19,6 +20,7 @@ using Robust.Shared.Interfaces.Map;
 using Robust.Shared.Interfaces.Network;
 using Robust.Shared.IoC;
 using Robust.Shared.Log;
+using Robust.Shared.Players;
 using Robust.Shared.Serialization;
 
 namespace Content.Server.GameObjects
@@ -33,7 +35,6 @@ namespace Content.Server.GameObjects
     {
 #pragma warning disable 649
         [Dependency] private readonly IMapManager _mapManager;
-        [Dependency] private readonly IPlayerManager _playerManager;
         [Dependency] private readonly IEntityManager _entityManager;
 #pragma warning restore 649
 
@@ -255,18 +256,22 @@ namespace Content.Server.GameObjects
         /// and puts the removed entity in hand or on the ground
         /// </summary>
         /// <param name="message"></param>
-        /// <param name="netChannel"></param>
+        /// <param name="channel"></param>
-        /// <param name="component"></param>
+        /// <param name="session"></param>
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleNetworkMessage(message, channel, session);
+
+            if (session == null)
+            {
+                throw new ArgumentException(nameof(session));
+            }
 
             switch (message)
             {
                 case RemoveEntityMessage _:
                 {
                     _ensureInitialCalculated();
-                    var session = _playerManager.GetSessionByChannel(netChannel);
                     var playerentity = session.AttachedEntity;
 
                     var ourtransform = Owner.GetComponent<ITransformComponent>();
@@ -296,7 +301,7 @@ namespace Content.Server.GameObjects
                 case InsertEntityMessage _:
                 {
                     _ensureInitialCalculated();
-                    var playerEntity = _playerManager.GetSessionByChannel(netChannel).AttachedEntity;
+                    var playerEntity = session.AttachedEntity;
                     var storageTransform = Owner.GetComponent<ITransformComponent>();
                     var playerTransform = playerEntity.GetComponent<ITransformComponent>();
                     // TODO: Replace by proper entity range check once it is implemented.
@@ -311,9 +316,7 @@ namespace Content.Server.GameObjects
                 }
                 case CloseStorageUIMessage _:
                 {
-                    var session = _playerManager.GetSessionByChannel(netChannel);
+                    UnsubscribeSession(session as IPlayerSession);
-
-                    UnsubscribeSession(session);
                     break;
                 }
             }

Content.Server/GameObjects/Components/Mobs/SpeciesComponent.cs

@@ -54,9 +54,10 @@ namespace Content.Server.GameObjects
             serializer.DataFieldCached(ref _heatResistance, "HeatResistance", 323);
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleMessage(ComponentMessage message, IComponent component)
-            IComponent component = null)
         {
+            base.HandleMessage(message, component);
+
             switch (message)
             {
                 case PlayerAttachedMsg _:

Content.Server/GameObjects/Components/Movement/ShuttleControllerComponent.cs

@@ -105,12 +105,9 @@ namespace Content.Server.GameObjects.Components.Movement
         }
 
         /// <inheritdoc />
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null, IComponent component = null)
+        public override void HandleMessage(ComponentMessage message, IComponent component)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
-
-            if(netChannel != null)
-                return;
 
             switch (message)
             {

Content.Server/GameObjects/Components/Observer/GhostComponent.cs

@@ -7,6 +7,7 @@ using Robust.Server.Interfaces.GameObjects;
 using Robust.Shared.GameObjects;
 using Robust.Shared.Interfaces.GameObjects;
 using Robust.Shared.Interfaces.Network;
+using Robust.Shared.Players;
 using Robust.Shared.ViewVariables;
 using Timer = Robust.Shared.Timers.Timer;
 
@@ -38,10 +39,27 @@ namespace Content.Server.GameObjects.Components.Observer
 
         public override ComponentState GetComponentState() => new GhostComponentState(CanReturnToBody);
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleMessage(ComponentMessage message, IComponent component)
-            IComponent component = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleMessage(message, component);
+
+            switch (message)
+            {
+                case PlayerAttachedMsg msg:
+                    msg.NewPlayer.VisibilityMask |= (int)VisibilityFlags.Ghost;
+                    Dirty();
+                    break;
+                case PlayerDetachedMsg msg:
+                    msg.OldPlayer.VisibilityMask &= ~(int)VisibilityFlags.Ghost;
+                    break;
+                default:
+                    break;
+            }
+        }
+
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel netChannel, ICommonSession session = null)
+        {
+            base.HandleNetworkMessage(message, netChannel, session);
 
             switch (message)
             {
@@ -53,13 +71,6 @@ namespace Content.Server.GameObjects.Components.Observer
                         Owner.Delete();
                     }
                     break;
-                case PlayerAttachedMsg msg:
-                    msg.NewPlayer.VisibilityMask |= (int)VisibilityFlags.Ghost;
-                    Dirty();
-                    break;
-                case PlayerDetachedMsg msg:
-                    msg.OldPlayer.VisibilityMask &= ~(int)VisibilityFlags.Ghost;
-                    break;
                 default:
                     break;
             }

Content.Server/GameObjects/Components/Weapon/Ranged/RangedWeapon.cs

@@ -9,6 +9,7 @@ using Robust.Shared.Interfaces.Network;
 using Robust.Shared.Interfaces.Timing;
 using Robust.Shared.IoC;
 using Robust.Shared.Map;
+using Robust.Shared.Players;
 using Robust.Shared.Timers;
 
 namespace Content.Server.GameObjects.Components.Weapon.Ranged
@@ -40,16 +41,18 @@ namespace Content.Server.GameObjects.Components.Weapon.Ranged
             FireHandler?.Invoke(user, clickLocation);
         }
 
-        public override void HandleMessage(ComponentMessage message, INetChannel netChannel = null,
+        public override void HandleNetworkMessage(ComponentMessage message, INetChannel channel, ICommonSession session = null)
-            IComponent component = null)
         {
-            base.HandleMessage(message, netChannel, component);
+            base.HandleNetworkMessage(message, channel, session);
+
+            if (session == null)
+            {
+                throw new ArgumentNullException(nameof(session));
+            }
 
             switch (message)
             {
                 case SyncFirePosMessage msg:
-                    var playerMgr = IoCManager.Resolve<IPlayerManager>();
-                    var session = playerMgr.GetSessionByChannel(netChannel);
                     var user = session.AttachedEntity;
                     if (user == null)
                     {