From f6a63582a26336b7ec285e1e9acb128ba494b715 Mon Sep 17 00:00:00 2001
From: Stefan Markovic <57057282+stefansjfw@users.noreply.github.com>
Date: Fri, 5 Jan 2024 20:46:34 +0100
Subject: [PATCH] [CmdNotFound]Disable on Arm64 as there is no MSI installer
 for PowerShell 7.4 yet  (#30759)

* [Command Not Found] Disable on Arm64

* Hide deps

* Update src/settings-ui/Settings.UI/Strings/en-us/Resources.resw

Co-authored-by: Jaime Bernardo <jaime@janeasystems.com>

* Update Resources.resw

---------

Co-authored-by: Jaime Bernardo <jaime@janeasystems.com>
Co-authored-by: Clint Rutkas <clint@rutkas.com>
---
 .../SettingsXAML/Views/CmdNotFoundPage.xaml   | 45 ++++++++++++-------
 .../Settings.UI/Strings/en-us/Resources.resw  | 25 ++++++-----
 .../ViewModels/CmdNotFoundViewModel.cs        |  6 +++
 3 files changed, 48 insertions(+), 28 deletions(-)

diff --git a/src/settings-ui/Settings.UI/SettingsXAML/Views/CmdNotFoundPage.xaml b/src/settings-ui/Settings.UI/SettingsXAML/Views/CmdNotFoundPage.xaml
index 43f064ce7d3..2832874f7be 100644
--- a/src/settings-ui/Settings.UI/SettingsXAML/Views/CmdNotFoundPage.xaml
+++ b/src/settings-ui/Settings.UI/SettingsXAML/Views/CmdNotFoundPage.xaml
@@ -17,7 +17,10 @@
             TrueValue="Collapsed" />
         <toolkitConverters:BoolToVisibilityConverter x:Key="BoolToVisibilityConverter" />
     </Page.Resources>
-    <custom:SettingsPageControl x:Uid="CmdNotFound" ModuleImageSource="ms-appx:///Assets/Settings/Modules/CmdNotFound.png">
+    <custom:SettingsPageControl
+        x:Uid="CmdNotFound"
+        IsEnabled="{x:Bind Mode=OneWay, Path=ViewModel.IsArm64Arch, Converter={StaticResource BoolNegationConverter}}"
+        ModuleImageSource="ms-appx:///Assets/Settings/Modules/CmdNotFound.png">
         <custom:SettingsPageControl.ModuleContent>
             <StackPanel ChildrenTransitions="{StaticResource SettingsCardsAnimations}" Orientation="Vertical">
                 <InfoBar
@@ -56,24 +59,32 @@
                         </controls:Case>
                     </controls:SwitchPresenter>
                     <controls:SettingsExpander.ItemsHeader>
-                        <InfoBar
-                            x:Uid="CmdNotFound_RequirementsBar"
-                            BorderThickness="0"
-                            CornerRadius="0"
-                            IsClosable="False"
-                            IsOpen="True">
-                            <InfoBar.ActionButton>
-                                <HyperlinkButton x:Uid="CmdNotFound_CheckCompatibility" Command="{x:Bind ViewModel.CheckRequirementsEventHandler}">
-                                    <ToolTipService.ToolTip>
-                                        <TextBlock x:Uid="CmdNotFound_CheckCompatibilityTooltip" TextWrapping="Wrap" />
-                                    </ToolTipService.ToolTip>
-                                </HyperlinkButton>
-                            </InfoBar.ActionButton>
-                        </InfoBar>
+                        <StackPanel>
+                            <InfoBar
+                                x:Uid="CmdNotFound_RequirementsBar"
+                                BorderThickness="0"
+                                CornerRadius="0"
+                                IsClosable="False"
+                                IsOpen="{x:Bind Mode=OneWay, Path=ViewModel.IsArm64Arch, Converter={StaticResource BoolNegationConverter}}">
+                                <InfoBar.ActionButton>
+                                    <HyperlinkButton x:Uid="CmdNotFound_CheckCompatibility" Command="{x:Bind ViewModel.CheckRequirementsEventHandler}">
+                                        <ToolTipService.ToolTip>
+                                            <TextBlock x:Uid="CmdNotFound_CheckCompatibilityTooltip" TextWrapping="Wrap" />
+                                        </ToolTipService.ToolTip>
+                                    </HyperlinkButton>
+                                </InfoBar.ActionButton>
+                            </InfoBar>
+                            <InfoBar
+                                x:Uid="CmdNotFound_Arm64ArchBar"
+                                BorderThickness="0"
+                                CornerRadius="0"
+                                IsClosable="False"
+                                IsOpen="{x:Bind Mode=OneWay, Path=ViewModel.IsArm64Arch}" />
+                        </StackPanel>
                     </controls:SettingsExpander.ItemsHeader>
                     <controls:SettingsExpander.Items>
 
-                        <controls:SettingsCard x:Uid="CmdNotFound_PowerShellDetection">
+                        <controls:SettingsCard x:Uid="CmdNotFound_PowerShellDetection" Visibility="{x:Bind Mode=OneWay, Path=ViewModel.IsArm64Arch, Converter={StaticResource BoolToInvertedVisibilityConverter}}">
                             <controls:SwitchPresenter TargetType="x:Boolean" Value="{x:Bind ViewModel.IsPowerShell7Detected, Mode=OneWay}">
                                 <controls:Case Value="True">
                                     <StackPanel Orientation="Horizontal" Spacing="8">
@@ -104,7 +115,7 @@
                             </controls:SwitchPresenter>
                         </controls:SettingsCard>
 
-                        <controls:SettingsCard x:Uid="CmdNotFound_WinGetClientDetection">
+                        <controls:SettingsCard x:Uid="CmdNotFound_WinGetClientDetection" Visibility="{x:Bind Mode=OneWay, Path=ViewModel.IsArm64Arch, Converter={StaticResource BoolToInvertedVisibilityConverter}}">
                             <controls:SwitchPresenter TargetType="x:Boolean" Value="{x:Bind ViewModel.IsWinGetClientModuleDetected, Mode=OneWay}">
                                 <controls:Case Value="True">
                                     <StackPanel Orientation="Horizontal" Spacing="8">
diff --git a/src/settings-ui/Settings.UI/Strings/en-us/Resources.resw b/src/settings-ui/Settings.UI/Strings/en-us/Resources.resw
index 73314466091..25fc9390fc4 100644
--- a/src/settings-ui/Settings.UI/Strings/en-us/Resources.resw
+++ b/src/settings-ui/Settings.UI/Strings/en-us/Resources.resw
@@ -3677,16 +3677,16 @@ Activate by holding the key for the character you want to add an accent to, then
     <value>Command Not Found</value>
     <comment>"Command Not Found" is a product name</comment>
   </data>
-	<data name="InstalledLabel.Text" xml:space="preserve">
+  <data name="InstalledLabel.Text" xml:space="preserve">
     <value>Installed</value>
   </data>
-	<data name="DetectedLabel.Text" xml:space="preserve">
+  <data name="DetectedLabel.Text" xml:space="preserve">
     <value>Detected</value>
   </data>
-	<data name="NotDetectedLabel.Text" xml:space="preserve">
+  <data name="NotDetectedLabel.Text" xml:space="preserve">
     <value>Not detected</value>
   </data>
-	<data name="CmdNotFound_RequirementsBar.Title" xml:space="preserve">
+  <data name="CmdNotFound_RequirementsBar.Title" xml:space="preserve">
     <value>The following components are required</value>
   </data>
   <data name="CmdNotFound_PowerShellDetection.Header" xml:space="preserve">
@@ -3695,11 +3695,11 @@ Activate by holding the key for the character you want to add an accent to, then
   <data name="CmdNotFound_WinGetClientDetection.Header" xml:space="preserve">
     <value>WinGet Client PowerShell module</value>
   </data>
-	<data name="CmdNotFound_Enable.Header" xml:space="preserve">
+  <data name="CmdNotFound_Enable.Header" xml:space="preserve">
     <value>Command Not Found</value>
     <comment>"Command Not Found" is a product name</comment>
   </data>
-	<data name="CmdNotFound_Enable.Description" xml:space="preserve">
+  <data name="CmdNotFound_Enable.Description" xml:space="preserve">
     <value>Add this module to the PowerShell 7 profile script so that it is enabled with every new session</value>
   </data>
   <data name="CmdNotFound_ModuleInstallationLogs.Text" xml:space="preserve">
@@ -3708,16 +3708,16 @@ Activate by holding the key for the character you want to add an accent to, then
   <data name="CmdNotFound_CheckPowerShellVersionButtonControl.Description" xml:space="preserve">
     <value>PowerShell 7 is required to use this module</value>
   </data>
-	<data name="CmdNotFound_CheckCompatibility.Content" xml:space="preserve">
+  <data name="CmdNotFound_CheckCompatibility.Content" xml:space="preserve">
     <value>Refresh</value>
   </data>
-	<data name="CmdNotFound_CheckCompatibilityTooltip.Text" xml:space="preserve">
+  <data name="CmdNotFound_CheckCompatibilityTooltip.Text" xml:space="preserve">
     <value>Check if your PowerShell configuration is compatible and configured correctly</value>
   </data>
-	<data name="CmdNotFound_InstallButton.Content" xml:space="preserve">
+  <data name="CmdNotFound_InstallButton.Content" xml:space="preserve">
     <value>Install</value>
   </data>
-	<data name="CmdNotFound_UninstallButton.Content" xml:space="preserve">
+  <data name="CmdNotFound_UninstallButton.Content" xml:space="preserve">
     <value>Uninstall</value>
   </data>
   <data name="Oobe_CmdNotFound.Description" xml:space="preserve">
@@ -3998,4 +3998,7 @@ Activate by holding the key for the character you want to add an accent to, then
   <data name="PowerLauncher_TextFontSizeSlider.[using:Microsoft.UI.Xaml.Automation]AutomationProperties.Name" xml:space="preserve">
     <value>Text size of result titles</value>
   </data>
-</root>
\ No newline at end of file
+  <data name="CmdNotFound_Arm64ArchBar.Title" xml:space="preserve">
+    <value>Command Not Found is not supported on the ARM64 architecture currently. We are actively working on a solution.</value>
+  </data>
+</root>
diff --git a/src/settings-ui/Settings.UI/ViewModels/CmdNotFoundViewModel.cs b/src/settings-ui/Settings.UI/ViewModels/CmdNotFoundViewModel.cs
index 712194b453e..85b5c3bf2e5 100644
--- a/src/settings-ui/Settings.UI/ViewModels/CmdNotFoundViewModel.cs
+++ b/src/settings-ui/Settings.UI/ViewModels/CmdNotFoundViewModel.cs
@@ -6,6 +6,7 @@
 using System.Diagnostics;
 using System.IO;
 using System.Reflection;
+using System.Runtime.InteropServices;
 using global::PowerToys.GPOWrapper;
 using ManagedCommon;
 using Microsoft.PowerToys.Settings.UI.Library.Helpers;
@@ -123,6 +124,11 @@ public bool IsEnabledGpoConfigured
             get => _enabledStateIsGPOConfigured;
         }
 
+        public bool IsArm64Arch
+        {
+            get => RuntimeInformation.OSArchitecture == System.Runtime.InteropServices.Architecture.Arm64;
+        }
+
         public string RunPowerShellScript(string powershellExecutable, string powershellArguments, bool hidePowerShellWindow = false)
         {
             string outputLog = string.Empty;