Hot patch for server issues by limiting enchantment price calculator

+ Hot Patch for server issues by limiting enchantment price calculator and not requesting info at the correct times
noahbclarkson · Dec 18, 2020 · 3cb1b6b · 3cb1b6b
1 parent ed16072
commit 3cb1b6b
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 18 deletions.
diff --git a/Auto-Tune/dependency-reduced-pom.xml b/Auto-Tune/dependency-reduced-pom.xml
@@ -4,7 +4,7 @@
   <groupId>unprotesting.com.github</groupId>
   <artifactId>Auto-Tune</artifactId>
   <name>Auto-Tune</name>
-  <version>0.12.0-pre-release-5</version>
+  <version>0.12.0-pre-release-6</version>
   <description>The automatic pricing plugin for minecraft</description>
   <url>https://github.com/Unprotesting/Auto-Tune</url>
   <issueManagement>

diff --git a/Auto-Tune/pom.xml b/Auto-Tune/pom.xml
@@ -9,7 +9,7 @@
   <!-- Project information -->
   <groupId>unprotesting.com.github</groupId>
   <artifactId>Auto-Tune</artifactId>
-  <version>0.12.0-pre-release-5</version>
+  <version>0.12.0-pre-release-6</version>
  <!-- Info -->
   <name>Auto-Tune</name>
   <url>https://github.com/Unprotesting/Auto-Tune</url>

diff --git a/Auto-Tune/src/unprotesting/com/github/Main.java b/Auto-Tune/src/unprotesting/com/github/Main.java
@@ -275,7 +275,7 @@ public void onEnable() {
     debugLog("Loaded " + enchMap.get("Auto-Tune").size() + " enchantments");
     AutoTuneBuyCommand.shopTypes.add("enchantments");
     loadItemPriceData();
-    scheduler.scheduleAsyncRepeatingTask(this, new EnchantmentPriceHandler(), 1200*Config.getTimePeriod(), (Config.getTimePeriod()*1200));
+    scheduler.scheduleAsyncRepeatingTask(this, new EnchantmentPriceHandler(), 1200*Config.getTimePeriod(), (Config.getTimePeriod()*3600));
   }
 
   private boolean setupEconomy() {

diff --git a/Auto-Tune/src/unprotesting/com/github/util/EnchantmentPriceHandler.java b/Auto-Tune/src/unprotesting/com/github/util/EnchantmentPriceHandler.java
@@ -2,6 +2,7 @@
 
 import java.util.concurrent.ConcurrentHashMap;
 
+import org.bukkit.Bukkit;
 import org.json.simple.parser.ParseException;
 
 import unprotesting.com.github.Main;
@@ -11,21 +12,23 @@ public class EnchantmentPriceHandler implements Runnable {
     @Override
     public void run() {
         Main.debugLog("Loading Enchantment Price Update Algorithm");
-        for (String str : Main.enchMap.get("Auto-Tune").keySet()) {
-            ConcurrentHashMap<String, EnchantmentSetting> inputMap = Main.enchMap.get("Auto-Tune");
-            EnchantmentSetting setting = inputMap.get(str);
-            ConcurrentHashMap<Integer, Double[]> buySellMap = setting.buySellData;
-            try {
-                buySellMap = EnchantmentAlgorithm.loadAverageBuyAndSellValue(buySellMap, setting);
-            } catch (ParseException e) {
-                e.printStackTrace();
+        Integer playerCount = Bukkit.getServer().getOnlinePlayers().size();
+        if (Config.isUpdatePricesWhenInactive() || (!Config.isUpdatePricesWhenInactive() && playerCount > 0)){
+            for (String str : Main.enchMap.get("Auto-Tune").keySet()) {
+                ConcurrentHashMap<String, EnchantmentSetting> inputMap = Main.enchMap.get("Auto-Tune");
+                EnchantmentSetting setting = inputMap.get(str);
+                ConcurrentHashMap<Integer, Double[]> buySellMap = setting.buySellData;
+                try {
+                    buySellMap = EnchantmentAlgorithm.loadAverageBuyAndSellValue(buySellMap, setting);
+                } catch (ParseException e) {
+                    e.printStackTrace();
+                }
+                setting.buySellData = buySellMap;
+                Double[] arr = buySellMap.get(buySellMap.size()-1);
+                setting.price = arr[0];
+                inputMap.put(setting.name, setting);
+                Main.enchMap.put("Auto-Tune", inputMap);
             }
-            setting.buySellData = buySellMap;
-            Double[] arr = buySellMap.get(buySellMap.size()-1);
-            setting.price = arr[0];
-            inputMap.put(setting.name, setting);
-            Main.enchMap.put("Auto-Tune", inputMap);
         }
-    }
-
+    } 
 }