Catch thrown errors by API-Users

Author: TBlueF

src/main/java/de/bluecolored/bluemap/api/BlueMapAPI.java

@@ -28,13 +28,13 @@
 import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Collection;
+import java.util.List;
 import java.util.Optional;
 import java.util.UUID;
+import java.util.concurrent.ExecutionException;
 
 import de.bluecolored.bluemap.api.marker.Marker;
 import de.bluecolored.bluemap.api.marker.MarkerAPI;
-import de.bluecolored.bluemap.api.renderer.BlueMapMap;
-import de.bluecolored.bluemap.api.renderer.BlueMapWorld;
 import de.bluecolored.bluemap.api.renderer.RenderAPI;
 
 /**
@@ -140,13 +140,26 @@ public static synchronized Optional<BlueMapAPI> getInstance() {
 	 * Used by BlueMap to register the API and call the listeners properly. 
 	 * @param instance {@link BlueMapAPI}-instance
 	 */
-	protected static synchronized boolean registerInstance(BlueMapAPI instance) {
+	protected static synchronized boolean registerInstance(BlueMapAPI instance) throws ExecutionException {
 		if (BlueMapAPI.instance != null) return false;
 
 		BlueMapAPI.instance = instance;
 
+		List<Exception> thrownExceptions = new ArrayList<>(0);
 		for (BlueMapAPIListener listener : BlueMapAPI.listener) {
-			listener.onEnable(BlueMapAPI.instance);
+			try {
+				listener.onEnable(BlueMapAPI.instance);
+			} catch (Exception ex) {
+				thrownExceptions.add(ex);
+			}
+		}
+		
+		if (!thrownExceptions.isEmpty()) {
+			ExecutionException ex = new ExecutionException(thrownExceptions.get(0));
+			for (int i = 1; i < thrownExceptions.size(); i++) {
+				ex.addSuppressed(thrownExceptions.get(i));
+			}
+			throw ex;
 		}
 
 		return true;
@@ -155,14 +168,27 @@ protected static synchronized boolean registerInstance(BlueMapAPI instance) {
 	/**
 	 * Used by BlueMap to unregister the API and call the listeners properly.
 	 */
-	protected static synchronized boolean unregisterInstance(BlueMapAPI instance) {
+	protected static synchronized boolean unregisterInstance(BlueMapAPI instance) throws ExecutionException {
 		if (BlueMapAPI.instance != instance) return false;
-		
+
+		List<Exception> thrownExceptions = new ArrayList<>(0);
 		for (BlueMapAPIListener listener : BlueMapAPI.listener) {
-			listener.onDisable(BlueMapAPI.instance);
+			try {
+				listener.onDisable(BlueMapAPI.instance);	
+			} catch (Exception ex) {
+				thrownExceptions.add(ex);
+			}
 		}
 
 		BlueMapAPI.instance = null;
+
+		if (!thrownExceptions.isEmpty()) {
+			ExecutionException ex = new ExecutionException(thrownExceptions.get(0));
+			for (int i = 1; i < thrownExceptions.size(); i++) {
+				ex.addSuppressed(thrownExceptions.get(i));
+			}
+			throw ex;
+		}
 
 		return true;
 	}

…red/bluemap/api/renderer/BlueMapMap.java …/bluecolored/bluemap/api/BlueMapMap.javasrc/main/java/de/bluecolored/bluemap/api/renderer/BlueMapMap.java renamed to src/main/java/de/bluecolored/bluemap/api/BlueMapMap.java src/main/java/de/bluecolored/bluemap/api/renderer/BlueMapMap.java renamed to src/main/java/de/bluecolored/bluemap/api/BlueMapMap.java

@@ -22,7 +22,7 @@
  * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
  * THE SOFTWARE.
  */
-package de.bluecolored.bluemap.api.renderer;
+package de.bluecolored.bluemap.api;
 
 import com.flowpowered.math.vector.Vector2i;
 import com.flowpowered.math.vector.Vector3d;

…d/bluemap/api/renderer/BlueMapWorld.java …luecolored/bluemap/api/BlueMapWorld.javasrc/main/java/de/bluecolored/bluemap/api/renderer/BlueMapWorld.java renamed to src/main/java/de/bluecolored/bluemap/api/BlueMapWorld.java src/main/java/de/bluecolored/bluemap/api/renderer/BlueMapWorld.java renamed to src/main/java/de/bluecolored/bluemap/api/BlueMapWorld.java

@@ -22,7 +22,7 @@
  * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
  * THE SOFTWARE.
  */
-package de.bluecolored.bluemap.api.renderer;
+package de.bluecolored.bluemap.api;
 
 import java.nio.file.Path;
 import java.util.Collection;

src/main/java/de/bluecolored/bluemap/api/marker/Marker.java

@@ -28,7 +28,7 @@
 
 import com.flowpowered.math.vector.Vector3d;
 
-import de.bluecolored.bluemap.api.renderer.BlueMapMap;
+import de.bluecolored.bluemap.api.BlueMapMap;
 
 /**
  * A marker that is displayed on one of the maps in the web-app.

src/main/java/de/bluecolored/bluemap/api/marker/MarkerSet.java

@@ -29,7 +29,7 @@
 
 import com.flowpowered.math.vector.Vector3d;
 
-import de.bluecolored.bluemap.api.renderer.BlueMapMap;
+import de.bluecolored.bluemap.api.BlueMapMap;
 
 /**
  * A set of {@link Marker}s that are displayed on the maps in the web-app.

src/main/java/de/bluecolored/bluemap/api/renderer/RenderAPI.java

@@ -29,6 +29,9 @@
 import com.flowpowered.math.vector.Vector2i;
 import com.flowpowered.math.vector.Vector3i;
 
+import de.bluecolored.bluemap.api.BlueMapMap;
+import de.bluecolored.bluemap.api.BlueMapWorld;
+
 /**
  * The {@link RenderAPI} is used to schedule tile-renders and process them on a number of different threads.
  */