Techs from ancient ruins now added properly

core/src/com/unciv/logic/GameInfo.kt

@@ -92,6 +92,8 @@ class GameInfo {
             tile = viableTiles.getRandom()
         }
 
+        // if we don't make this into a separate list then the retain() will happen on the Tech keys,
+        // which effectively removes those techs from the game and causes all sorts of problems
         val allResearchedTechs = GameBasics.Technologies.keys.toMutableList()
         for (civ in civilizations.filter { !it.isBarbarianCivilization() && !it.isDefeated() }) {
             allResearchedTechs.retainAll(civ.tech.techsResearched)

core/src/com/unciv/logic/civilization/CivilizationInfo.kt

@@ -70,7 +70,7 @@ class CivilizationInfo {
 
     constructor(civName: String) {
         this.civName = civName
-        tech.techsResearched.add("Agriculture")
+        tech.addTechnology("Agriculture")
     }
 
     fun clone(): CivilizationInfo {

core/src/com/unciv/logic/map/MapUnit.kt

@@ -416,8 +416,7 @@ class MapUnit {
         if(researchableAncientEraTechs.isNotEmpty())
             actions.add {
                 val tech = researchableAncientEraTechs.getRandom().name
-                civInfo.tech.techsResearched.add(tech)
-                if(civInfo.tech.techsToResearch.contains(tech)) civInfo.tech.techsToResearch.remove(tech)
+                civInfo.tech.addTechnology(tech)
                 civInfo.addNotification("We have discovered the lost technology of [$tech] in the ruins!",currentTile.position, Color.BLUE)
             }
 

core/src/com/unciv/ui/saves/LoadScreen.kt

@@ -82,7 +82,6 @@ class LoadScreen : PickerScreen() {
             UnCivGame.Current.loadGame(selectedSave)
         }
 
-
     }
 
 }