Fix Forge project generation for MC1.12+

Author: marvin-roesch

src/main/kotlin/com/demonwav/mcdev/creator/ForgeProjectSettingsWizard.kt

@@ -201,6 +201,7 @@ class ForgeProjectSettingsWizard(private val creator: MinecraftProjectCreator) :
         if (fullVersion != null) {
             settings!!.forgeVersion = fullVersion
         }
+        settings!!.mcVersion = minecraftVersionBox.selectedItem as String
     }
 
     fun error() {

src/main/kotlin/com/demonwav/mcdev/platform/forge/ForgeProjectConfiguration.kt

@@ -27,6 +27,7 @@ open class ForgeProjectConfiguration : ProjectConfiguration() {
 
     var mcpVersion: String = ""
     var forgeVersion: String = ""
+    var mcVersion: String = ""
 
     init {
         type = PlatformType.FORGE

src/main/kotlin/com/demonwav/mcdev/platform/forge/ForgeTemplate.kt

@@ -13,11 +13,13 @@ package com.demonwav.mcdev.platform.forge
 import com.demonwav.mcdev.platform.BaseTemplate
 import com.demonwav.mcdev.platform.hybrid.SpongeForgeProjectConfiguration
 import com.demonwav.mcdev.util.MinecraftFileTemplateGroupFactory
+import com.demonwav.mcdev.util.SemanticVersion
 import com.intellij.openapi.project.Project
 import com.intellij.openapi.vfs.VirtualFile
 import java.util.Properties
 
 object ForgeTemplate {
+    private val MC_1_12 = SemanticVersion("1.12")
 
     fun applyBuildGradleTemplate(project: Project,
                                  file: VirtualFile,
@@ -32,6 +34,13 @@ object ForgeTemplate {
         if (configuration is SpongeForgeProjectConfiguration) {
             properties.setProperty("SPONGE_FORGE", "true")
         }
+        // Fixes builds for MC1.12+, requires FG 2.3
+        val mcVersion = SemanticVersion(configuration.mcVersion)
+        if (mcVersion >= MC_1_12) {
+            properties.setProperty("FORGEGRADLE_VERSION", "2.3")
+        } else {
+            properties.setProperty("FORGEGRADLE_VERSION", "2.2")
+        }
 
         BaseTemplate.applyTemplate(project, file, MinecraftFileTemplateGroupFactory.FORGE_BUILD_GRADLE_TEMPLATE, properties)
 

src/main/kotlin/com/demonwav/mcdev/platform/forge/version/ForgeVersion.kt

@@ -11,6 +11,7 @@
 package com.demonwav.mcdev.platform.forge.version
 
 import com.demonwav.mcdev.util.gson
+import com.demonwav.mcdev.util.SemanticVersion
 import com.demonwav.mcdev.util.sortVersions
 import java.io.IOException
 import java.net.URL
@@ -23,16 +24,16 @@ class ForgeVersion private constructor(private val map: Map<*, *>) {
     }
 
     fun getRecommended(versions: List<String>): String {
-        var recommended = "1.7"
+        var recommended = SemanticVersion("1.7")
         for (version in versions) {
             getPromo(version) ?: continue
-
-            if (recommended < version) {
-                recommended = version
+            val semantic = SemanticVersion(version)
+            if (recommended < semantic) {
+                recommended = semantic
             }
         }
 
-        return recommended
+        return recommended.value
     }
 
     fun getPromo(version: String): Double? {

src/main/kotlin/com/demonwav/mcdev/util/SemanticVersion.kt

@@ -0,0 +1,63 @@
+/*
+ * Minecraft Dev for IntelliJ
+ *
+ * https://minecraftdev.org
+ *
+ * Copyright (c) 2017 minecraft-dev
+ *
+ * MIT License
+ */
+
+package com.demonwav.mcdev.util
+
+import com.demonwav.mcdev.util.SemanticVersion.Companion.VersionPart.PreReleasePart
+import com.demonwav.mcdev.util.SemanticVersion.Companion.VersionPart.ReleasePart
+
+class SemanticVersion(val value: String) : Comparable<SemanticVersion> {
+    val parts = value.split(".").map {
+        if (it.contains("_pre")) {
+            val subParts = it.split("_pre")
+            PreReleasePart(subParts[0].toInt(), subParts[1].toInt())
+        } else {
+            ReleasePart(it.toInt())
+        }
+    }
+
+    override fun compareTo(other: SemanticVersion): Int {
+        val result = parts.zip(other.parts).fold(0, { acc, (a, b) -> if (acc == -1) acc else a.compareTo(b) })
+        if (parts.size != other.parts.size && result == 0) {
+            return parts.size - other.parts.size
+        }
+        return result
+    }
+
+    override fun equals(other: Any?) =
+        when (other) {
+            is SemanticVersion -> parts.size == other.parts.size && parts.zip(other.parts).all { (a, b) -> a == b }
+            else -> false
+        }
+
+    override fun hashCode(): Int {
+        return parts.hashCode()
+    }
+
+    companion object {
+        sealed class VersionPart : Comparable<VersionPart> {
+            data class ReleasePart(val version: Int) : VersionPart() {
+                override fun compareTo(other: VersionPart) =
+                    when (other) {
+                        is PreReleasePart -> if (version != other.version) version - other.version else 1
+                        is ReleasePart -> version - other.version
+                    }
+            }
+
+            data class PreReleasePart(val version: Int, val pre: Int) : VersionPart() {
+                override fun compareTo(other: VersionPart) =
+                    when (other) {
+                        is PreReleasePart -> if (version != other.version) version - other.version else pre - other.pre
+                        is ReleasePart -> if (version != other.version) version - other.version else -1
+                    }
+            }
+        }
+    }
+}

src/main/resources/fileTemplates/j2ee/forge_build.gradle.ft

@@ -12,7 +12,7 @@ buildscript {
 #end
     }
     dependencies {
-        classpath "net.minecraftforge.gradle:ForgeGradle:2.2-SNAPSHOT"
+        classpath "net.minecraftforge.gradle:ForgeGradle:${FORGEGRADLE_VERSION}-SNAPSHOT"
 #if (${SPONGE_FORGE})
         classpath "gradle.plugin.org.spongepowered:spongegradle:0.8.1"
 #end