Fix being able to start pulling something while incapacitated. (#4240)

Content.Client/MobState/MobStateComponent.cs

@@ -1,4 +1,5 @@
 using Content.Shared.MobState;
+using Content.Shared.MobState.Components;
 using Content.Shared.MobState.State;
 using Robust.Shared.GameObjects;
 

Content.Server/MobState/States/MobStateManager.cs

@@ -1,4 +1,5 @@
 using Content.Shared.MobState;
+using Content.Shared.MobState.Components;
 using Content.Shared.MobState.State;
 using Robust.Shared.GameObjects;
 

Content.Shared/MobState/Components/SharedMobStateComponent.cs

@@ -7,6 +7,7 @@ using Content.Shared.ActionBlocker;
 using Content.Shared.Alert;
 using Content.Shared.Damage;
 using Content.Shared.Damage.Components;
+using Content.Shared.MobState.State;
 using Content.Shared.NetIDs;
 using Robust.Shared.GameObjects;
 using Robust.Shared.Players;
@@ -14,7 +15,7 @@ using Robust.Shared.Serialization;
 using Robust.Shared.Serialization.Manager.Attributes;
 using Robust.Shared.ViewVariables;
 
-namespace Content.Shared.MobState.State
+namespace Content.Shared.MobState.Components
 {
     /// <summary>
     ///     When attached to an <see cref="IDamageableComponent"/>,

Content.Shared/MobState/EntitySystems/SharedMobStateSystem.cs

@@ -0,0 +1,22 @@
+using Content.Shared.MobState.Components;
+using Content.Shared.Pulling.Events;
+using Robust.Shared.GameObjects;
+
+namespace Content.Shared.MobState.EntitySystems
+{
+    public class SharedMobStateSystem : EntitySystem
+    {
+        public override void Initialize()
+        {
+            base.Initialize();
+
+            SubscribeLocalEvent<SharedMobStateComponent, StartPullAttemptEvent>(OnStartPullAttempt);
+        }
+
+        private void OnStartPullAttempt(EntityUid uid, SharedMobStateComponent component, StartPullAttemptEvent args)
+        {
+            if(component.IsIncapacitated())
+                args.Cancel();
+        }
+    }
+}

Content.Shared/Pulling/Components/SharedPullableComponent.cs

@@ -204,6 +204,11 @@ namespace Content.Shared.Pulling.Components
                 return false;
             }
 
+            if (!EntitySystem.Get<SharedPullingSystem>().CanPull(puller, Owner))
+            {
+                return false;
+            }
+
             if (_physics == null)
             {
                 return false;

Content.Shared/Pulling/Events/StartPullAttemptEvent.cs

@@ -0,0 +1,19 @@
+using Robust.Shared.GameObjects;
+
+namespace Content.Shared.Pulling.Events
+{
+    /// <summary>
+    ///     Directed event raised on the puller to see if it can start pulling something.
+    /// </summary>
+    public class StartPullAttemptEvent : CancellableEntityEventArgs
+    {
+        public StartPullAttemptEvent(IEntity puller, IEntity pulled)
+        {
+            Puller = puller;
+            Pulled = pulled;
+        }
+
+        public IEntity Puller { get; }
+        public IEntity Pulled { get; }
+    }
+}

Content.Shared/Pulling/SharedPullingSystem.cs

@@ -6,6 +6,7 @@ using Content.Shared.GameTicking;
 using Content.Shared.Input;
 using Content.Shared.Physics.Pull;
 using Content.Shared.Pulling.Components;
+using Content.Shared.Pulling.Events;
 using Content.Shared.Rotatable;
 using JetBrains.Annotations;
 using Robust.Shared.Containers;
@@ -237,5 +238,12 @@ namespace Content.Shared.Pulling
                     pulled.Transform.WorldRotation = newAngle;
             }
         }
+
+        public bool CanPull(IEntity puller, IEntity pulled)
+        {
+            var startPull = new StartPullAttemptEvent(puller, pulled);
+            RaiseLocalEvent(puller.Uid, startPull);
+            return !startPull.Cancelled;
+        }
     }
 }