refactor: drop server-specific fields, use per-download post action

Author: bouclem

src/main/java/org/mcphackers/mcp/tools/versions/DownloadData.java

@@ -28,8 +28,6 @@ public class DownloadData {
 	public int totalSize;
 	protected List<Download> downloadQueue = new ArrayList<>();
 	protected AssetIndex assets;
-	private Path serverZip;
-	private Path serverJar;
 
 	public DownloadData(MCP mcp, Version version) {
 		this(mcp, MCPPaths.get(mcp, MCPPaths.LIB), MCPPaths.get(mcp, MCPPaths.JARS), MCPPaths.get(mcp, MCPPaths.JAR_ORIGINAL, Side.CLIENT), MCPPaths.get(mcp, MCPPaths.JAR_ORIGINAL, Side.SERVER), version);
@@ -43,12 +41,14 @@ public DownloadData(MCP mcp, Path libraries, Path gameDir, Path client, Path ser
 		Artifact serverArtifact = version.downloads.artifacts.get("server");
 		if (mcp.getOptions().getSide().includesServer() && serverArtifact != null) {
 			Path serverOut = server;
+			IORunnable onComplete = null;
 			if (serverArtifact.url.endsWith(".zip")) {
-				serverOut = server.getParent().resolve("minecraft_server.zip");
-				this.serverZip = serverOut;
-				this.serverJar = server;
+				Path serverZip = server.getParent().resolve("minecraft_server.zip");
+				Path serverJar = server;
+				serverOut = serverZip;
+				onComplete = () -> extractServerZip(serverZip, serverJar);
 			}
-			queueDownload(serverArtifact, serverOut);
+			queueDownload(serverArtifact, serverOut, onComplete);
 		}
 		for (DependDownload dependencyDownload : version.libraries) {
 			if (Rule.apply(dependencyDownload.rules)) {
@@ -126,11 +126,15 @@ public void setAssets(AssetIndex assets) {
 	}
 
 	public void queueDownload(IDownload dl, Path baseDir) {
+		queueDownload(dl, baseDir, null);
+	}
+
+	public void queueDownload(IDownload dl, Path baseDir, IORunnable onComplete) {
 		if (dl == null) {
 			return;
 		}
 		totalSize += dl.downloadSize();
-		downloadQueue.add(new Download(dl, baseDir));
+		downloadQueue.add(new Download(dl, baseDir, onComplete));
 	}
 
 	public void performDownload(DownloadListener listener) throws IOException {
@@ -146,12 +150,14 @@ public void performDownload(DownloadListener listener) throws IOException {
 				if (parent != null) Files.createDirectories(parent);
 				FileUtil.downloadFile(download.downloadURL(), file);
 			}
+			if (dl.onComplete != null) {
+				dl.onComplete.run();
+			}
 		}
-		extractServerZip();
 	}
 
-	private void extractServerZip() throws IOException {
-		if (serverZip == null || !Files.exists(serverZip)) {
+	private static void extractServerZip(Path serverZip, Path serverJar) throws IOException {
+		if (!Files.exists(serverZip)) {
 			return;
 		}
 		FileUtil.extractByExtension(serverZip, serverZip.getParent(), ".jar");
@@ -164,10 +170,17 @@ private void extractServerZip() throws IOException {
 	private static class Download {
 		IDownload download;
 		Path dir;
+		IORunnable onComplete;
 
-		public Download(IDownload dl, Path path) {
+		public Download(IDownload dl, Path path, IORunnable onComplete) {
 			download = dl;
 			dir = path;
+			this.onComplete = onComplete;
 		}
 	}
+
+	@FunctionalInterface
+	private interface IORunnable {
+		void run() throws IOException;
+	}
 }