DYN-7268 Improve UX when IDSDK is not installed on system running Dynamo #2 (#15799)

zeusongit · web-flow · commit ec3ad206ef38 · 2025-02-06T17:44:01.000-05:00
diff --git a/src/DynamoCore/Core/AuthenticationManager.cs b/src/DynamoCore/Core/AuthenticationManager.cs
@@ -126,5 +126,20 @@ private void OnLoginStateChanged(LoginState status)
             }
         }
 
+        /// <summary>
+        /// Returns whether the IDSDK is initialized or not for Dynamo Sandbox, in host environment defaults to true.
+        /// </summary>
+        internal bool IsIDSDKInitialized()
+        {
+            if (authProvider is IDSDKManager idsdkProvider)
+            {
+                if (!idsdkProvider.IsIDSDKInitialized)
+                {
+                    return false;
+                }
+            }
+            return true;
+        }
+
     }
 }
diff --git a/src/DynamoCore/Core/IDSDKManager.cs b/src/DynamoCore/Core/IDSDKManager.cs
@@ -24,29 +24,27 @@ public class IDSDKManager : IOAuth2AuthProvider, IOAuth2AccessTokenProvider, IOA
         public event Action<LoginState> LoginStateChanged;
 
         /// <summary>
-        /// The event is fired when IDSDK initialization fails
-        /// </summary>
-        internal event EventHandler ErrorInitializingIDSDK;
-
-        /// <summary>
-        /// The flag is used to prevent multiple error messages from being shown, since initialization error can be thrown multiple times.
+        /// Returns the login status of the current session.
         /// </summary>
-        internal bool isErrorInitializingMsgShown;
-        private void OnErrorInitializingIDSDK()
+        /// <returns>LoginState Enum value</returns>
+        public LoginState LoginState
         {
-            ErrorInitializingIDSDK?.Invoke(this, EventArgs.Empty);
+            get
+            {
+                var result = IDSDK_IsLoggedIn();
+                return result ? LoginState.LoggedIn : LoginState.LoggedOut;
+            }
         }
 
         /// <summary>
-        /// Returns the login status of the current session.
+        /// Returns the status of IDSDK installation, will return false if IDSDK fails to initialize.
         /// </summary>
         /// <returns>LoginState Enum value</returns>
-        public LoginState LoginState
+        public bool IsIDSDKInitialized
         {
             get
             {
-                var result = IDSDK_IsLoggedIn();
-                return result ? LoginState.LoggedIn : LoginState.LoggedOut;
+                return Initialize();
             }
         }
 
