Use also the release date to determine differences in versions

Signed-off-by: DeathsGun <deathsgun@protonmail.com>

Author: DeathsGun

src/main/kotlin/xyz/deathsgun/modmanager/api/mod/Version.kt

@@ -16,9 +16,12 @@
 
 package xyz.deathsgun.modmanager.api.mod
 
+import java.time.LocalDate
+
 data class Version(
     val version: String,
     val changelog: String,
+    val releaseDate: LocalDate,
     val type: VersionType,
     val gameVersions: List<String>,
     val assets: List<Asset>

src/main/kotlin/xyz/deathsgun/modmanager/providers/modrinth/Modrinth.kt

@@ -39,6 +39,8 @@ import java.net.http.HttpClient
 import java.net.http.HttpRequest
 import java.net.http.HttpResponse
 import java.time.Duration
+import java.time.Instant
+import java.time.ZoneOffset
 
 @OptIn(ExperimentalSerializationApi::class)
 class Modrinth : IModProvider, IModUpdateProvider {
@@ -211,6 +213,10 @@ class Modrinth : IModProvider, IModUpdateProvider {
                     Version(
                         modVersion.version,
                         modVersion.changelog,
+                        // 2021-09-03T10:56:59.402790Z
+                        Instant.parse(modVersion.releaseDate).atOffset(
+                            ZoneOffset.UTC
+                        ).toLocalDate(),
                         getVersionType(modVersion.type),
                         modVersion.gameVersions,
                         assets

src/main/kotlin/xyz/deathsgun/modmanager/providers/modrinth/models/ModrinthVersion.kt

@@ -24,6 +24,8 @@ data class ModrinthVersion(
     @SerialName("version_number")
     val version: String,
     val changelog: String,
+    @SerialName("date_published")
+    val releaseDate: String,
     @SerialName("version_type")
     val type: String,
     @SerialName("game_versions")

src/main/kotlin/xyz/deathsgun/modmanager/update/UpdateManager.kt

@@ -136,7 +136,7 @@ class UpdateManager {
         result = updateProvider.getVersionsForMod(mod.id)
         val versions = when (result) {
             is VersionResult.Error -> {
-                logger.error("Error while getting versions for mod {}", metadata.id)
+                logger.error("Error while getting versions for mod {}", metadata.id, result.cause)
                 ModManager.modManager.setModState(metadata.id, mod.id, ModState.INSTALLED)
                 return
             }
@@ -177,7 +177,7 @@ class UpdateManager {
         }
         val versions = when (val result = provider.getVersionsForMod(id)) {
             is VersionResult.Error -> {
-                logger.error("Error while getting versions for mod {}", metadata.id)
+                logger.error("Error while getting versions for mod {}", metadata.id, result.cause)
                 ModManager.modManager.setModState(metadata.id, id, ModState.INSTALLED)
                 return
             }
@@ -261,25 +261,38 @@ class UpdateManager {
     private fun findLatestCompatible(installedVersion: String, versions: List<Version>): Version? {
         var latest: Version? = null
         var latestVersion: SemanticVersion? = null
+        var installed: Version? = null
         val installVersion =
-            VersionDeserializer.deserializeSemantic(installedVersion.split("+")[0]) // Remove additional info from version
+            VersionDeserializer.deserializeSemantic(installedVersion)
         for (version in versions) {
+            if (version.version == installedVersion) {
+                installed = version
+            }
             if (!version.gameVersions.contains(ModManager.getMinecraftVersion()) ||
                 !ModManager.modManager.config.isReleaseAllowed(version.type)
             ) {
                 continue
             }
             val ver = try {
-                VersionDeserializer.deserializeSemantic(version.version.split("+")[0]) // Remove additional info from version
+                VersionDeserializer.deserializeSemantic(version.version) // Remove additional info from version
             } catch (e: Exception) {
+                if (latestVersion == null || version.releaseDate > latest?.releaseDate) {
+                    logger.info("Setting version {} via release date", version.version)
+                    latest = version
+                    latestVersion = null
+                    continue
+                }
                 logger.warn("Skipping error producing version {}", version.version)
                 continue
             }
-            if (latestVersion == null || ver > latestVersion) {
+            if (latestVersion == null || ver > latestVersion || version.releaseDate > latest?.releaseDate) {
                 latest = version
                 latestVersion = ver
             }
         }
+        if (installed != null && installed.releaseDate > latest?.releaseDate) {
+            return null
+        }
         if (latestVersion?.compareTo(installVersion) == 0) {
             return null
         }