1.0.4

+ Fixed AutoSpawn systems not spawning
+ Fixed debug saying mechanic failed because I flipped the boolean

plugin-modules/Core/src/com/songoda/epicbosses/autospawns/AutoSpawn.java

@@ -37,16 +37,12 @@ public class AutoSpawn {
         return null;
     }
 
-    public boolean isLocked() {
-        return this.editing;
-    }
-
     public boolean isCompleteEnoughToSpawn() {
         if(this.type == null) return false;
 
-        List<String> entities = this.entities;
+        List<String> entities = getEntities();
 
-        if(this.entities == null || this.entities.isEmpty()) return false;
+        if(entities == null || entities.isEmpty()) return false;
 
         return true;
     }

plugin-modules/Core/src/com/songoda/epicbosses/commands/boss/BossTimeCmd.java

@@ -53,10 +53,19 @@ public class BossTimeCmd extends SubCommand {
         ActiveAutoSpawnHolder activeAutoSpawnHolder = this.autoSpawnManager.getActiveAutoSpawnHolder(section);
         ActiveIntervalAutoSpawnHolder activeIntervalAutoSpawnHolder = (ActiveIntervalAutoSpawnHolder) activeAutoSpawnHolder;
         long remainingMs = activeIntervalAutoSpawnHolder.getRemainingMs();
+
+        if(remainingMs == 0 && activeIntervalAutoSpawnHolder.isSpawnAfterLastBossIsKilled()) {
+            Message.Boss_Time_CurrentlyActive.msg(sender);
+            return;
+        }
+
         String s = Message.General_TimeLayout.toString();
-        int remainingHours = (int) TimeUnit.MILLISECONDS.to(TimeUnit.HOURS, remainingMs);
+        TimeUnit unit = TimeUnit.MILLISECONDS;
-        int remainingMins = (int) TimeUnit.MILLISECONDS.to(TimeUnit.MINUTES, remainingMs);
+        int remainingHours = (int) unit.to(TimeUnit.HOURS, remainingMs);
-        int remainingSecs = (int) TimeUnit.MILLISECONDS.to(TimeUnit.SECONDS, remainingMs);
+        remainingMs -= TimeUnit.HOURS.to(unit, remainingHours);
+        int remainingMins = (int) unit.to(TimeUnit.MINUTES, remainingMs);
+        remainingMs -= TimeUnit.MINUTES.to(unit, remainingMins);
+        int remainingSecs = (int) unit.to(TimeUnit.SECONDS, remainingMs);
 
         s = s.replace("{hours}", NumberUtils.get().formatDouble(remainingHours));
         s = s.replace("{mins}", NumberUtils.get().formatDouble(remainingMins));

plugin-modules/Core/src/com/songoda/epicbosses/holder/autospawn/ActiveIntervalAutoSpawnHolder.java

@@ -39,7 +39,7 @@ public class ActiveIntervalAutoSpawnHolder extends ActiveAutoSpawnHolder {
 
     @Override
     public boolean canSpawn() {
-        if(getAutoSpawn().isLocked()) return false;
+        if(getAutoSpawn().isEditing()) return false;
         if(!getAutoSpawn().getType().equalsIgnoreCase("INTERVAL")) return false;
 
         int currentActiveAmount = getCurrentActiveBossHolders();
@@ -47,19 +47,22 @@ public class ActiveIntervalAutoSpawnHolder extends ActiveAutoSpawnHolder {
 
         Location location = this.intervalSpawnElement.getSpawnLocation();
         boolean spawnIfChunkNotLoaded = ObjectUtils.getValue(getAutoSpawn().getAutoSpawnSettings().getSpawnWhenChunkIsntLoaded(), false);
-        boolean spawnAfterLastBossIsKilled = ObjectUtils.getValue(this.intervalSpawnElement.getSpawnAfterLastBossIsKilled(), false);
 
         if(location == null) return false;
         if(!spawnIfChunkNotLoaded && !location.getChunk().isLoaded()) return false;
-        if(spawnAfterLastBossIsKilled && !getActiveBossHolders().isEmpty()) return false;
+        if(isSpawnAfterLastBossIsKilled() && !getActiveBossHolders().isEmpty()) return false;
 
         return currentActiveAmount < maxAmount;
     }
 
+    public boolean isSpawnAfterLastBossIsKilled() {
+        return ObjectUtils.getValue(this.intervalSpawnElement.getSpawnAfterLastBossIsKilled(), false);
+    }
+
     public void restartInterval() {
         stopInterval();
 
-        if(getAutoSpawn().isLocked()) return;
+        if(getAutoSpawn().isEditing()) return;
 
         Integer delay = this.intervalSpawnElement.getSpawnRate();
 
@@ -68,28 +71,28 @@ public class ActiveIntervalAutoSpawnHolder extends ActiveAutoSpawnHolder {
             return;
         }
 
-        long delayMs = (long) TimeUnit.SECONDS.to(TimeUnit.MILLISECONDS, delay);
+        int delaySec = (int) TimeUnit.MINUTES.to(TimeUnit.SECONDS, delay);
-        boolean spawnAfterLastBossIsKilled = ObjectUtils.getValue(this.intervalSpawnElement.getSpawnAfterLastBossIsKilled(), false);
 
-        updateNextCompleteTime(delayMs);
+        this.intervalTask = ServerUtils.get().runTimer(delaySec*20, delaySec*20, () -> {
+            boolean canSpawn = canSpawn();
 
-        this.intervalTask = ServerUtils.get().runTimer(delayMs, delayMs, () -> {
+            if(!canSpawn) return;
-            if(!canSpawn()) return;
 
-            if(this.intervalSpawnElement.attemptSpawn(this) && spawnAfterLastBossIsKilled) {
+            this.intervalSpawnElement.attemptSpawn(this);
-                stopInterval();
+
+            if(isSpawnAfterLastBossIsKilled()) {
+                cancelCurrentInterval();
                 return;
             }
 
-            updateNextCompleteTime(delayMs);
+            updateNextCompleteTime();
         });
+
+        updateNextCompleteTime();
     }
 
     public void stopInterval() {
-        if(this.intervalTask != null) ServerUtils.get().cancelTask(this.intervalTask);
+        cancelCurrentInterval();
-
-        this.nextCompletedTime = 0;
-
         getActiveBossHolders().forEach(ActiveBossHolder::killAll);
         getActiveBossHolders().clear();
     }
@@ -112,12 +115,28 @@ public class ActiveIntervalAutoSpawnHolder extends ActiveAutoSpawnHolder {
 
                 if(spawnAfterLastBossIsKilled) {
                     restartInterval();
+                    updateNextCompleteTime();
                 }
             }
         };
     }
 
-    private void updateNextCompleteTime(long delayMs) {
+    private void cancelCurrentInterval() {
+        if(this.intervalTask != null) ServerUtils.get().cancelTask(this.intervalTask);
+
+        this.nextCompletedTime = 0;
+    }
+
+    private void updateNextCompleteTime() {
+        Integer delay = this.intervalSpawnElement.getSpawnRate();
+
+        if(delay == null) {
+            Debug.AUTOSPAWN_INTERVALNOTREAL.debug("null", BossAPI.getAutoSpawnName(getAutoSpawn()));
+            return;
+        }
+
+        long delayMs = (long) TimeUnit.MINUTES.to(TimeUnit.MILLISECONDS, delay);
+
         this.nextCompletedTime = System.currentTimeMillis() + delayMs;
     }
 }

plugin-modules/Core/src/com/songoda/epicbosses/managers/AutoSpawnManager.java

@@ -34,19 +34,17 @@ public class AutoSpawnManager {
     public void startIntervalSystems() {
         Map<String, AutoSpawn> autoSpawnMap = this.autoSpawnFileManager.getAutoSpawnMap();
 
-        autoSpawnMap.forEach((name, autoSpawn) -> {
+        if(!this.activeAutoSpawnHolders.isEmpty()) {
-            String autoSpawnType = autoSpawn.getType();
+            stopIntervalSystems();
-            SpawnType spawnType = SpawnType.getCurrent(autoSpawnType);
+        }
 
-            if(spawnType == SpawnType.INTERVAL) {
+        autoSpawnMap.forEach(this::addAndCreateActiveAutoSpawnHolder);
-                ActiveIntervalAutoSpawnHolder autoSpawnHolder = new ActiveIntervalAutoSpawnHolder(spawnType, autoSpawn);
+    }
 
-                if(!autoSpawn.isLocked()) {
+    public void stopIntervalSystems() {
-                    autoSpawnHolder.restartInterval();
+        Map<String, ActiveAutoSpawnHolder> autoSpawnHolderMap = new HashMap<>(this.activeAutoSpawnHolders);
-                }
+
-                this.activeAutoSpawnHolders.put(name, autoSpawnHolder);
+        autoSpawnHolderMap.forEach((name, autoSpawnHolder) -> removeActiveAutoSpawnHolder(name));
-            }
-        });
     }
 
     public List<String> getIntervalAutoSpawns() {
@@ -72,33 +70,23 @@ public class AutoSpawnManager {
         return false;
     }
 
-    public ActiveAutoSpawnHolder getAutoSpawnHolder(String name) {
-        if(!exists(name)) return null;
-
-        List<String> keyList = new ArrayList<>(this.activeAutoSpawnHolders.keySet());
-
-        for (String s : keyList) {
-            if(s.equalsIgnoreCase(name)) return this.activeAutoSpawnHolders.get(s);
-        }
-
-        return null;
-    }
-
-    public void addActiveAutoSpawnHolder(String name, ActiveAutoSpawnHolder autoSpawnHolder) {
-        if(this.activeAutoSpawnHolders.containsKey(name)) return;
-
-        this.activeAutoSpawnHolders.put(name, autoSpawnHolder);
-    }
-
-    public ActiveAutoSpawnHolder getActiveAutoSpawnHolder(AutoSpawn autoSpawn) {
-        return getActiveAutoSpawnHolder(BossAPI.getAutoSpawnName(autoSpawn));
-    }
-
     public ActiveAutoSpawnHolder getActiveAutoSpawnHolder(String name) {
         return this.activeAutoSpawnHolders.getOrDefault(name, null);
     }
 
-    public void removeActiveAutoSpawnHolder(String name) {
+    public void addAndCreateActiveAutoSpawnHolder(AutoSpawn autoSpawn) {
+        String name = BossAPI.getAutoSpawnName(autoSpawn);
+
+        addAndCreateActiveAutoSpawnHolder(name, autoSpawn);
+    }
+
+    public void removeActiveAutoSpawnHolder(AutoSpawn autoSpawn) {
+        String name = BossAPI.getAutoSpawnName(autoSpawn);
+
+        removeActiveAutoSpawnHolder(name);
+    }
+
+    private void removeActiveAutoSpawnHolder(String name) {
         ActiveAutoSpawnHolder autoSpawnHolder = this.activeAutoSpawnHolders.getOrDefault(name, null);
 
         if(autoSpawnHolder != null) {
@@ -107,28 +95,9 @@ public class AutoSpawnManager {
         }
     }
 
-    public void stopIntervalSystems() {
+    private void stopInterval(ActiveAutoSpawnHolder autoSpawnHolder) {
-        Map<String, ActiveAutoSpawnHolder> autoSpawnHolderMap = new HashMap<>(this.activeAutoSpawnHolders);
+        if (autoSpawnHolder != null) {
-
+            if (autoSpawnHolder.getSpawnType() == SpawnType.INTERVAL) {
-        autoSpawnHolderMap.forEach((name, autoSpawnHolder) -> removeActiveAutoSpawnHolder(name));
-    }
-
-    public void stopInterval(AutoSpawn autoSpawn) {
-        String name = BossAPI.getAutoSpawnName(autoSpawn);
-
-        stopInterval(name);
-    }
-
-
-    public void stopInterval(String name) {
-        ActiveAutoSpawnHolder autoSpawnHolder = this.activeAutoSpawnHolders.getOrDefault(name, null);
-
-        stopInterval(autoSpawnHolder);
-    }
-
-    public void stopInterval(ActiveAutoSpawnHolder autoSpawnHolder) {
-        if(autoSpawnHolder != null) {
-            if(autoSpawnHolder.getSpawnType() == SpawnType.INTERVAL) {
                 ActiveIntervalAutoSpawnHolder intervalAutoSpawnHolder = (ActiveIntervalAutoSpawnHolder) autoSpawnHolder;
 
                 intervalAutoSpawnHolder.stopInterval();
@@ -136,6 +105,20 @@ public class AutoSpawnManager {
         }
     }
 
+    private void addAndCreateActiveAutoSpawnHolder(String name, AutoSpawn autoSpawn) {
+        String autoSpawnType = autoSpawn.getType();
+        SpawnType spawnType = SpawnType.getCurrent(autoSpawnType);
+
+        if(spawnType == SpawnType.INTERVAL) {
+            ActiveIntervalAutoSpawnHolder autoSpawnHolder = new ActiveIntervalAutoSpawnHolder(spawnType, autoSpawn);
+
+            if(autoSpawn.isEditing()) return;
+
+            autoSpawnHolder.restartInterval();
+            this.activeAutoSpawnHolders.put(name, autoSpawnHolder);
+        }
+    }
+
     public static ICustomSettingAction createAutoSpawnAction(String name, String current, List<String> extraInformation, ItemStack displayStack, ClickAction clickAction) {
         return new CustomAutoSpawnActionCreator(name, current, extraInformation, displayStack, clickAction);
     }

plugin-modules/Core/src/com/songoda/epicbosses/managers/BossMechanicManager.java

@@ -62,7 +62,7 @@ public class BossMechanicManager implements IMechanicManager<BossEntity, ActiveB
 
                 ServerUtils.get().logDebug("Applying " + mechanic.getClass().getSimpleName());
 
-                if(!didMechanicApplicationFail(mechanic, bossEntity, activeBossHolder)) {
+                if(didMechanicApplicationFail(mechanic, bossEntity, activeBossHolder)) {
                     Debug.FAILED_TO_APPLY_MECHANIC.debug(mechanic.getClass().getSimpleName());
                 }
             }

plugin-modules/Core/src/com/songoda/epicbosses/panel/autospawns/MainAutoSpawnEditorPanel.java

@@ -99,17 +99,9 @@ public class MainAutoSpawnEditorPanel extends VariablePanelHandler<AutoSpawn> {
             player.closeInventory();
 
             if(autoSpawn.isEditing()) {
-                this.autoSpawnManager.stopInterval(autoSpawn);
+                this.autoSpawnManager.removeActiveAutoSpawnHolder(autoSpawn);
             } else {
-                ActiveAutoSpawnHolder autoSpawnHolder = this.autoSpawnManager.getActiveAutoSpawnHolder(autoSpawn);
+                this.autoSpawnManager.addAndCreateActiveAutoSpawnHolder(autoSpawn);
-
-                if(autoSpawnHolder != null) {
-                    if(autoSpawnHolder.getSpawnType() == SpawnType.INTERVAL) {
-                        ActiveIntervalAutoSpawnHolder intervalAutoSpawnHolder = (ActiveIntervalAutoSpawnHolder) autoSpawnHolder;
-
-                        intervalAutoSpawnHolder.restartInterval();
-                    }
-                }
             }
         };
     }

plugin-modules/Core/src/com/songoda/epicbosses/utils/Message.java

@@ -210,7 +210,8 @@ public enum Message {
 
     Boss_Time_NoPermission("&c&l(!) &cYou do not have access to this command."),
     Boss_Time_InvalidArgs("&c&l(!) &cYou must use &n/boss time [section]&c to check the time left on a boss spawn."),
-    Boss_Time_DoesntExist("&c&l(!) &cThe specified interval spawn system doesn't exist. Please use one of the following active system names: &f{0}&c."),
+    Boss_Time_DoesntExist("&c&l(!) &cThe specified interval spawn system doesn't exist or editing has been toggled on so the section isn't ticking at the moment. Please use one of the following active system names: &f{0}&c."),
+    Boss_Time_CurrentlyActive("&b&lEpicBosses &8» &7There is currently a boss spawned from this section so the countdown will not begin for the next to spawn until the last boss is killed."),
     Boss_Time_GetRemainingTime("&b&lEpicBosses &8» &7There is currently &f{0}&7 remaining on the &f{1}&7 interval spawn system.");
 
     private static FileConfiguration LANG;

pom.xml

@@ -19,7 +19,7 @@
     </modules>
 
     <properties>
-        <plugin.version>1.0.3-U3</plugin.version>
+        <plugin.version>1.0.4</plugin.version>
         <plugin.name>EpicBosses</plugin.name>
         <plugin.main>com.songoda.epicbosses.CustomBosses</plugin.main>
         <plugin.author>AMinecraftDev</plugin.author>