@@ -167,7 +165,7 @@ private bool IDSDK_Login()
             }
             else
             {
-                if (Initialize())
+                if (IsIDSDKInitialized)
                 {
                     idsdk_status_code statusCode = Client.Login();
                     if (Client.IsSuccess(statusCode))
@@ -180,7 +178,7 @@ private bool IDSDK_Login()
         }
         private bool IDSDK_IsLoggedIn()
         {
-            if (Initialize())
+            if (IsIDSDKInitialized)
             {
                 bool ret = Client.IsLoggedIn();
                 return ret;
@@ -244,7 +242,6 @@ private bool Initialize()
             {
                 if (Client.IsInitialized())
                 {
-                    isErrorInitializingMsgShown = false;
                     return true;
                 }
 
@@ -264,7 +261,6 @@ private bool Initialize()
                         Client.LoginCompleteEvent += AuthCompleteEventHandler;
                         ret = SetProductConfigs(Configurations.DynamoAsString, server, client_id);
                         Client.SetServer(server);
-                        isErrorInitializingMsgShown = false;
                         return ret;
                     }
                 }
@@ -274,8 +270,6 @@ private bool Initialize()
             {
                 DynamoConsoleLogger.OnLogMessageToDynamoConsole("An error occurred while initializing Auth Service (IDSDK).");
             }
-
-            OnErrorInitializingIDSDK();
             return false;
         }
         private bool Deinitialize()
diff --git a/src/DynamoCore/PublicAPI.Unshipped.txt b/src/DynamoCore/PublicAPI.Unshipped.txt
@@ -364,6 +364,7 @@ Dynamo.Core.IDSDKManager
 Dynamo.Core.IDSDKManager.Dispose() -> void
 Dynamo.Core.IDSDKManager.GetAccessToken() -> string
 Dynamo.Core.IDSDKManager.IDSDKManager() -> void
+Dynamo.Core.IDSDKManager.IsIDSDKInitialized.get -> bool
 Dynamo.Core.IDSDKManager.IsLoggedIn() -> bool
 Dynamo.Core.IDSDKManager.Login() -> bool
 Dynamo.Core.IDSDKManager.LoginState.get -> Greg.AuthProviders.LoginState
diff --git a/src/DynamoCoreWpf/Controls/ShortcutToolbar.xaml.cs b/src/DynamoCoreWpf/Controls/ShortcutToolbar.xaml.cs
@@ -126,6 +126,7 @@ private void exportMenu_MouseLeave(object sender, System.Windows.Input.MouseEven
 
         private void LoginButton_OnClick(object sender, RoutedEventArgs e)
         {
+            if(!DynamoViewModel.IsIDSDKInitialized()) return;
             if (authManager.LoginState == LoginState.LoggedIn)
             {
                 var button = (Button)sender;
@@ -149,6 +150,7 @@ private void LoginButton_OnClick(object sender, RoutedEventArgs e)
 
         private void LogoutOption_Click(object sender, RoutedEventArgs e)
         {
+            if (!DynamoViewModel.IsIDSDKInitialized()) return;
             if (authManager.LoginState == LoginState.LoggedIn)
             {
                 var result = Wpf.Utilities.MessageBoxService.Show(Application.Current?.MainWindow, Wpf.Properties.Resources.SignOutConfirmationDialogText, Wpf.Properties.Resources.SignOutConfirmationDialogTitle,MessageBoxButton.OKCancel, new List<string>() { "Sign Out", "Cancel"}, MessageBoxImage.Information);
diff --git a/src/DynamoCoreWpf/ViewModels/Core/DynamoViewModel.cs b/src/DynamoCoreWpf/ViewModels/Core/DynamoViewModel.cs
@@ -863,10 +863,7 @@ protected DynamoViewModel(StartConfiguration startConfiguration)
 
             FileTrustViewModel = new FileTrustWarningViewModel();
             MLDataPipelineExtension = model.ExtensionManager.Extensions.OfType<DynamoMLDataPipelineExtension>().FirstOrDefault();
-            if (Model.AuthenticationManager?.AuthProvider is IDSDKManager idsdkProvider)
-            {
-                idsdkProvider.ErrorInitializingIDSDK += OnErrorInitializingIDSDK;
-            }
+            IsIDSDKInitialized();
         }
 
         private void TaskScheduler_UnobservedTaskException(object sender, UnobservedTaskExceptionEventArgs e)
@@ -1088,18 +1085,22 @@ internal void OnNodeViewReady(object nodeView)
         }
 
         /// <summary>
-        /// The event handler for cases when IDSDK fails to initialize, probably because of missing Adsk Identity Manager.
-        /// A flag is used to show the error message only once per session.
+        /// Returns whether the IDSDK is initialized or not for Dynamo Sandbox, in host environment defaults to true.
+        /// If showWarning is true, a warning message will be shown to the user if the IDSDK is not initialized.
+        /// If owner is not null, the warning message will be shown as a dialog with the owner as the parent.
         /// </summary>
-        private void OnErrorInitializingIDSDK(object sender, EventArgs e)
+        internal bool IsIDSDKInitialized(bool showWarning = true, Window owner = null)
         {
-            if (Model.AuthenticationManager?.AuthProvider is IDSDKManager idsdkProvider)
+            if (!Model.AuthenticationManager.IsIDSDKInitialized())
             {
-                if (idsdkProvider.isErrorInitializingMsgShown) return;
-
-                DynamoMessageBox.Show(Owner, WpfResources.IDSDKErrorMessage, WpfResources.IDSDKErrorMessageTitle, true, MessageBoxButton.OK, MessageBoxImage.Information);
-                idsdkProvider.isErrorInitializingMsgShown = true;
+                if (showWarning)
+                {
+                    var ownerWindow = owner ?? Owner ?? System.Windows.Application.Current.MainWindow;
+                    DynamoMessageBox.Show(ownerWindow, WpfResources.IDSDKErrorMessage, WpfResources.IDSDKErrorMessageTitle, true, MessageBoxButton.OK, MessageBoxImage.Information);
+                }
+                return false;
             }
+            return true;
         }
 
         #region Event handler destroy/create
@@ -1123,10 +1124,6 @@ protected virtual void UnsubscribeAllEvents()
 
             DynamoSelection.Instance.Selection.CollectionChanged -= SelectionOnCollectionChanged;
             UsageReportingManager.Instance.PropertyChanged -= CollectInfoManager_PropertyChanged;
-            if (Model.AuthenticationManager?.AuthProvider is IDSDKManager idsdkProvider)
-            {
-                idsdkProvider.ErrorInitializingIDSDK -= OnErrorInitializingIDSDK;
-            }
         }
 
         private void InitializeRecentFiles()
diff --git a/src/DynamoCoreWpf/ViewModels/Core/PortViewModel.cs b/src/DynamoCoreWpf/ViewModels/Core/PortViewModel.cs
@@ -502,8 +502,15 @@ private void AutoCompleteCluster(object parameter)
 
             if (wsViewModel.DynamoViewModel.IsDNAClusterPlacementEnabled)
             {
-               MLNodeClusterAutoCompletionResponse results = wsViewModel.NodeAutoCompleteSearchViewModel.GetMLNodeClusterAutocompleteResults();
-               wsViewModel.OnRequestNodeAutoCompleteViewExtension(results);
+                try
+                {
+                    MLNodeClusterAutoCompletionResponse results = wsViewModel.NodeAutoCompleteSearchViewModel.GetMLNodeClusterAutocompleteResults();
+                    wsViewModel.OnRequestNodeAutoCompleteViewExtension(results);
+                }
+                catch (Exception e)
+                {
+                    // Log the exception and show a notification to the user
+                }
             }
         }
 
diff --git a/src/DynamoCoreWpf/ViewModels/PackageManager/PackageManagerClientViewModel.cs b/src/DynamoCoreWpf/ViewModels/PackageManager/PackageManagerClientViewModel.cs
@@ -284,6 +284,7 @@ internal PackageManagerClientViewModel(DynamoViewModel dynamoViewModel, PackageM
 
         private void ToggleLoginState()
         {
+            if(!this.DynamoViewModel.IsIDSDKInitialized()) return;
             if (AuthenticationManager.LoginState == LoginState.LoggedIn)
             {
                 AuthenticationManager.Logout();
@@ -301,6 +302,7 @@ private bool CanToggleLoginState()
 
         public void PublishCurrentWorkspace(object m)
         {
+            if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;
             var ws = (CustomNodeWorkspaceModel)DynamoViewModel.CurrentSpace;
 
             CustomNodeDefinition currentFunDef;
@@ -346,6 +348,7 @@ public bool CanPublishCurrentWorkspace(object m)
 
         public void PublishNewPackage(object m)
         {
+            if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;
             var termsOfUseCheck = new TermsOfUseHelper(new TermsOfUseHelperParams
             {
                 PackageManagerClient = Model,
@@ -365,6 +368,7 @@ public bool CanPublishNewPackage(object m)
 
         public void PublishCustomNode(Function m)
         {
+            if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;
             CustomNodeInfo currentFunInfo;
             if (DynamoViewModel.Model.CustomNodeManager.TryGetNodeInfo(
                 m.Definition.FunctionId,
@@ -394,6 +398,7 @@ public bool CanPublishCustomNode(Function m)
 
         public void PublishSelectedNodes(object m)
         {
+            if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;
             var nodeList = DynamoSelection.Instance.Selection
                                 .Where(x => x is Function)
                                 .Cast<Function>()
@@ -453,12 +458,14 @@ public bool CanPublishSelectedNodes(object m)
 
         private void ShowNodePublishInfo()
         {
+            if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;
             var newPkgVm = new PublishPackageViewModel(DynamoViewModel);
             DynamoViewModel.OnRequestPackagePublishDialog(newPkgVm);
         }
 
         private void ShowNodePublishInfo(ICollection<Tuple<CustomNodeInfo, CustomNodeDefinition>> funcDefs)
         {
+            if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;
             foreach (var f in funcDefs)
             {
                 var pkg = PackageManagerExtension.PackageLoader.GetOwnerPackage(f.Item1);
@@ -513,6 +520,7 @@ public List<PackageManagerSearchElement> ListAll()
         /// <returns></returns>
         public List<PackageManagerSearchElement> GetInfectedPackages()
         {
+            if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return null;
             InfectedPackageList = new List<PackageManagerSearchElement>();
             var latestPkgs = Model.GetUsersLatestPackages();
             if (latestPkgs != null && latestPkgs.maintains?.Count > 0)
@@ -537,6 +545,7 @@ public List<PackageManagerSearchElement> GetInfectedPackages()
         public void DownloadAndInstallPackage(IPackageInfo packageInfo, string downloadPath = null)
         {
             // User needs to accept terms of use before any packages can be downloaded from package manager
+            if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;
             var prefSettings = DynamoViewModel.Model.PreferenceSettings;
             var touAccepted = prefSettings.PackageDownloadTouAccepted;
             if (!touAccepted)
diff --git a/src/DynamoCoreWpf/ViewModels/PackageManager/PackageViewModel.cs b/src/DynamoCoreWpf/ViewModels/PackageManager/PackageViewModel.cs
@@ -467,6 +467,7 @@ private void PublishNewPackage()
 
         private void ExecuteWithTou(Action acceptanceCallback)
         {
+            if (!dynamoViewModel.IsIDSDKInitialized(true, packageManagerClientViewModel.ViewModelOwner)) return;
             // create TermsOfUseHelper object to check asynchronously whether the Terms of Use has 
             // been accepted, and if so, continue to execute the provided Action.
             var termsOfUseCheck = new TermsOfUseHelper(new TermsOfUseHelperParams
diff --git a/src/DynamoCoreWpf/ViewModels/PackageManager/PublishPackageViewModel.cs b/src/DynamoCoreWpf/ViewModels/PackageManager/PublishPackageViewModel.cs
@@ -2277,6 +2277,7 @@ private void AddDllFile(string filename)
         /// Delegate used to submit the publish online request</summary>
         private void Submit()
         {
+            if (!dynamoViewModel.IsIDSDKInitialized(true, Owner)) return;
             MessageBoxResult response = DynamoModel.IsTestMode ? MessageBoxResult.OK : MessageBoxService.Show(Owner, Resources.PrePackagePublishMessage, Resources.PrePackagePublishTitle, MessageBoxButton.OKCancel, MessageBoxImage.Information);
             if (response == MessageBoxResult.Cancel)
             {
diff --git a/src/DynamoCoreWpf/ViewModels/Search/NodeAutoCompleteSearchViewModel.cs b/src/DynamoCoreWpf/ViewModels/Search/NodeAutoCompleteSearchViewModel.cs
@@ -429,11 +429,15 @@ internal void ShowNodeAutocompleMLResults()
         private MLNodeAutoCompletionResponse GetMLNodeAutocompleteResults(string requestJSON)
         {
             MLNodeAutoCompletionResponse results = null;
-            var authProvider = dynamoViewModel.Model.AuthenticationManager.AuthProvider;
-
-            if (authProvider is IOAuth2AuthProvider oauth2AuthProvider && authProvider is IOAuth2AccessTokenProvider tokenprovider)
+            try
             {
-                try
+                var authProvider = dynamoViewModel.Model.AuthenticationManager.AuthProvider;
+                if (!dynamoViewModel.IsIDSDKInitialized())
+                {
+                    throw new Exception("IDSDK missing or failed initialization.");
+                }
+
+                if (authProvider is IOAuth2AuthProvider oauth2AuthProvider && authProvider is IOAuth2AccessTokenProvider tokenprovider)
                 {
                     var uri = DynamoUtilities.PathHelper.GetServiceBackendAddress(this, nodeAutocompleteMLEndpoint);
                     var client = new RestClient(uri);
@@ -450,11 +454,11 @@ private MLNodeAutoCompletionResponse GetMLNodeAutocompleteResults(string request
                     //TODO maybe worth moving to system.text json in phases?
                     results = JsonConvert.DeserializeObject<MLNodeAutoCompletionResponse>(response.Content);
                 }
-                catch (Exception ex)
-                {
-                    dynamoViewModel.Model.Logger.Log(ex.Message);
-                    throw new Exception("Authentication failed.");
-                }
+            }
+            catch (Exception ex)
+            {
+                dynamoViewModel.Model.Logger.Log(ex.Message);
+                throw new Exception("Authentication failed.");
             }
 
             return results;
@@ -464,37 +468,48 @@ private MLNodeAutoCompletionResponse GetMLNodeAutocompleteResults(string request
         internal MLNodeClusterAutoCompletionResponse GetMLNodeClusterAutocompleteResults()
         {
             MLNodeClusterAutoCompletionResponse results = null;
+            try
+            {
+                var MLRequest = GenerateRequestForMLAutocomplete();
+                string jsonRequest = JsonConvert.SerializeObject(MLRequest);
 
-            var MLRequest = GenerateRequestForMLAutocomplete();
-            string jsonRequest = JsonConvert.SerializeObject(MLRequest);
-
-            var authProvider = dynamoViewModel.Model.AuthenticationManager.AuthProvider;
+                var authProvider = dynamoViewModel.Model.AuthenticationManager.AuthProvider;
+                if (!dynamoViewModel.IsIDSDKInitialized())
+                {
+                    throw new Exception("IDSDK missing or failed initialization.");
+                }
 
-            if (authProvider is IOAuth2AuthProvider oauth2AuthProvider && authProvider is IOAuth2AccessTokenProvider tokenprovider)
-            {
-                try
+                if (authProvider is IOAuth2AuthProvider oauth2AuthProvider && authProvider is IOAuth2AccessTokenProvider tokenprovider)
                 {
-                    var uri = DynamoUtilities.PathHelper.GetServiceBackendAddress(this, nodeClusterAutocompleteMLEndpoint);
-                    var client = new RestClient(uri);
-                    var request = new RestRequest(string.Empty, Method.Post);
-                    var tkn = tokenprovider?.GetAccessToken();
-                    if (string.IsNullOrEmpty(tkn))
+                    try
                     {
-                        throw new Exception("Authentication required.");
-                    }
-                    request.AddHeader("Authorization", $"Bearer {tkn}");
-                    request = request.AddJsonBody(jsonRequest);
-                    request.RequestFormat = DataFormat.Json;
-                    RestResponse response = client.Execute(request);
+                        var uri = DynamoUtilities.PathHelper.GetServiceBackendAddress(this, nodeClusterAutocompleteMLEndpoint);
+                        var client = new RestClient(uri);
+                        var request = new RestRequest(string.Empty, Method.Post);
+                        var tkn = tokenprovider?.GetAccessToken();
+                        if (string.IsNullOrEmpty(tkn))
+                        {
+                            throw new Exception("Authentication required.");
+                        }
+                        request.AddHeader("Authorization", $"Bearer {tkn}");
+                        request = request.AddJsonBody(jsonRequest);
+                        request.RequestFormat = DataFormat.Json;
+                        RestResponse response = client.Execute(request);
 
-                    results = JsonConvert.DeserializeObject<MLNodeClusterAutoCompletionResponse>(response.Content);
-                }
-                catch (Exception ex)
-                {
-                    dynamoViewModel.Model.Logger.Log(ex.Message);
-                    throw new Exception("Authentication failed.");
+                        results = JsonConvert.DeserializeObject<MLNodeClusterAutoCompletionResponse>(response.Content);
+                    }
+                    catch (Exception ex)
+                    {
+                        dynamoViewModel.Model.Logger.Log(ex.Message);
+                        throw new Exception("Authentication failed.");
+                    }
                 }
             }
+            catch (Exception ex)
+            {
+                dynamoViewModel.Model.Logger.Log(ex.Message);
+                throw new Exception("Authentication failed.");
+            }
 
             return results;
         }

Original file line number	Diff line number	Diff line change
`@@ -126,5 +126,20 @@ private void OnLoginStateChanged(LoginState status)`
`126`	`126`	`}`
`127`	`127`	`}`
`128`	`128`
	`129`	`+ /// <summary>`
	`130`	`+ /// Returns whether the IDSDK is initialized or not for Dynamo Sandbox, in host environment defaults to true.`
	`131`	`+ /// </summary>`
	`132`	`+ internal bool IsIDSDKInitialized()`
	`133`	`+ {`
	`134`	`+ if (authProvider is IDSDKManager idsdkProvider)`
	`135`	`+ {`
	`136`	`+ if (!idsdkProvider.IsIDSDKInitialized)`
	`137`	`+ {`
	`138`	`+ return false;`
	`139`	`+ }`
	`140`	`+ }`
	`141`	`+ return true;`
	`142`	`+ }`
	`143`	`+`
`129`	`144`	`}`
`130`	`145`	`}`
Original file line number	Diff line number	Diff line change
`@@ -24,29 +24,27 @@ public class IDSDKManager : IOAuth2AuthProvider, IOAuth2AccessTokenProvider, IOA`
`24`	`24`	`public event Action<LoginState> LoginStateChanged;`
`25`	`25`
`26`	`26`	`/// <summary>`
`27`		`- /// The event is fired when IDSDK initialization fails`
`28`		`- /// </summary>`
`29`		`- internal event EventHandler ErrorInitializingIDSDK;`
`30`		`-`
`31`		`- /// <summary>`
`32`		`- /// The flag is used to prevent multiple error messages from being shown, since initialization error can be thrown multiple times.`
	`27`	`+ /// Returns the login status of the current session.`
`33`	`28`	`/// </summary>`
`34`		`- internal bool isErrorInitializingMsgShown;`
`35`		`- private void OnErrorInitializingIDSDK()`
	`29`	`+ /// <returns>LoginState Enum value</returns>`
	`30`	`+ public LoginState LoginState`
`36`	`31`	`{`
`37`		`- ErrorInitializingIDSDK?.Invoke(this, EventArgs.Empty);`
	`32`	`+ get`
	`33`	`+ {`
	`34`	`+ var result = IDSDK_IsLoggedIn();`
	`35`	`+ return result ? LoginState.LoggedIn : LoginState.LoggedOut;`
	`36`	`+ }`
`38`	`37`	`}`
`39`	`38`
`40`	`39`	`/// <summary>`
`41`		`- /// Returns the login status of the current session.`
	`40`	`+ /// Returns the status of IDSDK installation, will return false if IDSDK fails to initialize.`
`42`	`41`	`/// </summary>`
`43`	`42`	`/// <returns>LoginState Enum value</returns>`
`44`		`- public LoginState LoginState`
	`43`	`+ public bool IsIDSDKInitialized`
`45`	`44`	`{`
`46`	`45`	`get`
`47`	`46`	`{`
`48`		`- var result = IDSDK_IsLoggedIn();`
`49`		`- return result ? LoginState.LoggedIn : LoginState.LoggedOut;`
	`47`	`+ return Initialize();`
`50`	`48`	`}`
`51`	`49`	`}`
`52`	`50`
`@@ -167,7 +165,7 @@ private bool IDSDK_Login()`
`167`	`165`	`}`
`168`	`166`	`else`
`169`	`167`	`{`
`170`		`- if (Initialize())`
	`168`	`+ if (IsIDSDKInitialized)`
`171`	`169`	`{`
`172`	`170`	`idsdk_status_code statusCode = Client.Login();`
`173`	`171`	`if (Client.IsSuccess(statusCode))`
`@@ -180,7 +178,7 @@ private bool IDSDK_Login()`
`180`	`178`	`}`
`181`	`179`	`private bool IDSDK_IsLoggedIn()`
`182`	`180`	`{`
`183`		`- if (Initialize())`
	`181`	`+ if (IsIDSDKInitialized)`
`184`	`182`	`{`
`185`	`183`	`bool ret = Client.IsLoggedIn();`
`186`	`184`	`return ret;`
`@@ -244,7 +242,6 @@ private bool Initialize()`
`244`	`242`	`{`
`245`	`243`	`if (Client.IsInitialized())`
`246`	`244`	`{`
`247`		`- isErrorInitializingMsgShown = false;`
`248`	`245`	`return true;`
`249`	`246`	`}`
`250`	`247`
`@@ -264,7 +261,6 @@ private bool Initialize()`
`264`	`261`	`Client.LoginCompleteEvent += AuthCompleteEventHandler;`
`265`	`262`	`ret = SetProductConfigs(Configurations.DynamoAsString, server, client_id);`
`266`	`263`	`Client.SetServer(server);`
`267`		`- isErrorInitializingMsgShown = false;`
`268`	`264`	`return ret;`
`269`	`265`	`}`
`270`	`266`	`}`
`@@ -274,8 +270,6 @@ private bool Initialize()`
`274`	`270`	`{`
`275`	`271`	`DynamoConsoleLogger.OnLogMessageToDynamoConsole("An error occurred while initializing Auth Service (IDSDK).");`
`276`	`272`	`}`
`277`		`-`
`278`		`- OnErrorInitializingIDSDK();`
`279`	`273`	`return false;`
`280`	`274`	`}`
`281`	`275`	`private bool Deinitialize()`
Original file line number	Diff line number	Diff line change
`@@ -126,6 +126,7 @@ private void exportMenu_MouseLeave(object sender, System.Windows.Input.MouseEven`
`126`	`126`
`127`	`127`	`private void LoginButton_OnClick(object sender, RoutedEventArgs e)`
`128`	`128`	`{`
	`129`	`+ if(!DynamoViewModel.IsIDSDKInitialized()) return;`
`129`	`130`	`if (authManager.LoginState == LoginState.LoggedIn)`
`130`	`131`	`{`
`131`	`132`	`var button = (Button)sender;`
`@@ -149,6 +150,7 @@ private void LoginButton_OnClick(object sender, RoutedEventArgs e)`
`149`	`150`
`150`	`151`	`private void LogoutOption_Click(object sender, RoutedEventArgs e)`
`151`	`152`	`{`
	`153`	`+ if (!DynamoViewModel.IsIDSDKInitialized()) return;`
`152`	`154`	`if (authManager.LoginState == LoginState.LoggedIn)`
`153`	`155`	`{`
`154`	`156`	`var result = Wpf.Utilities.MessageBoxService.Show(Application.Current?.MainWindow, Wpf.Properties.Resources.SignOutConfirmationDialogText, Wpf.Properties.Resources.SignOutConfirmationDialogTitle,MessageBoxButton.OKCancel, new List<string>() { "Sign Out", "Cancel"}, MessageBoxImage.Information);`
Original file line number	Diff line number	Diff line change
`@@ -502,8 +502,15 @@ private void AutoCompleteCluster(object parameter)`
`502`	`502`
`503`	`503`	`if (wsViewModel.DynamoViewModel.IsDNAClusterPlacementEnabled)`
`504`	`504`	`{`
`505`		`- MLNodeClusterAutoCompletionResponse results = wsViewModel.NodeAutoCompleteSearchViewModel.GetMLNodeClusterAutocompleteResults();`
`506`		`- wsViewModel.OnRequestNodeAutoCompleteViewExtension(results);`
	`505`	`+ try`
	`506`	`+ {`
	`507`	`+ MLNodeClusterAutoCompletionResponse results = wsViewModel.NodeAutoCompleteSearchViewModel.GetMLNodeClusterAutocompleteResults();`
	`508`	`+ wsViewModel.OnRequestNodeAutoCompleteViewExtension(results);`
	`509`	`+ }`
	`510`	`+ catch (Exception e)`
	`511`	`+ {`
	`512`	`+ // Log the exception and show a notification to the user`
	`513`	`+ }`
`507`	`514`	`}`
`508`	`515`	`}`
`509`	`516`
Original file line number	Diff line number	Diff line change
`@@ -284,6 +284,7 @@ internal PackageManagerClientViewModel(DynamoViewModel dynamoViewModel, PackageM`
`284`	`284`
`285`	`285`	`private void ToggleLoginState()`
`286`	`286`	`{`
	`287`	`+ if(!this.DynamoViewModel.IsIDSDKInitialized()) return;`
`287`	`288`	`if (AuthenticationManager.LoginState == LoginState.LoggedIn)`
`288`	`289`	`{`
`289`	`290`	`AuthenticationManager.Logout();`
`@@ -301,6 +302,7 @@ private bool CanToggleLoginState()`
`301`	`302`
`302`	`303`	`public void PublishCurrentWorkspace(object m)`
`303`	`304`	`{`
	`305`	`+ if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;`
`304`	`306`	`var ws = (CustomNodeWorkspaceModel)DynamoViewModel.CurrentSpace;`
`305`	`307`
`306`	`308`	`CustomNodeDefinition currentFunDef;`
`@@ -346,6 +348,7 @@ public bool CanPublishCurrentWorkspace(object m)`
`346`	`348`
`347`	`349`	`public void PublishNewPackage(object m)`
`348`	`350`	`{`
	`351`	`+ if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;`
`349`	`352`	`var termsOfUseCheck = new TermsOfUseHelper(new TermsOfUseHelperParams`
`350`	`353`	`{`
`351`	`354`	`PackageManagerClient = Model,`
`@@ -365,6 +368,7 @@ public bool CanPublishNewPackage(object m)`
`365`	`368`
`366`	`369`	`public void PublishCustomNode(Function m)`
`367`	`370`	`{`
	`371`	`+ if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;`
`368`	`372`	`CustomNodeInfo currentFunInfo;`
`369`	`373`	`if (DynamoViewModel.Model.CustomNodeManager.TryGetNodeInfo(`
`370`	`374`	`m.Definition.FunctionId,`
`@@ -394,6 +398,7 @@ public bool CanPublishCustomNode(Function m)`
`394`	`398`
`395`	`399`	`public void PublishSelectedNodes(object m)`
`396`	`400`	`{`
	`401`	`+ if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;`
`397`	`402`	`var nodeList = DynamoSelection.Instance.Selection`
`398`	`403`	`.Where(x => x is Function)`
`399`	`404`	`.Cast<Function>()`
`@@ -453,12 +458,14 @@ public bool CanPublishSelectedNodes(object m)`
`453`	`458`
`454`	`459`	`private void ShowNodePublishInfo()`
`455`	`460`	`{`
	`461`	`+ if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;`
`456`	`462`	`var newPkgVm = new PublishPackageViewModel(DynamoViewModel);`
`457`	`463`	`DynamoViewModel.OnRequestPackagePublishDialog(newPkgVm);`
`458`	`464`	`}`
`459`	`465`
`460`	`466`	`private void ShowNodePublishInfo(ICollection<Tuple<CustomNodeInfo, CustomNodeDefinition>> funcDefs)`
`461`	`467`	`{`
	`468`	`+ if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;`
`462`	`469`	`foreach (var f in funcDefs)`
`463`	`470`	`{`
`464`	`471`	`var pkg = PackageManagerExtension.PackageLoader.GetOwnerPackage(f.Item1);`
`@@ -513,6 +520,7 @@ public List<PackageManagerSearchElement> ListAll()`
`513`	`520`	`/// <returns></returns>`
`514`	`521`	`public List<PackageManagerSearchElement> GetInfectedPackages()`
`515`	`522`	`{`
	`523`	`+ if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return null;`
`516`	`524`	`InfectedPackageList = new List<PackageManagerSearchElement>();`
`517`	`525`	`var latestPkgs = Model.GetUsersLatestPackages();`
`518`	`526`	`if (latestPkgs != null && latestPkgs.maintains?.Count > 0)`
`@@ -537,6 +545,7 @@ public List<PackageManagerSearchElement> GetInfectedPackages()`
`537`	`545`	`public void DownloadAndInstallPackage(IPackageInfo packageInfo, string downloadPath = null)`
`538`	`546`	`{`
`539`	`547`	`// User needs to accept terms of use before any packages can be downloaded from package manager`
	`548`	`+ if (!this.DynamoViewModel.IsIDSDKInitialized(true, ViewModelOwner)) return;`
`540`	`549`	`var prefSettings = DynamoViewModel.Model.PreferenceSettings;`
`541`	`550`	`var touAccepted = prefSettings.PackageDownloadTouAccepted;`
`542`	`551`	`if (!touAccepted)`
Original file line number	Diff line number	Diff line change
`@@ -467,6 +467,7 @@ private void PublishNewPackage()`
`467`	`467`
`468`	`468`	`private void ExecuteWithTou(Action acceptanceCallback)`
`469`	`469`	`{`
	`470`	`+ if (!dynamoViewModel.IsIDSDKInitialized(true, packageManagerClientViewModel.ViewModelOwner)) return;`
`470`	`471`	`// create TermsOfUseHelper object to check asynchronously whether the Terms of Use has`
`471`	`472`	`// been accepted, and if so, continue to execute the provided Action.`
`472`	`473`	`var termsOfUseCheck = new TermsOfUseHelper(new TermsOfUseHelperParams`
Original file line number	Diff line number	Diff line change
`@@ -2277,6 +2277,7 @@ private void AddDllFile(string filename)`
`2277`	`2277`	`/// Delegate used to submit the publish online request</summary>`
`2278`	`2278`	`private void Submit()`
`2279`	`2279`	`{`
	`2280`	`+ if (!dynamoViewModel.IsIDSDKInitialized(true, Owner)) return;`
`2280`	`2281`	`MessageBoxResult response = DynamoModel.IsTestMode ? MessageBoxResult.OK : MessageBoxService.Show(Owner, Resources.PrePackagePublishMessage, Resources.PrePackagePublishTitle, MessageBoxButton.OKCancel, MessageBoxImage.Information);`
`2281`	`2282`	`if (response == MessageBoxResult.Cancel)`
`2282`	`2283`	`{`