apply 's review

space-wizards · Sep 20, 2024 · 0aca706 · 0aca706
1 parent 31747a0
commit 0aca706
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/Content.Server/Objectives/Components/PickRandomPersonComponent.cs b/Content.Server/Objectives/Components/PickRandomPersonComponent.cs
@@ -13,6 +13,6 @@ public sealed partial class PickRandomPersonComponent : Component
     /// <summary>
     /// List of jobs, that won't be kill objectives
     /// </summary>
-    [DataField, ViewVariables(VVAccess.ReadWrite)]
+    [DataField]
     public List<ProtoId<JobPrototype>?> IgnoredJobs = new();
 }
diff --git a/Content.Server/Objectives/Systems/KillPersonConditionSystem.cs b/Content.Server/Objectives/Systems/KillPersonConditionSystem.cs
@@ -58,9 +58,7 @@ private void OnPersonAssigned(EntityUid uid, PickRandomPersonComponent comp, ref
         var allHumans = _mind.GetAliveHumansExcept(args.MindId);
 
         // filter jobs that we don't want to be kill objectives
-        allHumans = allHumans
-            .Where(human => !_job.MindTryGetJobId(human, out var jobId) || !comp.IgnoredJobs.Contains(jobId))
-            .ToList();
+        allHumans.RemoveAll(human => !_job.MindTryGetJobId(human, out var jobId) || !comp.IgnoredJobs.Contains(jobId))
 
         if (allHumans.Count == 0)
         {