From 5fab21d09909446ba7c59cd145eb07a5941f27a0 Mon Sep 17 00:00:00 2001 From: Silvan Fuhrer Date: Fri, 5 Apr 2024 15:48:50 +0200 Subject: [PATCH] MissionBase: hasMissionLandStart should only return true if mission is valid Signed-off-by: Silvan Fuhrer --- src/modules/navigator/mission_base.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/modules/navigator/mission_base.h b/src/modules/navigator/mission_base.h index 76f07882b7..405291969e 100644 --- a/src/modules/navigator/mission_base.h +++ b/src/modules/navigator/mission_base.h @@ -118,12 +118,12 @@ protected: void getNextPositionItems(int32_t start_index, int32_t items_index[], size_t &num_found_items, uint8_t max_num_items); /** - * @brief Has Mission a Land Start or Land Item + * @brief Mission has a land start, a land, and is valid * - * @return true If mission has a land start of land item and a land item + * @return true If mission has a land start and a land item and is valid * @return false otherwise */ - bool hasMissionLandStart() const { return _mission.land_start_index >= 0 && _mission.land_index >= 0;}; + bool hasMissionLandStart() const { return _mission.land_start_index >= 0 && _mission.land_index >= 0 && isMissionValid();}; /** * @brief Go to next Mission Item * Go to next non jump mission item