Don't pushback when throwing while buckled (#4232)

* Don't pushback when throwing while buckled

* Address reviews

* Fix

Content.Server/Throwing/ThrowHelper.cs

@@ -54,7 +54,13 @@ namespace Content.Server.Throwing
             // Give thrower an impulse in the other direction
             if (user != null && pushbackRatio > 0.0f && user.TryGetComponent(out IPhysBody? body))
             {
-                body.ApplyLinearImpulse(-direction * pushbackRatio);
+                var msg = new ThrowPushbackAttemptEvent();
+                body.Owner.EntityManager.EventBus.RaiseLocalEvent(body.Owner.Uid, msg);
+
+                if (!msg.Cancelled)
+                {
+                    body.ApplyLinearImpulse(-direction * pushbackRatio);
+                }
             }
         }
     }

Content.Shared/Buckle/SharedBuckleSystem.cs

@@ -1,5 +1,6 @@
 using Content.Shared.Buckle.Components;
 using Content.Shared.Standing;
+using Content.Shared.Throwing;
 using Robust.Shared.GameObjects;
 using Robust.Shared.Physics.Dynamics;
 
@@ -13,6 +14,7 @@ namespace Content.Shared.Buckle
             SubscribeLocalEvent<SharedBuckleComponent, PreventCollideEvent>(PreventCollision);
             SubscribeLocalEvent<SharedBuckleComponent, DownAttemptEvent>(HandleDown);
             SubscribeLocalEvent<SharedBuckleComponent, StandAttemptEvent>(HandleStand);
+            SubscribeLocalEvent<SharedBuckleComponent, ThrowPushbackAttemptEvent>(HandleThrowPushback);
         }
 
         private void HandleStand(EntityUid uid, SharedBuckleComponent component, StandAttemptEvent args)
@@ -31,6 +33,12 @@ namespace Content.Shared.Buckle
             }
         }
 
+        private void HandleThrowPushback(EntityUid uid, SharedBuckleComponent component, ThrowPushbackAttemptEvent args)
+        {
+            if (!component.Buckled) return;
+            args.Cancel();
+        }
+
         private void PreventCollision(EntityUid uid, SharedBuckleComponent component, PreventCollideEvent args)
         {
             if (args.BodyB.Owner.Uid != component.LastEntityBuckledTo) return;

Content.Shared/Throwing/ThrowAttemptEvent.cs

@@ -11,4 +11,9 @@ namespace Content.Shared.Throwing
 
         public IEntity Entity { get; }
     }
+
+    /// <summary>
+    /// Raised when we try to pushback an entity from throwing
+    /// </summary>
+    public sealed class ThrowPushbackAttemptEvent : CancellableEntityEventArgs {}
 }