From 174fa467e018f429d1fabbc738b71233df3c3db5 Mon Sep 17 00:00:00 2001 From: Alexander Holstrup <117829001+aholstrup1@users.noreply.github.com> Date: Fri, 23 Feb 2024 20:44:36 +0100 Subject: [PATCH] NextMajor fixes: Sorting is broken in Get-BcContainerAppInfo (#3367) In BCApps we use Get-BcContainerAppInfo to get a sorted list of apps so we can uninstall them in the right order. In the latest artifacts they are not sorted in the right order though See failures in: https://github.com/microsoft/BCApps/actions/runs/8015836387 or https://github.com/aholstrup1/BCApps/actions/runs/8023595403 With this change, the build passes again https://github.com/aholstrup1/BCApps/actions/runs/8023619694 --- AppHandling/Get-NavContainerAppInfo.ps1 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/AppHandling/Get-NavContainerAppInfo.ps1 b/AppHandling/Get-NavContainerAppInfo.ps1 index 34cc7f50d..dc95af768 100644 --- a/AppHandling/Get-NavContainerAppInfo.ps1 +++ b/AppHandling/Get-NavContainerAppInfo.ps1 @@ -90,7 +90,7 @@ try { function AddDependency { Param($dependency) #Write-Host "Add Dependency $($dependency.Name) $($dependency.Version)" - $dependentApp = $apps | Where-Object { $_.AppId -eq $dependency.AppId } + $dependentApp = $apps | Where-Object { "$($_.AppId)" -eq "$($dependency.AppId)" } if ($dependentApp) { @($dependentApp) | ForEach-Object { AddAnApp -AnApp $_ } }