Fixed bug yannprada#14

From https://github.com/cookie-crumb/cookie-garden-helper
hyoretsu · Feb 13, 2021 · 826582e · 826582e
1 parent 5d8692f
commit 826582e
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/cookie-garden-helper.js b/cookie-garden-helper.js
@@ -72,6 +72,10 @@ class Garden {
     return (this.minigame.nextStep - Date.now()) / 1000;
   }
 
+  static hasHarvestBenefit(plant) {
+    return typeof plant.onHarvest === 'function';
+  }
+
   static get selectedSeed() {
     return this.minigame.seedSelected;
   }
@@ -167,7 +171,11 @@ class Garden {
   static handleMature(config, plant, x, y) {
     if (!plant.unlocked && config.autoHarvestNewSeeds) {
       this.harvest(x, y);
-    } else if (config.autoHarvestCheckCpSMult && this.CpSMult >= config.autoHarvestMiniCpSMult.value) {
+    } else if (
+      config.autoHarvestCheckCpSMult &&
+      this.hasHarvestBenefit(plant) &&
+      this.CpSMult >= config.autoHarvestMiniCpSMult.value
+    ) {
       this.harvest(x, y);
     }
   }

diff --git a/src/Garden.js b/src/Garden.js
@@ -22,6 +22,10 @@ class Garden {
     return (this.minigame.nextStep - Date.now()) / 1000;
   }
 
+  static hasHarvestBenefit(plant) {
+    return typeof plant.onHarvest === 'function';
+  }
+
   static get selectedSeed() {
     return this.minigame.seedSelected;
   }
@@ -117,7 +121,11 @@ class Garden {
   static handleMature(config, plant, x, y) {
     if (!plant.unlocked && config.autoHarvestNewSeeds) {
       this.harvest(x, y);
-    } else if (config.autoHarvestCheckCpSMult && this.CpSMult >= config.autoHarvestMiniCpSMult.value) {
+    } else if (
+      config.autoHarvestCheckCpSMult &&
+      this.hasHarvestBenefit(plant) &&
+      this.CpSMult >= config.autoHarvestMiniCpSMult.value
+    ) {
       this.harvest(x, y);
     }
   }