make async metrics actually async

Author: Floweynt

src/main/java/de/sldk/mc/metrics/Metric.java

@@ -33,29 +33,29 @@ public CompletableFuture<Void> collect() {
         }
 
         if (isAsyncCapable()) {
-            CompletableFuture.runAsync(() -> {
+            return CompletableFuture.runAsync(() -> {
                 try {
                     doCollect();
                 } catch (Exception e) {
                     logException(e);
                 }
             });
-        }
-
-        CompletableFuture<Void> future = new CompletableFuture<>();
+        } else {
+            CompletableFuture<Void> future = new CompletableFuture<>();
 
-        // don't call .get() - this blocks the ForkJoinPool.commonPool and may deadlock the server in some cases
-        Bukkit.getScheduler().callSyncMethod(plugin, () -> {
-            try {
-                doCollect();
-            } catch (Exception e) {
-                logException(e);
-            }
-            future.complete(null);
-            return null;
-        });
+            // don't call .get() - this blocks the ForkJoinPool.commonPool and may deadlock the server in some cases
+            Bukkit.getScheduler().callSyncMethod(plugin, () -> {
+                try {
+                    doCollect();
+                } catch (Exception e) {
+                    logException(e);
+                }
+                future.complete(null);
+                return null;
+            });
 
-        return future;
+            return future;
+        }
     }
 
     protected abstract void doCollect();