From 2c8f960696e2f7a764ba3e43daff5be064d5d0b7 Mon Sep 17 00:00:00 2001 From: Dan Albert Date: Wed, 1 Sep 2021 19:19:57 -0700 Subject: [PATCH] Prevent creating empty ferry packages. An empty squadron or a fully-assigned squadron won't have anything to assign to the ferry mission. Fixes https://github.com/dcs-liberation/dcs_liberation/issues/1588 --- game/squadrons/squadron.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/game/squadrons/squadron.py b/game/squadrons/squadron.py index b9b242af..607a8d8a 100644 --- a/game/squadrons/squadron.py +++ b/game/squadrons/squadron.py @@ -369,9 +369,12 @@ class Squadron: raise RuntimeError( f"Cannot plan ferry flights for {self} because there is no destination." ) + remaining = self.untasked_aircraft + if not remaining: + return + package = Package(self.destination) builder = FlightPlanBuilder(package, self.coalition, theater) - remaining = self.untasked_aircraft while remaining: size = min(remaining, self.aircraft.max_group_size) self.plan_ferry_flight(builder, package, size)