From 86c5ed52080f0ae39c66e2c5365389c7ec7ab083 Mon Sep 17 00:00:00 2001
From: Petr Onderka <gsvick@gmail.com>
Date: Sat, 12 Oct 2019 09:40:55 +0200
Subject: [PATCH] Use NuGet.Versioning.SemanticVersion to correctly compare
 package versions

---
 Data/PackageVersions.cs | 46 ++++++++---------------------------------
 FuGetGallery.csproj     |  1 +
 2 files changed, 10 insertions(+), 37 deletions(-)

diff --git a/Data/PackageVersions.cs b/Data/PackageVersions.cs
index 39f03e6..b42d89f 100644
--- a/Data/PackageVersions.cs
+++ b/Data/PackageVersions.cs
@@ -1,11 +1,11 @@
 using System;
 using System.Collections.Generic;
-using System.IO;
 using System.Linq;
-using System.Threading.Tasks;
 using System.Net.Http;
-using Newtonsoft.Json.Linq;
 using System.Threading;
+using System.Threading.Tasks;
+using Newtonsoft.Json.Linq;
+using NuGet.Versioning;
 
 namespace FuGetGallery
 {
@@ -120,11 +120,8 @@ public class PackageVersion : IComparable<PackageVersion>
     {
         string versionString = "";
 
-        public int Major { get; private set; }
-        public int Minor { get; private set; }
-        public int Patch { get; private set; }
-        public int Build { get; private set; }
-        public string Rest { get; private set; } = "";
+        public SemanticVersion SemanticVersion { get; private set; }
+
         public DateTime? PublishTime { get; set; }
 
         public bool IsPublished => PublishTime.HasValue && PublishTime.Value.Year > 1970;
@@ -135,38 +132,13 @@ public class PackageVersion : IComparable<PackageVersion>
                 if (versionString == value || string.IsNullOrEmpty (value))
                     return;
                 versionString = value;
-                var di = value.IndexOf ('-');
-                var vpart = di > 0 ? value.Substring (0, di) : value;
-                var rest = di > 0 ? value.Substring (di) : "";
-                var parts = vpart.Split ('.');
-                var maj = 0;
-                var min = 0;
-                var patch = 0;
-                var build = 0;
-                if (parts.Length > 0) int.TryParse(parts[0], out maj);
-                if (parts.Length > 1) int.TryParse(parts[1], out min);
-                if (parts.Length > 2) int.TryParse(parts[2], out patch);
-                if (parts.Length > 3) int.TryParse (parts[3], out build);
-                Major = maj;
-                Minor = min;
-                Patch = patch;
-                Build = build;
-                Rest = rest;
+
+                SemanticVersion.TryParse (value, out var semanticVersion);
+                SemanticVersion = semanticVersion;
             }
         }
 
-        public int CompareTo(PackageVersion other)
-        {
-            var c = Major.CompareTo(other.Major);
-            if (c != 0) return c;
-            c = Minor.CompareTo(other.Minor);
-            if (c != 0) return c;
-            c = Patch.CompareTo(other.Patch);
-            if (c != 0) return c;
-            c = Build.CompareTo (other.Build);
-            if (c != 0) return c;
-            return string.Compare(Rest, other.Rest, StringComparison.Ordinal);
-        }
+        public int CompareTo(PackageVersion other) => Comparer<SemanticVersion>.Default.Compare (SemanticVersion, other.SemanticVersion);
 
         public override bool Equals(object obj)
         {
diff --git a/FuGetGallery.csproj b/FuGetGallery.csproj
index 6453411..bf3986b 100644
--- a/FuGetGallery.csproj
+++ b/FuGetGallery.csproj
@@ -11,6 +11,7 @@
     </PackageReference>
     <PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="2.2.0" />
     <PackageReference Include="NGraphics" Version="0.6.0-beta1" />
+    <PackageReference Include="NuGet.Versioning" Version="5.3.0" />
     <PackageReference Include="sqlite-net-pcl" Version="1.6.258-beta" />
     <PackageReference Include="System.CodeDom" Version="4.5.0" />
     <PackageReference Include="System.Configuration.ConfigurationManager" Version="4.5.0" />