Skip to content

Commit

Permalink
Version check fix
Browse files Browse the repository at this point in the history
  • Loading branch information
erwinvanhunen committed Nov 22, 2024
1 parent eabe53e commit 16ecb6e
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/Commands/Base/GetDiagnostics.cs
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ void FillCurrentSite(Diagnostics result)
void FillNewerVersionAvailable(Diagnostics result)
{
var versionAvailable = VersionChecker.GetAvailableVersion();
if (versionAvailable != null && VersionChecker.IsNewer(versionAvailable.Version))
if (versionAvailable != null && VersionChecker.IsNewer(versionAvailable.SemanticVersion))
{
result.NewerVersionAvailable = versionAvailable.ToString();
}
Expand Down
13 changes: 11 additions & 2 deletions src/Commands/Utilities/VersionChecker.cs
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
using System.Net.Http;
using System.Net.Http.Headers;
using System.Reflection;
using System.Text.Json.Serialization;
using System.Xml;
using System.Xml.Linq;

Expand Down Expand Up @@ -73,7 +74,7 @@ public static void CheckVersion(PSCmdlet cmdlet)
if (onlineVersion != null)
{

if (IsNewer(onlineVersion.Version))
if (IsNewer(onlineVersion.SemanticVersion))
{
if (cmdlet != null)
{
Expand Down Expand Up @@ -229,7 +230,15 @@ public static PnPVersionResult GetAvailableVersion()

public class PnPVersionResult
{
public SemanticVersion Version { get; set; }
public string Version { get; set; }
public SemanticVersion SemanticVersion
{
get
{
SemanticVersion.TryParse(Version, out SemanticVersion result);
return result;
}
}
public string Message { get; set; }
}
}

0 comments on commit 16ecb6e

Please sign in to comment.