DYN-6140: Fix ADP logging hang (#15560)

Co-authored-by: Bogdan Zavu <[email protected]>

Author: twastvedt

src/DynamoCore/Configuration/PreferenceSettings.cs

@@ -28,6 +28,9 @@ public class PreferenceSettings : NotificationObject, IPreferences, IRenderPreci
         private bool isBackgroundGridVisible;
         private bool disableTrustWarnings = false;
         private bool isNotificationCenterEnabled;
+        private bool isADPChecked = false;
+        private bool isADPOptedIn = false;
+
         #region Constants
         /// <summary>
         /// Indicates the maximum number of files shown in Recent Files
@@ -77,12 +80,23 @@ public class PreferenceSettings : NotificationObject, IPreferences, IRenderPreci
 
         /// <summary>
         /// Indicates whether ADP analytics reporting is approved or not.
+        /// Note that this property is called often and the inner call to IsADPOptinIn can be slow sometimes
+        /// especially when there is an error involved. And therefore we will only check this once per instance.
         /// </summary>
         [XmlIgnore]
         [Obsolete("Setter is obsolete - ADP consent should not be set directly, it should be set using the consent dialog.")]
         public bool IsADPAnalyticsReportingApproved
         {
-            get { return Logging.AnalyticsService.IsADPOptedIn; }
+            get
+            {
+                if (!isADPChecked)
+                {
+                    isADPChecked = true;
+                    isADPOptedIn = AnalyticsService.IsADPOptedIn;
+                }
+
+                return isADPOptedIn;
+            }
             set { throw new Exception("do not use"); }
         }
         #endregion

src/DynamoCore/Logging/AnalyticsService.cs

@@ -1,4 +1,4 @@
-using Dynamo.Graph.Workspaces;
+using Dynamo.Graph.Workspaces;
 using Dynamo.Models;
 using Analytics.NET.ADP;
 using Autodesk.Analytics.Core;
@@ -66,7 +66,7 @@ internal static bool IsADPOptedIn
                 {
                     return false;
                 }
-                return adpAnalyticsUI.IsOptedIn(150,200);
+                return adpAnalyticsUI.IsOptedIn(5,500);
             }
 
             set
@@ -121,4 +121,4 @@ internal static string GetUserIDForSession()
             return null;
         }
     }
-}
+}