From 950e51e0feefd1e38d011dbc76c0614560aada0d Mon Sep 17 00:00:00 2001
From: austinio7116 <austinio@hotmail.com>
Date: Sun, 11 Apr 2021 16:06:53 +0100
Subject: [PATCH] Improved snow land decision making on LDA deck generator

---
 .../gamemodes/limited/ArchetypeDeckBuilder.java   |  2 +-
 .../gamemodes/limited/CardThemedDeckBuilder.java  | 15 +--------------
 2 files changed, 2 insertions(+), 15 deletions(-)

diff --git a/forge-gui/src/main/java/forge/gamemodes/limited/ArchetypeDeckBuilder.java b/forge-gui/src/main/java/forge/gamemodes/limited/ArchetypeDeckBuilder.java
index 5ab0c81547d..be3152ca171 100644
--- a/forge-gui/src/main/java/forge/gamemodes/limited/ArchetypeDeckBuilder.java
+++ b/forge-gui/src/main/java/forge/gamemodes/limited/ArchetypeDeckBuilder.java
@@ -21,7 +21,7 @@ public class ArchetypeDeckBuilder extends CardThemedDeckBuilder{
         archetype = archetype0;
         for(Pair<String, Double> pair : archetype.getCardProbabilities()){
             for(int i=0;i<5;++i){
-                if (pair.getLeft().equals(MagicColor.Constant.SNOW_LANDS.get(i))) {
+                if (pair.getLeft().equals(MagicColor.Constant.SNOW_LANDS.get(i)) && pair.getRight() > 0.04) {
                     needsSnowLands=true;
                     return;
                 }
diff --git a/forge-gui/src/main/java/forge/gamemodes/limited/CardThemedDeckBuilder.java b/forge-gui/src/main/java/forge/gamemodes/limited/CardThemedDeckBuilder.java
index fe7abad4058..08854347cae 100644
--- a/forge-gui/src/main/java/forge/gamemodes/limited/CardThemedDeckBuilder.java
+++ b/forge-gui/src/main/java/forge/gamemodes/limited/CardThemedDeckBuilder.java
@@ -704,21 +704,8 @@ public class CardThemedDeckBuilder extends DeckGeneratorBase {
                     System.out.printf("Basics[%s]: %d/%d = %f%% = %d cards%n", MagicColor.Constant.BASIC_LANDS.get(i), clrCnts[i], totalColor, 100*p, nLand + 1);
                 }
 
-                PaperCard snowLand = null;
-                // if appropriate snow-covered lands are available, add them
-                for (final PaperCard cp : basicLands) {
-                    if (cp.getName().equals(MagicColor.Constant.SNOW_LANDS.get(i))) {
-                        snowLand = cp;
-                        break;
-                    }
-                }
-
                 for (int j = 0; j < nLand; j++) {
-                    if(snowLand!=null){
-                        deckList.add(snowLand);
-                    }else {
-                        deckList.add(getBasicLand(i));
-                    }
+                    deckList.add(getBasicLand(i));
                 }
             }
         }