Admin Respawn Verb now works as intended (#21840)

* Respawn verb actually works now

For some reason the respawn verb was using the name of the person who used it instead of the person that was targetted.

* added changelog

* typo moment

* Removed unnecessary check

Content.Server/Administration/Systems/AdminVerbSystem.cs

@@ -156,6 +156,19 @@ namespace Content.Server.Administration.Systems
                         Impact = LogImpact.Extreme,
                         ConfirmationPopup = true
                     });
+
+                // Respawn
+                    args.Verbs.Add(new Verb()
+                    {
+                        Text = Loc.GetString("admin-player-actions-respawn"),
+                        Category = VerbCategory.Admin,
+                        Act = () =>
+                        {
+                            _console.ExecuteCommand(player, $"respawn {targetActor.PlayerSession.Name}");
+                        },
+                        ConfirmationPopup = true,
+                        // No logimpact as the command does it internally.
+                    });
                 }
 
                 // Admin Logs
@@ -211,21 +224,6 @@ namespace Content.Server.Administration.Systems
                     Impact = LogImpact.Low
                 });
 
-                // Respawn
-                if (HasComp<ActorComponent>(args.Target))
-                {
-                    args.Verbs.Add(new Verb()
-                    {
-                        Text = Loc.GetString("admin-player-actions-respawn"),
-                        Category = VerbCategory.Admin,
-                        Act = () =>
-                        {
-                            _console.ExecuteCommand(player, $"respawn {actor.PlayerSession.Name}");
-                        },
-                        ConfirmationPopup = true,
-                        // No logimpact as the command does it internally.
-                    });
-                }
             }
         }
 

Resources/Changelog/Admin.yml

@@ -1,4 +1,4 @@
-Name: Admin
+Name: Admin
 AdminOnly: true
 Order: 1
 Entries:
@@ -66,3 +66,7 @@ Entries:
   - {message: 'AME and PA make a sound effect when they are being overloaded. Similar to being sent a fax.', type: Add}
   id: 10
   time: '2023-11-07T15:03:00.0000000+00:00'
+- author: nikthechampiongr
+  changes:
+    - {message: 'The respawn verb now respawns the targeted player instead of the admin', type: Fix}
+  time: '2023-11-22T16:39:00.0000000+00:00'