From dae3835eb0ef8571d15f8df515587578d8d40381 Mon Sep 17 00:00:00 2001
From: Dan Albert <dan@gingerhq.net>
Date: Thu, 27 May 2021 18:20:26 -0700
Subject: [PATCH] Don't assign pilots to unused aircraft.

These "flights" are only created so that we can spawn the aircraft on
the ramp for OCA strikes. They shouldn't have pilots assigned.
---
 gen/aircraft.py       | 1 +
 gen/flights/flight.py | 7 ++++++-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/gen/aircraft.py b/gen/aircraft.py
index fef009ba..5e64ee5e 100644
--- a/gen/aircraft.py
+++ b/gen/aircraft.py
@@ -1074,6 +1074,7 @@ class AircraftConflictGenerator:
                 departure=control_point,
                 arrival=control_point,
                 divert=None,
+                no_pilots=True,
             )
 
             group = self._generate_at_airport(
diff --git a/gen/flights/flight.py b/gen/flights/flight.py
index 711e1657..69e1ec3c 100644
--- a/gen/flights/flight.py
+++ b/gen/flights/flight.py
@@ -214,11 +214,16 @@ class Flight:
         divert: Optional[ControlPoint],
         custom_name: Optional[str] = None,
         cargo: Optional[TransferOrder] = None,
+        no_pilots: bool = False,
     ) -> None:
         self.package = package
         self.country = country
         self.squadron = squadron
-        self.pilots = [squadron.claim_available_pilot() for _ in range(count)]
+        self.pilots: List[Optional[Pilot]]
+        if no_pilots:
+            self.pilots = [None for _ in range(count)]
+        else:
+            self.pilots = [squadron.claim_available_pilot() for _ in range(count)]
         self.departure = departure
         self.arrival = arrival
         self.divert = divert