Bram-Hub · charlestian23 · Apr 2, 2024 · Oct 20, 2023 · Oct 20, 2023 · Oct 27, 2023
diff --git a/src/main/java/edu/rpi/legup/history/AutoCaseRuleCommand.java b/src/main/java/edu/rpi/legup/history/AutoCaseRuleCommand.java
@@ -117,9  117,14 @@ public String getErrorString() {
             return "The selection must produce at least one case";
         }
 
-        if (caseRule.getCases(caseBoard.getBaseBoard(), elementView.getPuzzleElement()).size() > caseRule.MAX_CASES) {
         int numberOfCaseRules = caseRule.getCases(caseBoard.getBaseBoard(), elementView.getPuzzleElement()).size();
         System.out.println("Number of cases:"   numberOfCaseRules);
         if (numberOfCaseRules > caseRule.MAX_CASES) {
             return "The selection can produce a max of "   caseRule.MAX_CASES   " cases";
         }
         if (numberOfCaseRules < caseRule.MIN_CASES) {
             return "The selection must produce a minimum of "   caseRule.MIN_CASES   " cases";
         }
 
         return null;
     }

diff --git a/src/main/java/edu/rpi/legup/model/rules/CaseRule.java b/src/main/java/edu/rpi/legup/model/rules/CaseRule.java
@@ -15,6  15,7 @@ public abstract class CaseRule extends Rule {
 
     private final String INVALID_USE_MESSAGE;
     public int MAX_CASES;
     public int MIN_CASES;
 
     /**
      * CaseRule Constructor creates a new case rule.
@@ -29,6  30,7 @@ public CaseRule(String ruleID, String ruleName, String description, String image
         this.ruleType = CASE;
         this.INVALID_USE_MESSAGE = "Invalid use of the case rule "   this.ruleName;
         this.MAX_CASES = 10;
         this.MIN_CASES = 0; // By default, this will not actually have any effect
     }
 
     /**
@@ -95,17  97,8 @@ public String checkRule(TreeTransition transition) {
     public abstract String checkRuleRaw(TreeTransition transition);
 
     /**
-     * Checks whether the amount of cases generated is within bounds of the given rule.
-     * This method is the one that should overridden in child classes.
-     *
-     * @param transition    transition to check
-     * @param puzzleElement equivalent puzzleElement
-     * @return
-     */
-
-    /**
-     * Checks whether the child node logically follows from the parent node at the specific puzzleElement index using
-     * this rule.
      * Checks whether the child node logically follows from the parent node at the specific
      * puzzleElement index using this rule.
      *
      * @param transition transition to check
      * @param puzzleElement equivalent puzzleElement
@@ -155,6  148,3 @@ public List<PuzzleElement> dependentElements(Board board, PuzzleElement puzzleEl
         return elements;
     }
 }
-
-
-