diff --git a/changelog.md b/changelog.md index 675913e3..a989789f 100644 --- a/changelog.md +++ b/changelog.md @@ -83,6 +83,14 @@ Saves from 6.x are not compatible with 7.0. ## Fixes +# 6.1.1 + +## Fixes + +* **[Data]** Fixed unit ID for the KS-19 AAA. KS-19 would not previously generate correctly in missions. A new game is required for this fix to take effect. +* **[Flight Planning]** Automatic flight planning will no longer accidentally plan a recovery tanker instead of a theater refueling package. This fixes a potential crash during mission generation when opfor plans a refueling task at a sunk carrier. You'll need to skip the current turn to force opfor to replan their flights to get the fix. +* **[Mission Generation]** Using heliports (airports without any runways) will no longer cause mission generation to fail. + # 6.1.0 Saves from 6.0.0 are compatible with 6.1.0 diff --git a/game/theater/controlpoint.py b/game/theater/controlpoint.py index 3011a72d..9a408141 100644 --- a/game/theater/controlpoint.py +++ b/game/theater/controlpoint.py @@ -877,6 +877,11 @@ class ControlPoint(MissionTarget, SidcDescribable, ABC): ) -> RunwayData: ... + def stub_runway_data(self) -> RunwayData: + return RunwayData( + self.full_name, runway_heading=Heading.from_degrees(0), runway_name="" + ) + @property def airdrome_id_for_landing(self) -> Optional[int]: return None @@ -1140,6 +1145,14 @@ class Airfield(ControlPoint, CTLD): conditions: Conditions, dynamic_runways: Dict[str, RunwayData], ) -> RunwayData: + if not self.airport.runways: + # Some airfields are heliports and don't have any runways. This isn't really + # the best fix, since we should still try to generate partial data for TACAN + # beacons, but it'll do for a bug fix, and the proper fix probably involves + # making heliports their own CP type. + # https://github.com/dcs-liberation/dcs_liberation/issues/2710 + return self.stub_runway_data() + assigner = RunwayAssigner(conditions) return assigner.get_preferred_runway(theater, self.airport) @@ -1278,7 +1291,7 @@ class Carrier(NavalControlPoint): return SymbolSet.SEA_SURFACE, SeaSurfaceEntity.CARRIER def mission_types(self, for_player: bool) -> Iterator[FlightType]: - from game.ato import FlightType + from game.ato.flighttype import FlightType yield from super().mission_types(for_player) if self.is_friendly(for_player): @@ -1385,9 +1398,7 @@ class OffMapSpawn(ControlPoint): dynamic_runways: Dict[str, RunwayData], ) -> RunwayData: logging.warning("TODO: Off map spawns have no runways.") - return RunwayData( - self.full_name, runway_heading=Heading.from_degrees(0), runway_name="" - ) + return self.stub_runway_data() @property def runway_status(self) -> RunwayStatus: @@ -1435,9 +1446,7 @@ class Fob(ControlPoint, RadioFrequencyContainer, CTLD): dynamic_runways: Dict[str, RunwayData], ) -> RunwayData: logging.warning("TODO: FOBs have no runways.") - return RunwayData( - self.full_name, runway_heading=Heading.from_degrees(0), runway_name="" - ) + return self.stub_runway_data() @property def runway_status(self) -> RunwayStatus: