ALSO-5603 Add parameter hiding autocomplete options from the UI (#13834)

spodo99 · web-flow · commit 1167f83d53a5 · 2023-03-29T09:41:35.000-07:00
* ALSO-5603 Add parameter hiding autocomplete options from the UI

* ALSO-5603 Use IPreferences for hide autocomplete option

This needs to avoid breaking compatibility

* ALSO-5603 Add hide autocomplete option to the settings

* ALSO-5603 Fix unit test
diff --git a/src/DynamoCore/Configuration/IPreferences.cs b/src/DynamoCore/Configuration/IPreferences.cs
@@ -182,6 +182,18 @@ internal interface IDisablePackageLoadingPreferences
         bool DisableCustomPackageLocations { get; set; }
     }
 
+    /// <summary>
+    /// Temporary interface to avoid breaking changes.
+    /// TODO: Merge with StartupConfiguration for 3.0 (DYN-1699)
+    /// </summary>
+    internal interface IHideAutocompleteMethodOptions
+    {
+        /// <summary>
+        /// If true, autocomplete method options are hidden from UI 
+        /// </summary>
+        bool HideAutocompleteMethodOptions { get; set; }
+    }
+
     /// <summary>
     /// Represents data about active state of preview background
     /// </summary>
diff --git a/src/DynamoCore/Configuration/PreferenceSettings.cs b/src/DynamoCore/Configuration/PreferenceSettings.cs
@@ -45,7 +45,7 @@ internal static void CopyProperties(this PreferenceSettings source, PreferenceSe
     /// from a XML file from DYNAMO_SETTINGS_FILE.
     /// When GUI is closed, the settings are saved back into the XML file.
     /// </summary>
-    public class PreferenceSettings : NotificationObject, IPreferences, IRenderPrecisionPreference, IDisablePackageLoadingPreferences, ILogSource
+    public class PreferenceSettings : NotificationObject, IPreferences, IRenderPrecisionPreference, IDisablePackageLoadingPreferences, ILogSource, IHideAutocompleteMethodOptions
     {
         private string numberFormat;
         private string lastUpdateDownloadPath;
@@ -552,6 +552,11 @@ public string PythonTemplateFilePath
         /// </summary>
         public int MLRecommendationNumberOfResults { get; set; }
 
+        /// <summary>
+        /// If true, autocomplete method options are hidden from UI 
+        /// </summary>
+        public bool HideAutocompleteMethodOptions { get; set; }
+
         /// <summary>
         /// This defines if user wants to see the enabled Dynamo Notification Center.
         /// </summary>
@@ -814,6 +819,7 @@ public PreferenceSettings()
             HideNodesBelowSpecificConfidenceLevel = false;
             MLRecommendationConfidenceLevel = 10;
             MLRecommendationNumberOfResults = 10;
+            HideAutocompleteMethodOptions = false;
             EnableNotificationCenter = true;
             isStaticSplashScreenEnabled = true;
             DefaultPythonEngine = string.Empty;
diff --git a/src/DynamoCoreWpf/ViewModels/Core/DynamoViewModel.cs b/src/DynamoCoreWpf/ViewModels/Core/DynamoViewModel.cs
@@ -749,8 +749,7 @@ protected DynamoViewModel(StartConfiguration startConfiguration)
             model.ComputeModelDeserialized += model_ComputeModelDeserialized;
             model.RequestNotification += model_RequestNotification;
 
-            preferencesViewModel = new PreferencesViewModel(this);
-
+            preferencesViewModel = new PreferencesViewModel(this);            
 
             if (!DynamoModel.IsTestMode && !DynamoModel.IsHeadless)
             {
diff --git a/src/DynamoCoreWpf/ViewModels/Menu/PreferencesViewModel.cs b/src/DynamoCoreWpf/ViewModels/Menu/PreferencesViewModel.cs
@@ -838,6 +838,17 @@ public bool PersistExtensionsIsChecked
 
         #region [ Node Autocomplete ]
 
+        /// <summary>
+        /// If true, autocomplete method options are hidden from UI 
+        /// </summary>
+        public bool HideAutocompleteMethodOptions
+        {
+            get
+            {
+                return preferenceSettings.HideAutocompleteMethodOptions;
+            }
+        }
+
         /// <summary>
         /// Controls the IsChecked property in the "Node autocomplete" toogle button
         /// </summary>
diff --git a/src/DynamoCoreWpf/Views/Menu/PreferencesView.xaml b/src/DynamoCoreWpf/Views/Menu/PreferencesView.xaml
@@ -745,11 +745,11 @@
                                                 <RowDefinition Height="*" />
                                                 <RowDefinition Height="*" />
                                             </Grid.RowDefinitions>
-                                            <Label Grid.Row="0"
+                                            <Label Grid.Row="0" Visibility="{Binding HideAutocompleteMethodOptions, Converter={StaticResource InverseBoolToVisibilityCollapsedConverter}}"
                                                 Margin="-8,15,10,3"                                               
                                                 Content="{x:Static p:Resources.PreferencesNodeAutocompleteMethod}" 
                                                 Foreground="{StaticResource PreferencesWindowFontColor}"/>
-                                            <Grid Grid.Row="1">
+                                            <Grid Grid.Row="1" Visibility="{Binding HideAutocompleteMethodOptions, Converter={StaticResource InverseBoolToVisibilityCollapsedConverter}}">
                                                 <Grid.ColumnDefinitions>
                                                     <ColumnDefinition x:Name="colObjectType" Width="Auto"/>
                                                     <ColumnDefinition x:Name="colMachineLearning" Width="Auto"/>
diff --git a/test/settings/DynamoSettings-NewSettings.xml b/test/settings/DynamoSettings-NewSettings.xml
@@ -69,6 +69,7 @@
   <HideNodesBelowSpecificConfidenceLevel>true</HideNodesBelowSpecificConfidenceLevel>
   <MLRecommendationConfidenceLevel>99</MLRecommendationConfidenceLevel>
   <MLRecommendationNumberOfResults>5</MLRecommendationNumberOfResults>
+  <HideAutocompleteMethodOptions>true</HideAutocompleteMethodOptions>
   <EnableNotificationCenter>false</EnableNotificationCenter>
   <EnableStaticSplashScreen>false</EnableStaticSplashScreen>
   <EnablePersistExtensions>true</EnablePersistExtensions>

Original file line number	Diff line number	Diff line change
`@@ -749,8 +749,7 @@ protected DynamoViewModel(StartConfiguration startConfiguration)`
`749`	`749`	`model.ComputeModelDeserialized += model_ComputeModelDeserialized;`
`750`	`750`	`model.RequestNotification += model_RequestNotification;`
`751`	`751`
`752`		`- preferencesViewModel = new PreferencesViewModel(this);`
`753`		`-`
	`752`	`+ preferencesViewModel = new PreferencesViewModel(this);`
`754`	`753`
`755`	`754`	`if (!DynamoModel.IsTestMode && !DynamoModel.IsHeadless)`
`756`	`755`	`{`