Add error checking

Peter de Vroom · Peter de Vroom · commit 3abafc0a7188 · 2025-07-07T14:07:24.000+10:00
diff --git a/Flow.Launcher.Core/Plugin/PluginManager.cs b/Flow.Launcher.Core/Plugin/PluginManager.cs
@@ -550,9 +550,12 @@ private static string GetContainingFolderPathAfterUnzip(string unzippedParentFol
         private static bool SameOrLesserPluginVersionExists(string metadataPath)
         {
             var newMetadata = JsonSerializer.Deserialize<PluginMetadata>(File.ReadAllText(metadataPath));
-            var newVersion = new Version(newMetadata.Version);
+            if (!Version.TryParse(newMetadata.Version, out var newVersion))
+                throw new InvalidOperationException($"A plugin with the same ID and version already exists, or the version is greater than this downloaded plugin {plugin.Name}");
+
             return AllPlugins.Any(x => x.Metadata.ID == newMetadata.ID
-                          && newVersion <= (new Version(x.Metadata.Version)));
+                                       && Version.TryParse(x.Metadata.Version, out var version)
+                                       && newVersion <= version);
         }
 
         #region Public functions

Original file line number	Diff line number	Diff line change
`@@ -550,9 +550,12 @@ private static string GetContainingFolderPathAfterUnzip(string unzippedParentFol`
`550`	`550`	`private static bool SameOrLesserPluginVersionExists(string metadataPath)`
`551`	`551`	`{`
`552`	`552`	`var newMetadata = JsonSerializer.Deserialize<PluginMetadata>(File.ReadAllText(metadataPath));`
`553`		`- var newVersion = new Version(newMetadata.Version);`
	`553`	`+ if (!Version.TryParse(newMetadata.Version, out var newVersion))`
	`554`	`+ throw new InvalidOperationException($"A plugin with the same ID and version already exists, or the version is greater than this downloaded plugin {plugin.Name}");`
	`555`	`+`
`554`	`556`	`return AllPlugins.Any(x => x.Metadata.ID == newMetadata.ID`
`555`		`- && newVersion <= (new Version(x.Metadata.Version)));`
	`557`	`+ && Version.TryParse(x.Metadata.Version, out var version)`
	`558`	`+ && newVersion <= version);`
`556`	`559`	`}`
`557`	`560`
`558`	`561`	`#region Public functions`