Fix class cast error for holo manager & added new methods to text holo

Author: maximjsx

build.gradle

@@ -4,7 +4,7 @@ plugins {
 }
 
 group = 'com.maximde'
-version = '1.2.4'
+version = '1.2.5'
 
 repositories {
     mavenCentral()

src/main/java/com/maximde/hologramapi/hologram/HologramManager.java

@@ -6,22 +6,22 @@
 import org.bukkit.plugin.Plugin;
 import org.bukkit.scheduler.BukkitRunnable;
 
-import java.util.HashMap;
-import java.util.List;
+import java.util.*;
+import java.util.concurrent.ConcurrentHashMap;
 
 @AllArgsConstructor
 public class HologramManager {
 
     private final Plugin plugin;
 
     @Getter
-    private final HashMap<TextHologram, BukkitRunnable> hologramAnimations = new HashMap<>();
+    private final Map<TextHologram, BukkitRunnable> hologramAnimations = new ConcurrentHashMap<>();
 
     @Getter
-    private final HashMap<String, TextHologram> hologramsMap = new HashMap<>();
+    private final Map<String, TextHologram> hologramsMap = new ConcurrentHashMap<>();
 
     public List<TextHologram> getHolograms() {
-        return (List<TextHologram>) this.hologramsMap.values();
+        return new ArrayList<>(this.hologramsMap.values());
     }
 
     public void spawn(TextHologram textHologram, Location location) {
@@ -38,9 +38,7 @@ public void remove(TextHologram textHologram) {
     }
 
     public void remove(String id) {
-        if(!hologramsMap.containsKey(id)) return;
-        this.hologramsMap.get(id).kill();
-        this.hologramsMap.remove(id);
+        Optional.ofNullable(this.hologramsMap.remove(id)).ifPresent(TextHologram::kill);
     }
 
     public void removeAll() {
@@ -54,21 +52,17 @@ public void applyAnimation(TextHologram hologram, TextAnimation textAnimation) {
     }
 
     public void cancelAnimation(TextHologram hologram) {
-        if(hologramAnimations.containsKey(hologram)) {
-            hologramAnimations.get(hologram).cancel();
-            hologramAnimations.remove(hologram);
-        }
+        Optional.ofNullable(hologramAnimations.remove(hologram)).ifPresent(BukkitRunnable::cancel);
     }
 
     private BukkitRunnable animateHologram(TextHologram hologram, TextAnimation textAnimation) {
         final BukkitRunnable animation = new BukkitRunnable() {
             int currentFrame = 0;
             public void run() {
-                if(textAnimation.getTextFrames().isEmpty()) return;
+                if (textAnimation.getTextFrames().isEmpty()) return;
                 hologram.setMiniMessageText(textAnimation.getTextFrames().get(currentFrame));
                 hologram.update();
-                currentFrame++;
-                if(currentFrame >= textAnimation.getTextFrames().size()) currentFrame = 0;
+                currentFrame = (currentFrame + 1) % textAnimation.getTextFrames().size();
             }
         };