From 27854bc0312e13178dea4b9b35f3274fcb89b96f Mon Sep 17 00:00:00 2001
From: Skekdog <47750815+Skekdog@users.noreply.github.com>
Date: Mon, 27 Jun 2022 13:51:46 +0100
Subject: [PATCH] Fix crash when exploring (#7314)

* Exploring units now take several turns to reach a ruin (but only if it is in vision, not behind fog of war)
Re-did this because the last pull request was a mess and so was everything else I was doing

* Fix crash reported by @Aditya von Karma#8015 on discord

* Update UnitAutomation.kt

Use canReach instead of canMoveTo

* Update UnitAutomation.kt

Use both canReach and canMoveTo
---
 core/src/com/unciv/logic/automation/UnitAutomation.kt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/src/com/unciv/logic/automation/UnitAutomation.kt b/core/src/com/unciv/logic/automation/UnitAutomation.kt
index 2c432e207c..18c28f3ff4 100644
--- a/core/src/com/unciv/logic/automation/UnitAutomation.kt
+++ b/core/src/com/unciv/logic/automation/UnitAutomation.kt
@@ -55,7 +55,7 @@ object UnitAutomation {
                         (it.improvement != null && it.getTileImprovement()!!.isAncientRuinsEquivalent())
                                 || it.improvement == Constants.barbarianEncampment
                         )
-                        && unit.movement.canMoveTo(it)
+                        && unit.movement.canMoveTo(it) && unit.movement.canReach(it)
             } ?: return false
         unit.movement.headTowards(tileWithRuinOrEncampment)
         return true