Merge pull request #13 from cs-jako/develop

Possible Fix for review

core/src/main/java/net/crazy/badges/core/AddonConfiguration.java

@@ -50,4 +50,8 @@ public boolean compact() {
   public boolean showOwn() {
     return this.showOwn.get();
   }
+
+  public ConfigProperty<Boolean> getCompactBadges() {
+    return this.compactBadges;
+  }
 }

core/src/main/java/net/crazy/badges/core/Badges.java

@@ -44,6 +44,21 @@ protected void enable() {
         configuration().size()
     ));
 
+    addon.configuration().getCompactBadges().addChangeListener((state) -> {
+      if (state) {
+        // Don't purge "cache" when enabling compact mode as this doesn't cause any issues
+        return;
+      }
+
+      /**
+       * Purge "Cache"
+       * Meaning: Clear the playerCache as due to the #removeIf inside BadgeTag the badges list get's updated
+       * This should not really be a performance bottleneck as it's just resets everything back to the beginning
+       * and when a player is rendered, their correct badges are loaded from the normal badges cache
+       */
+      playerCache.clear();
+    });
+
     this.logger().info("[Badges] Addon enabled.");
   }
 

core/src/main/java/net/crazy/badges/core/tags/BadgeTag.java

@@ -37,7 +37,7 @@ public void render(Stack stack, Entity livingEntity) {
       return;
     }
 
-    LinkedList<Badge> badges = new LinkedList<>(addon.playerCache.get(player.getUniqueId()));
+    LinkedList<Badge> badges = addon.playerCache.get(player.getUniqueId());
 
     if (badges.isEmpty()) {
       return;