Fix salvage deletion (#16502)

diff --git a/Content.Server/Salvage/SalvageSystem.Expeditions.cs b/Content.Server/Salvage/SalvageSystem.Expeditions.cs
index e5bbce38ea06cd4706ee259f35c562df3d5e81fc..31a1c5fd633a7cf4205ac9f58207061b17d06df5 100644 (file)
--- a/Content.Server/Salvage/SalvageSystem.Expeditions.cs
+++ b/Content.Server/Salvage/SalvageSystem.Expeditions.cs
@@ -146,16 +146,6 @@ public sealed partial class SalvageSystem
             GenerateMissions(comp);
             UpdateConsoles(comp);
         }
-
-        var query = EntityQueryEnumerator<SalvageExpeditionComponent>();
-
-        while (query.MoveNext(out var uid, out var comp))
-        {
-            if (comp.EndTime < currentTime)
-            {
-                QueueDel(uid);
-            }
-        }
     }
 
     private void FinishExpedition(SalvageExpeditionDataComponent component, SalvageExpeditionComponent expedition, EntityUid? shuttle)

diff --git a/Content.Server/Salvage/SalvageSystem.Runner.cs b/Content.Server/Salvage/SalvageSystem.Runner.cs
index 710fb9f84e77dccaa8ecac4214669b325a8ee20e..01f7340d42360c0021ab4c122161e8365ae13626 100644 (file)
--- a/Content.Server/Salvage/SalvageSystem.Runner.cs
+++ b/Content.Server/Salvage/SalvageSystem.Runner.cs
@@ -122,9 +122,6 @@ public sealed partial class SalvageSystem
         // Run the basic mission timers (e.g. announcements, auto-FTL, completion, etc)
         while (query.MoveNext(out var uid, out var comp))
         {
-            if (comp.Completed)
-                continue;
-
             var remaining = comp.EndTime - _timing.CurTime;
 
             if (comp.Stage < ExpeditionStage.FinalCountdown && remaining < TimeSpan.FromSeconds(30))
@@ -174,6 +171,11 @@ public sealed partial class SalvageSystem
                     }
                 }
             }
+
+            if (remaining < TimeSpan.Zero)
+            {
+                QueueDel(uid);
+            }
         }
 
         // Mining missions: NOOP