From 404bc34e5fbaf542fa66b8970a8d6b1fc05ee8aa Mon Sep 17 00:00:00 2001
From: James Lyne <jim+github@not-null.co.uk>
Date: Fri, 1 Oct 2021 13:44:29 +0100
Subject: [PATCH] Always run LoadPlayerImages task in the thread pool. Prevents
 the main thread being blocked if the mysql connection is lost etc, when
 fetchskins is disabled.

---
 DynmapCore/src/main/java/org/dynmap/PlayerFaces.java | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/DynmapCore/src/main/java/org/dynmap/PlayerFaces.java b/DynmapCore/src/main/java/org/dynmap/PlayerFaces.java
index 83da181a..987bd20f 100644
--- a/DynmapCore/src/main/java/org/dynmap/PlayerFaces.java
+++ b/DynmapCore/src/main/java/org/dynmap/PlayerFaces.java
@@ -301,10 +301,7 @@ public class PlayerFaces {
             @Override
             public void playerEvent(DynmapPlayer p) {
                 Runnable job = new LoadPlayerImages(p.getName(), p.getSkinURL(), p.getUUID(), core.skinUrlProvider);
-                if(fetchskins)
-                    MapManager.scheduleDelayedJob(job, 0);
-                else
-                    job.run();
+                MapManager.scheduleDelayedJob(job, 0);
             }
         });
         storage = core.getDefaultMapStorage();