Skip to content

Commit 076d7de

Browse files
authored
[DYN-9230] Bump dynamo major version check for PackageManager. (#16428)
1 parent adb8567 commit 076d7de

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/DynamoCoreWpf/ViewModels/PackageManager/PackageManagerClientViewModel.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -950,7 +950,7 @@ internal async void ExecutePackageDownload(string name, PackageVersion package,
950950
// also identify packages that have a dynamo engine version less than 3.x as a special case,
951951
// as Dynamo 3.x uses .net8 and older versions used .net framework - these packages may not be compatible.
952952
// This check will return empty if the current major version is not 3.
953-
var preDYN3Deps = newPackageHeaders.Where(dep => dynamoVersion.Major == 3 && Version.Parse(dep.engine_version).Major < dynamoVersion.Major);
953+
var preDYN4Deps = newPackageHeaders.Where(dep => dynamoVersion.Major == 4 && Version.Parse(dep.engine_version).Major < dynamoVersion.Major);
954954

955955
// If any of the required packages use a newer version of Dynamo, show a dialog to the user
956956
// allowing them to cancel the package download
@@ -971,7 +971,7 @@ internal async void ExecutePackageDownload(string name, PackageVersion package,
971971

972972
//if any of the required packages use a pre 3.x version of Dynamo, show a dialog to the user
973973
//allowing them to cancel the package download
974-
if (preDYN3Deps.Any())
974+
if (preDYN4Deps.Any())
975975
{
976976
var res = MessageBoxService.Show(ViewModelOwner,
977977
$"{string.Format(Resources.MessagePackageOlderDynamo, DynamoViewModel.BrandingResourceProvider.ProductName)} {Resources.MessagePackOlderDynamoLink}",

src/DynamoPackages/PackageLoader.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ private void TryLoadPackageIntoLibrary(Package package)
276276
Log($"Loaded Package {package.Name} {package.VersionName} from {package.RootDirectory}");
277277
try
278278
{
279-
if (dynamoVersion.Major == 3 && Version.Parse(package.EngineVersion).Major < 3)
279+
if (dynamoVersion.Major == 4 && Version.Parse(package.EngineVersion).Major < 3)
280280
{
281281
Log($@"{package.Name} {package.VersionName} has an engine version of {package.EngineVersion},
282282
it may not be compatible with this version of Dynamo due to .NET runtime changes. ");

0 commit comments

Comments
 (0)