Properly fix chair collision & disallow pulling objects you're buckled to (#3241)

Co-authored-by: cyclowns <cyclowns@protonmail.ch>

Content.Client/GameObjects/Components/Buckle/BuckleComponent.cs

@@ -1,4 +1,5 @@
-using Content.Shared.GameObjects.Components.Buckle;
+#nullable enable
+using Content.Shared.GameObjects.Components.Buckle;
 using Robust.Client.GameObjects;
 using Robust.Shared.GameObjects;
 
@@ -19,7 +20,7 @@ namespace Content.Client.GameObjects.Components.Buckle
             return false;
         }
 
-        public override void HandleComponentState(ComponentState curState, ComponentState nextState)
+        public override void HandleComponentState(ComponentState? curState, ComponentState? nextState)
         {
             if (curState is not BuckleComponentState buckle)
             {
@@ -30,7 +31,7 @@ namespace Content.Client.GameObjects.Components.Buckle
             LastEntityBuckledTo = buckle.LastEntityBuckledTo;
             DontCollide = buckle.DontCollide;
 
-            if (!Owner.TryGetComponent(out SpriteComponent ownerSprite))
+            if (!Owner.TryGetComponent(out SpriteComponent? ownerSprite))
             {
                 return;
             }

Content.Server/GameObjects/Components/Buckle/BuckleComponent.cs

@@ -263,11 +263,20 @@ namespace Content.Server.GameObjects.Components.Buckle
 
             SendMessage(new BuckleMessage(Owner, to));
 
-            if (Owner.TryGetComponent(out PullableComponent? pullableComponent))
+            if (Owner.TryGetComponent(out PullableComponent? ownerPullable))
             {
-                if (pullableComponent.Puller != null)
+                if (ownerPullable.Puller != null)
                 {
-                    pullableComponent.TryStopPull();
+                    ownerPullable.TryStopPull();
+                }
+            }
+
+            if (to.TryGetComponent(out PullableComponent? toPullable))
+            {
+                if (toPullable.Puller == Owner)
+                {
+                    // can't pull it and buckle to it at the same time
+                    toPullable.TryStopPull();
                 }
             }
 

Resources/Prototypes/Entities/Constructible/Furniture/seats.yml

@@ -15,13 +15,6 @@
       bounds: "-0.5,-0.25,0.5,0.25"
       mask:
       - Impassable
-      - MobImpassable
-      - VaultImpassable
-      - SmallImpassable
-      layer:
-      - Opaque
-      - Impassable
-      - MobImpassable
       - VaultImpassable
       - SmallImpassable
   - type: Sprite
@@ -54,7 +47,10 @@
     shapes:
     - !type:PhysShapeAabb
       bounds: "-0.45, -0.45, 0.05, 0.45"
-        layer: [ Passable ]
+      mask:
+      - Impassable
+      - VaultImpassable
+      - SmallImpassable
 
 - type: entity
   name: stool
@@ -69,7 +65,10 @@
     shapes:
     - !type:PhysShapeAabb
       bounds: "-0.5, -0.25, 0.05, 0.25"
-        layer: [ Passable ]
+      mask:
+      - Impassable
+      - VaultImpassable
+      - SmallImpassable
 
 - type: entity
   name: bar stool
@@ -83,7 +82,10 @@
     shapes:
     - !type:PhysShapeAabb
       bounds: "-0.5, -0.2, 0.2, 0.25"
-        layer: [ Passable ]
+      mask:
+      - Impassable
+      - VaultImpassable
+      - SmallImpassable
 
 - type: entity
   name: white office chair
@@ -98,7 +100,10 @@
     shapes:
     - !type:PhysShapeAabb
       bounds: "-0.49, -0.25, 0.37, 0.25"
-        layer: [ Passable ]
+      mask:
+      - Impassable
+      - VaultImpassable
+      - SmallImpassable
 
 - type: entity
   name: dark office chair
@@ -120,7 +125,10 @@
     shapes:
     - !type:PhysShapeAabb
       bounds: "-0.45, -0.3, 0.35, 0.3"
-        layer: [ Passable ]
+      mask:
+      - Impassable
+      - VaultImpassable
+      - SmallImpassable
 
 - type: entity
   name: wooden chair
@@ -134,4 +142,7 @@
     shapes:
     - !type:PhysShapeAabb
       bounds: "-0.37, -0.25, 0.49, 0.24"
-        layer: [ Passable ]
+      mask:
+      - Impassable
+      - VaultImpassable
+      - SmallImpassable