DYN-6547: toggle depreciate revised (#15383)

dnenov · web-flow · commit aba9359a4656 · 2024-07-11T10:23:54.000-04:00
diff --git a/src/DynamoCoreWpf/ViewModels/PackageManager/PackageViewModel.cs b/src/DynamoCoreWpf/ViewModels/PackageManager/PackageViewModel.cs
@@ -22,6 +22,7 @@ public class PackageViewModel : NotificationObject
         private readonly DynamoViewModel dynamoViewModel;
         private readonly PackageManagerClient packageManagerClient;
         private readonly DynamoModel dynamoModel;
+        private readonly PackageManagerClientViewModel packageManagerClientViewModel;
 
         public Package Model { get; private set; }
 
@@ -172,6 +173,7 @@ public PackageViewModel(DynamoViewModel dynamoViewModel, Package model)
         {
             this.dynamoViewModel = dynamoViewModel;
             this.dynamoModel = dynamoViewModel.Model;
+            this.packageManagerClientViewModel = dynamoViewModel.PackageManagerClientViewModel;
 
             var pmExtension = dynamoModel.GetPackageManagerExtension();
             this.packageManagerClient = pmExtension.PackageManagerClient;
@@ -410,10 +412,9 @@ private bool IsOwner()
 
         private bool CanDeprecate()
         {
-            var packageInfo = new PackageInfo(Model.Name, Version.Parse(Model.VersionName));
-            var packageHeader = this.packageManagerClient.GetPackageHeader(packageInfo);
+            var isDeprecated = IsPackageDeprecated(Model.Name);
 
-            return IsOwner() && !packageHeader.deprecated;
+            return IsOwner() && !isDeprecated;
         }
 
         private void Undeprecate()
@@ -427,10 +428,9 @@ private void Undeprecate()
 
         private bool CanUndeprecate()
         {
-            var packageInfo = new PackageInfo(Model.Name, Version.Parse(Model.VersionName));
-            var packageHeader = this.packageManagerClient.GetPackageHeader(packageInfo);
+            var isDeprecated = IsPackageDeprecated(Model.Name);
 
-            return IsOwner() && packageHeader.deprecated;
+            return IsOwner() && isDeprecated;
         }
 
         private void PublishNewPackageVersion()
@@ -465,5 +465,19 @@ private void ExecuteWithTou(Action acceptanceCallback)
 
             termsOfUseCheck.Execute(false);
         }
+
+        private PackageManagerSearchElement GetPackageVersionInformationFromCached(string packageName)
+        {
+            var package = this.packageManagerClientViewModel.CachedPackageList.FirstOrDefault(x => x.Name == packageName);
+
+            return package;
+        }
+
+        private bool IsPackageDeprecated(string packageName)
+        {
+            var package = GetPackageVersionInformationFromCached(packageName);
+
+            return package != null ? package.IsDeprecated : false;
+        }
     }
 }
diff --git a/src/DynamoPackages/PackageManagerClient.cs b/src/DynamoPackages/PackageManagerClient.cs
@@ -123,11 +123,11 @@ void CleanPackagesWithWrongVersions(List<PackageHeader> packages)
         }
 
         /// <summary>
-        /// Get the package Header which contains all the relevant information to the Package
+        /// Gets maintainers for a specific package
         /// </summary>
         /// <param name="packageInfo"></param>
         /// <returns></returns>
-        internal PackageHeader GetPackageHeader(IPackageInfo packageInfo)
+        internal PackageHeader GetPackageMaintainers(IPackageInfo packageInfo)
         {
             var header = FailFunc.TryExecute(() =>
             {
@@ -314,7 +314,7 @@ internal bool DoesCurrentUserOwnPackage(Package package,string username)
                 return value;
             }
             var pkg = new PackageInfo(package.Name, new Version(package.VersionName));
-            var mnt = GetPackageHeader(pkg);
+            var mnt = GetPackageMaintainers(pkg);
             value = (mnt != null) && (mnt.maintainers.Any(maintainer => maintainer.username.Equals(username)));
             this.packageMaintainers[package.Name] = value;
             return value;

Original file line number	Diff line number	Diff line change
`@@ -123,11 +123,11 @@ void CleanPackagesWithWrongVersions(List<PackageHeader> packages)`
`123`	`123`	`}`
`124`	`124`
`125`	`125`	`/// <summary>`
`126`		`- /// Get the package Header which contains all the relevant information to the Package`
	`126`	`+ /// Gets maintainers for a specific package`
`127`	`127`	`/// </summary>`
`128`	`128`	`/// <param name="packageInfo"></param>`
`129`	`129`	`/// <returns></returns>`
`130`		`- internal PackageHeader GetPackageHeader(IPackageInfo packageInfo)`
	`130`	`+ internal PackageHeader GetPackageMaintainers(IPackageInfo packageInfo)`
`131`	`131`	`{`
`132`	`132`	`var header = FailFunc.TryExecute(() =>`
`133`	`133`	`{`
`@@ -314,7 +314,7 @@ internal bool DoesCurrentUserOwnPackage(Package package,string username)`
`314`	`314`	`return value;`
`315`	`315`	`}`
`316`	`316`	`var pkg = new PackageInfo(package.Name, new Version(package.VersionName));`
`317`		`- var mnt = GetPackageHeader(pkg);`
	`317`	`+ var mnt = GetPackageMaintainers(pkg);`
`318`	`318`	`value = (mnt != null) && (mnt.maintainers.Any(maintainer => maintainer.username.Equals(username)));`
`319`	`319`	`this.packageMaintainers[package.Name] = value;`
`320`	`320`	`return value;`