Merge 78ad9f4 into 0405986

shana · web-flow · commit f741b6659b27 · 2018-02-08T17:51:27.000Z
diff --git a/src/GitHub.Api/SimpleApiClient.cs b/src/GitHub.Api/SimpleApiClient.cs
@@ -72,7 +72,7 @@ async Task<Repository> GetRepositoryInternal()
                         if (repo != null)
                         {
                             hasWiki = await HasWikiInternal(repo);
-                            isEnterprise = await IsEnterpriseInternal();
+                            isEnterprise = isEnterprise ?? await IsEnterpriseInternal();
                             repositoryCache = repo;
                         }
                         owner = ownerLogin;
@@ -99,9 +99,14 @@ public bool HasWiki()
             return hasWiki.HasValue && hasWiki.Value;
         }
 
-        public bool IsEnterprise()
+        public async Task<bool> IsEnterprise()
         {
-            return isEnterprise.HasValue && isEnterprise.Value;
+            if (!isEnterprise.HasValue)
+            {
+                isEnterprise = await IsEnterpriseInternal();
+            }
+
+            return isEnterprise ?? false;
         }
 
         async Task<bool> HasWikiInternal(Repository repo)
diff --git a/src/GitHub.App/Api/ApiClient.cs b/src/GitHub.App/Api/ApiClient.cs
@@ -157,7 +157,7 @@ static string GetMachineNameSafe()
             }
             catch (Exception e)
             {
-                log.Information(e, "Failed to retrieve host name using `DNS.GetHostName`");
+                log.Warning(e, "Failed to retrieve host name using `DNS.GetHostName`");
                 try
                 {
                     return Environment.MachineName;
diff --git a/src/GitHub.App/Services/TeamExplorerContext.cs b/src/GitHub.App/Services/TeamExplorerContext.cs
@@ -65,7 +65,7 @@ void Refresh()
                 {
                     // Ignore when ActiveRepositories is empty and solution hasn't changed.
                     // https://github.com/github/VisualStudio/issues/1421
-                    log.Information("Ignoring no ActiveRepository when solution hasn't changed");
+                    log.Debug("Ignoring no ActiveRepository when solution hasn't changed");
                 }
                 else
                 {
@@ -76,22 +76,22 @@ void Refresh()
 
                     if (newRepositoryPath != repositoryPath)
                     {
-                        log.Information("Fire PropertyChanged event for ActiveRepository");
+                        log.Debug("Fire PropertyChanged event for ActiveRepository");
                         ActiveRepository = repo;
                     }
                     else if (newBranchName != branchName)
                     {
-                        log.Information("Fire StatusChanged event when BranchName changes for ActiveRepository");
+                        log.Debug("Fire StatusChanged event when BranchName changes for ActiveRepository");
                         StatusChanged?.Invoke(this, EventArgs.Empty);
                     }
                     else if (newHeadSha != headSha)
                     {
-                        log.Information("Fire StatusChanged event when HeadSha changes for ActiveRepository");
+                        log.Debug("Fire StatusChanged event when HeadSha changes for ActiveRepository");
                         StatusChanged?.Invoke(this, EventArgs.Empty);
                     }
                     else if (newTrackedSha != trackedSha)
                     {
-                        log.Information("Fire StatusChanged event when TrackedSha changes for ActiveRepository");
+                        log.Debug("Fire StatusChanged event when TrackedSha changes for ActiveRepository");
                         StatusChanged?.Invoke(this, EventArgs.Empty);
                     }
 
diff --git a/src/GitHub.App/ViewModels/Dialog/LoginTabViewModel.cs b/src/GitHub.App/ViewModels/Dialog/LoginTabViewModel.cs
@@ -200,7 +200,7 @@ void HandleError(Exception ex)
 
             if (ex.IsCriticalException()) return;
 
-            log.Information(ex, "Error logging into '{BaseUri}' as '{UsernameOrEmail}'", BaseUri, UsernameOrEmail);
+            log.Error(ex, "Error logging into '{BaseUri}' as '{UsernameOrEmail}'", BaseUri, UsernameOrEmail);
             if (ex is Octokit.ForbiddenException)
             {
                 Error = new UserError(Resources.LoginFailedForbiddenMessage, ex.Message);
diff --git a/src/GitHub.App/ViewModels/GitHubPane/GitHubPaneViewModel.cs b/src/GitHub.App/ViewModels/GitHubPane/GitHubPaneViewModel.cs
@@ -379,7 +379,7 @@ async Task UpdateContent(ILocalRepositoryModel repository)
             var repositoryUrl = repository.CloneUrl.ToRepositoryUrl();
             var isDotCom = HostAddress.IsGitHubDotComUri(repositoryUrl);
             var client = await apiClientFactory.Create(repository.CloneUrl);
-            var isEnterprise = isDotCom ? false : client.IsEnterprise();
+            var isEnterprise = isDotCom ? false : await client.IsEnterprise();
 
             if ((isDotCom || isEnterprise) && await IsValidRepository(client))
             {
diff --git a/src/GitHub.Exports/Api/ISimpleApiClient.cs b/src/GitHub.Exports/Api/ISimpleApiClient.cs
@@ -11,7 +11,7 @@ public interface ISimpleApiClient
         UriString OriginalUrl { get; }
         Task<Repository> GetRepository();
         bool HasWiki();
-        bool IsEnterprise();
+        Task<bool> IsEnterprise();
         bool IsAuthenticated();
     }
 }
diff --git a/src/GitHub.Exports/Models/UsageModel.cs b/src/GitHub.Exports/Models/UsageModel.cs
@@ -1,9 +1,7 @@
-﻿using System;
-using System.Reflection;
-
-namespace GitHub.Models
+﻿namespace GitHub.Models
 {
-    public class UsageModel
+    [System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Performance", "CA1815:OverrideEqualsAndOperatorEqualsOnValueTypes", Justification = "It'll use reflection by default and we're fine with that")]
+    public struct UsageModel
     {
         public bool IsGitHubUser { get; set; }
         public bool IsEnterpriseUser { get; set; }
@@ -43,25 +41,27 @@ public class UsageModel
 
         public UsageModel Clone(bool includeWeekly, bool includeMonthly)
         {
-            var result = new UsageModel();
-            var properties = result.GetType().GetRuntimeProperties();
-
-            foreach (var property in properties)
-            {
-                var cloneValue = property.PropertyType == typeof(int);
-
-                if (property.Name == nameof(result.NumberOfStartupsWeek))
-                    cloneValue = includeWeekly;
-                else if (property.Name == nameof(result.NumberOfStartupsMonth))
-                    cloneValue = includeMonthly;
+            var result = this;
+            if (!includeWeekly)
+                result.NumberOfStartupsWeek = 0;
+            if (!includeMonthly)
+                result.NumberOfStartupsMonth = 0;
+            return result;
+        }
 
-                if (cloneValue)
-                {
-                    var value = property.GetValue(this);
-                    property.SetValue(result, value);
-                }
-            }
+        public UsageModel ClearCounters(bool clearWeekly, bool clearMonthly)
+        {
+            var result = new UsageModel();
+            if (!clearWeekly)
+                result.NumberOfStartupsWeek = NumberOfStartupsWeek;
+            if (!clearMonthly)
+                result.NumberOfStartupsMonth = NumberOfStartupsMonth;
 
+            result.IsGitHubUser = IsGitHubUser;
+            result.IsEnterpriseUser = IsEnterpriseUser;
+            result.AppVersion = AppVersion;
+            result.VSVersion = VSVersion;
+            result.Lang = Lang;
             return result;
         }
     }
diff --git a/src/GitHub.Exports/SimpleJson.cs b/src/GitHub.Exports/SimpleJson.cs
@@ -67,6 +67,7 @@
 using System.Runtime.Serialization;
 using System.Text;
 using GitHub.Reflection;
+using System.Diagnostics;
 
 // ReSharper disable LoopCanBeConvertedToQuery
 // ReSharper disable RedundantExplicitArrayCreation
@@ -1838,7 +1839,13 @@ public static ConstructorDelegate GetConstructorByReflection(ConstructorInfo con
             public static ConstructorDelegate GetConstructorByReflection(Type type, params Type[] argsType)
             {
                 ConstructorInfo constructorInfo = GetConstructorInfo(type, argsType);
-                return constructorInfo == null ? null : GetConstructorByReflection(constructorInfo);
+                // if it's a value type (i.e., struct), it won't have a default constructor, so use Activator instead
+                return constructorInfo == null ? (type.IsValueType ? GetConstructorForValueType(type) : null) : GetConstructorByReflection(constructorInfo);
+            }
+
+            static ConstructorDelegate GetConstructorForValueType(Type type)
+            {
+                return delegate (object[] args) { return Activator.CreateInstance(type); };
             }
 
 #if !SIMPLE_JSON_NO_LINQ_EXPRESSION
@@ -1865,7 +1872,8 @@ public static ConstructorDelegate GetConstructorByExpression(ConstructorInfo con
             public static ConstructorDelegate GetConstructorByExpression(Type type, params Type[] argsType)
             {
                 ConstructorInfo constructorInfo = GetConstructorInfo(type, argsType);
-                return constructorInfo == null ? null : GetConstructorByExpression(constructorInfo);
+                // if it's a value type (i.e., struct), it won't have a default constructor, so use Activator instead
+                return constructorInfo == null ? (type.IsValueType ? GetConstructorForValueType(type) : null) : GetConstructorByExpression(constructorInfo);
             }
 
 #endif
@@ -1925,6 +1933,9 @@ public static SetDelegate GetSetMethod(PropertyInfo propertyInfo)
 #if SIMPLE_JSON_NO_LINQ_EXPRESSION
                 return GetSetMethodByReflection(propertyInfo);
 #else
+                // if it's a struct, we want to use reflection, as linq expressions modify copies of the object and not the real thing
+                if (propertyInfo.DeclaringType.IsValueType)
+                    return GetSetMethodByReflection(propertyInfo);
                 return GetSetMethodByExpression(propertyInfo);
 #endif
             }
@@ -1934,6 +1945,9 @@ public static SetDelegate GetSetMethod(FieldInfo fieldInfo)
 #if SIMPLE_JSON_NO_LINQ_EXPRESSION
                 return GetSetMethodByReflection(fieldInfo);
 #else
+                // if it's a struct, we want to use reflection, as linq expressions modify copies of the object and not the real thing
+                if (fieldInfo.DeclaringType.IsValueType)
+                    return GetSetMethodByReflection(fieldInfo);
                 return GetSetMethodByExpression(fieldInfo);
 #endif
             }
diff --git a/src/GitHub.Logging/Logging/LogManager.cs b/src/GitHub.Logging/Logging/LogManager.cs
@@ -20,7 +20,11 @@ static Logger CreateLogger()
 
             return new LoggerConfiguration()
                 .Enrich.WithThreadId()
+#if DEBUG
+                .MinimumLevel.Debug()
+#else
                 .MinimumLevel.Information()
+#endif
                 .WriteTo.File(logPath,
                     fileSizeLimitBytes: null,
                     outputTemplate: outputTemplate)
diff --git a/src/GitHub.TeamFoundation.14/Connect/GitHubConnectSection.cs b/src/GitHub.TeamFoundation.14/Connect/GitHubConnectSection.cs
@@ -360,9 +360,7 @@ void ShowNotification(ILocalRepositoryModel newrepo, string msg)
                     }
                 })
             );
-#if DEBUG
-            log.Information("Notification");
-#endif
+            log.Debug("Notification");
         }
 
         async Task RefreshRepositories()
@@ -496,12 +494,10 @@ void TrackState(object sender, PropertyChangedEventArgs e)
             {
                 if (machine.PermittedTriggers.Contains(e.PropertyName))
                 {
-#if DEBUG
-                    log.Information("{PropertyName} title:{Title} busy:{IsBusy}",
+                    log.Debug("{PropertyName} title:{Title} busy:{IsBusy}",
                         e.PropertyName,
                         ((ITeamExplorerSection)sender).Title,
                         ((ITeamExplorerSection)sender).IsBusy);
-#endif
                     machine.Fire(e.PropertyName);
                 }
             }
diff --git a/src/GitHub.TeamFoundation.14/Services/VSGitExt.cs b/src/GitHub.TeamFoundation.14/Services/VSGitExt.cs
@@ -55,7 +55,7 @@ public VSGitExt(IGitHubServiceProvider serviceProvider, IVSUIContextFactory fact
             {
                 // If we're not in the UIContext or TryInitialize fails, have another go when the UIContext changes.
                 context.UIContextChanged += ContextChanged;
-                log.Information("VSGitExt will be initialized later");
+                log.Debug("VSGitExt will be initialized later");
                 InitializeTask = Task.CompletedTask;
             }
         }
@@ -69,7 +69,7 @@ void ContextChanged(object sender, VSUIContextChangedEventArgs e)
                 // Refresh ActiveRepositories on background thread so we don't delay UI context change.
                 InitializeTask = Task.Run(() => RefreshActiveRepositories());
                 context.UIContextChanged -= ContextChanged;
-                log.Information("Initialized VSGitExt on UIContextChanged");
+                log.Debug("Initialized VSGitExt on UIContextChanged");
             }
         }
 
@@ -86,11 +86,11 @@ bool TryInitialize()
                     }
                 };
 
-                log.Information("Found IGitExt service and initialized VSGitExt");
+                log.Debug("Found IGitExt service and initialized VSGitExt");
                 return true;
             }
 
-            log.Information("Couldn't find IGitExt service");
+            log.Error("Couldn't find IGitExt service");
             return false;
         }
 
diff --git a/src/GitHub.VisualStudio.UI/Base/TeamExplorerItemBase.cs b/src/GitHub.VisualStudio.UI/Base/TeamExplorerItemBase.cs
@@ -126,7 +126,7 @@ protected async Task<RepositoryOrigin> GetRepositoryOrigin()
             {
                 var repo = await SimpleApiClient.GetRepository();
 
-                if ((repo.FullName == ActiveRepoName || repo.Id == 0) && SimpleApiClient.IsEnterprise())
+                if ((repo.FullName == ActiveRepoName || repo.Id == 0) && await SimpleApiClient.IsEnterprise())
                 {
                     return RepositoryOrigin.Enterprise;
                 }
diff --git a/src/GitHub.VisualStudio/Menus/MenuBase.cs b/src/GitHub.VisualStudio/Menus/MenuBase.cs
@@ -123,7 +123,7 @@ protected async Task<bool> IsGitHubRepo()
             {
                 var repo = await SimpleApiClient.GetRepository();
                 var activeRepoFullName = ActiveRepo.Owner + '/' + ActiveRepo.Name;
-                return (repo.FullName == activeRepoFullName || repo.Id == 0) && SimpleApiClient.IsEnterprise();
+                return (repo.FullName == activeRepoFullName || repo.Id == 0) && await SimpleApiClient.IsEnterprise();
             }
             return isdotcom;
         }
diff --git a/src/GitHub.VisualStudio/Services/UsageService.cs b/src/GitHub.VisualStudio/Services/UsageService.cs
@@ -8,12 +8,15 @@
 using GitHub.Helpers;
 using GitHub.Models;
 using Task = System.Threading.Tasks.Task;
+using GitHub.Logging;
+using Serilog;
 
 namespace GitHub.Services
 {
     [Export(typeof(IUsageService))]
     public class UsageService : IUsageService
     {
+        static readonly ILogger log = LogManager.ForContext<UsageService>();
         const string StoreFileName = "ghfvs.usage";
         static readonly Calendar cal = CultureInfo.InvariantCulture.Calendar;
         readonly IGitHubServiceProvider serviceProvider;
@@ -32,12 +35,12 @@ public bool IsSameDay(DateTimeOffset lastUpdated)
 
         public bool IsSameWeek(DateTimeOffset lastUpdated)
         {
-            return GetIso8601WeekOfYear(lastUpdated) == GetIso8601WeekOfYear(DateTimeOffset.Now);
+            return GetIso8601WeekOfYear(lastUpdated) == GetIso8601WeekOfYear(DateTimeOffset.Now) && lastUpdated.Year == DateTimeOffset.Now.Year;
         }
 
         public bool IsSameMonth(DateTimeOffset lastUpdated)
         {
-            return lastUpdated.Month == DateTimeOffset.Now.Month;
+            return lastUpdated.Month == DateTimeOffset.Now.Month && lastUpdated.Year == DateTimeOffset.Now.Year;
         }
 
         public IDisposable StartTimer(Func<Task> callback, TimeSpan dueTime, TimeSpan period)
@@ -65,8 +68,9 @@ public async Task<UsageData> ReadLocalData()
                     SimpleJson.DeserializeObject<UsageData>(json) :
                     new UsageData { Model = new UsageModel() };
             }
-            catch
+            catch(Exception ex)
             {
+                log.Error(ex, "Error deserializing usage");
                 return new UsageData { Model = new UsageModel() };
             }
         }
@@ -79,9 +83,9 @@ public async Task WriteLocalData(UsageData data)
                 var json = SimpleJson.SerializeObject(data);
                 await WriteAllTextAsync(storePath, json);
             }
-            catch
+            catch(Exception ex)
             {
-                // log.Warn("Failed to write usage data", ex);
+                log.Error(ex,"Failed to write usage data");
             }
         }
 
diff --git a/src/GitHub.VisualStudio/Services/UsageTracker.cs b/src/GitHub.VisualStudio/Services/UsageTracker.cs
diff --git a/test/UnitTests/GitHub.Api/SimpleApiClientTests.cs b/test/UnitTests/GitHub.Api/SimpleApiClientTests.cs
diff --git a/test/UnitTests/GitHub.VisualStudio/Services/MetricsTests.cs b/test/UnitTests/GitHub.VisualStudio/Services/MetricsTests.cs
diff --git a/test/UnitTests/GitHub.VisualStudio/Services/UsageTrackerTests.cs b/test/UnitTests/GitHub.VisualStudio/Services/UsageTrackerTests.cs
diff --git a/test/UnitTests/Helpers/SimpleJson.cs b/test/UnitTests/Helpers/SimpleJson.cs
diff --git a/test/UnitTests/Properties/AssemblyInfo.cs b/test/UnitTests/Properties/AssemblyInfo.cs
diff --git a/test/UnitTests/UnitTests.csproj b/test/UnitTests/UnitTests.csproj

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ async Task<Repository> GetRepositoryInternal()`
`72`	`72`	`if (repo != null)`
`73`	`73`	`{`
`74`	`74`	`hasWiki = await HasWikiInternal(repo);`
`75`		`- isEnterprise = await IsEnterpriseInternal();`
	`75`	`+ isEnterprise = isEnterprise ?? await IsEnterpriseInternal();`
`76`	`76`	`repositoryCache = repo;`
`77`	`77`	`}`
`78`	`78`	`owner = ownerLogin;`
`@@ -99,9 +99,14 @@ public bool HasWiki()`
`99`	`99`	`return hasWiki.HasValue && hasWiki.Value;`
`100`	`100`	`}`
`101`	`101`
`102`		`- public bool IsEnterprise()`
	`102`	`+ public async Task<bool> IsEnterprise()`
`103`	`103`	`{`
`104`		`- return isEnterprise.HasValue && isEnterprise.Value;`
	`104`	`+ if (!isEnterprise.HasValue)`
	`105`	`+ {`
	`106`	`+ isEnterprise = await IsEnterpriseInternal();`
	`107`	`+ }`
	`108`	`+`
	`109`	`+ return isEnterprise ?? false;`
`105`	`110`	`}`
`106`	`111`
`107`	`112`	`async Task<bool> HasWikiInternal(Repository repo)`
Original file line number	Diff line number	Diff line change
`@@ -157,7 +157,7 @@ static string GetMachineNameSafe()`
`157`	`157`	`}`
`158`	`158`	`catch (Exception e)`
`159`	`159`	`{`
`160`		- log.Information(e, "Failed to retrieve host name using `DNS.GetHostName`");
	`160`	+ log.Warning(e, "Failed to retrieve host name using `DNS.GetHostName`");
`161`	`161`	`try`
`162`	`162`	`{`
`163`	`163`	`return Environment.MachineName;`
Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ void Refresh()`
`65`	`65`	`{`
`66`	`66`	`// Ignore when ActiveRepositories is empty and solution hasn't changed.`
`67`	`67`	`// https://github.com/github/VisualStudio/issues/1421`
`68`		`- log.Information("Ignoring no ActiveRepository when solution hasn't changed");`
	`68`	`+ log.Debug("Ignoring no ActiveRepository when solution hasn't changed");`
`69`	`69`	`}`
`70`	`70`	`else`
`71`	`71`	`{`
`@@ -76,22 +76,22 @@ void Refresh()`
`76`	`76`
`77`	`77`	`if (newRepositoryPath != repositoryPath)`
`78`	`78`	`{`
`79`		`- log.Information("Fire PropertyChanged event for ActiveRepository");`
	`79`	`+ log.Debug("Fire PropertyChanged event for ActiveRepository");`
`80`	`80`	`ActiveRepository = repo;`
`81`	`81`	`}`
`82`	`82`	`else if (newBranchName != branchName)`
`83`	`83`	`{`
`84`		`- log.Information("Fire StatusChanged event when BranchName changes for ActiveRepository");`
	`84`	`+ log.Debug("Fire StatusChanged event when BranchName changes for ActiveRepository");`
`85`	`85`	`StatusChanged?.Invoke(this, EventArgs.Empty);`
`86`	`86`	`}`
`87`	`87`	`else if (newHeadSha != headSha)`
`88`	`88`	`{`
`89`		`- log.Information("Fire StatusChanged event when HeadSha changes for ActiveRepository");`
	`89`	`+ log.Debug("Fire StatusChanged event when HeadSha changes for ActiveRepository");`
`90`	`90`	`StatusChanged?.Invoke(this, EventArgs.Empty);`
`91`	`91`	`}`
`92`	`92`	`else if (newTrackedSha != trackedSha)`
`93`	`93`	`{`
`94`		`- log.Information("Fire StatusChanged event when TrackedSha changes for ActiveRepository");`
	`94`	`+ log.Debug("Fire StatusChanged event when TrackedSha changes for ActiveRepository");`
`95`	`95`	`StatusChanged?.Invoke(this, EventArgs.Empty);`
`96`	`96`	`}`
`97`	`97`
Original file line number	Diff line number	Diff line change
`@@ -200,7 +200,7 @@ void HandleError(Exception ex)`
`200`	`200`
`201`	`201`	`if (ex.IsCriticalException()) return;`
`202`	`202`
`203`		`- log.Information(ex, "Error logging into '{BaseUri}' as '{UsernameOrEmail}'", BaseUri, UsernameOrEmail);`
	`203`	`+ log.Error(ex, "Error logging into '{BaseUri}' as '{UsernameOrEmail}'", BaseUri, UsernameOrEmail);`
`204`	`204`	`if (ex is Octokit.ForbiddenException)`
`205`	`205`	`{`
`206`	`206`	`Error = new UserError(Resources.LoginFailedForbiddenMessage, ex.Message);`
Original file line number	Diff line number	Diff line change
`@@ -379,7 +379,7 @@ async Task UpdateContent(ILocalRepositoryModel repository)`
`379`	`379`	`var repositoryUrl = repository.CloneUrl.ToRepositoryUrl();`
`380`	`380`	`var isDotCom = HostAddress.IsGitHubDotComUri(repositoryUrl);`
`381`	`381`	`var client = await apiClientFactory.Create(repository.CloneUrl);`
`382`		`- var isEnterprise = isDotCom ? false : client.IsEnterprise();`
	`382`	`+ var isEnterprise = isDotCom ? false : await client.IsEnterprise();`
`383`	`383`
`384`	`384`	`if ((isDotCom \|\| isEnterprise) && await IsValidRepository(client))`
`385`	`385`	`{`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ public interface ISimpleApiClient`
`11`	`11`	`UriString OriginalUrl { get; }`
`12`	`12`	`Task<Repository> GetRepository();`
`13`	`13`	`bool HasWiki();`
`14`		`- bool IsEnterprise();`
	`14`	`+ Task<bool> IsEnterprise();`
`15`	`15`	`bool IsAuthenticated();`
`16`	`16`	`}`
`17`	`17`	`}`
Original file line number	Diff line number	Diff line change
`@@ -360,9 +360,7 @@ void ShowNotification(ILocalRepositoryModel newrepo, string msg)`
`360`	`360`	`}`
`361`	`361`	`})`
`362`	`362`	`);`
`363`		`-#if DEBUG`
`364`		`- log.Information("Notification");`
`365`		`-#endif`
	`363`	`+ log.Debug("Notification");`
`366`	`364`	`}`
`367`	`365`
`368`	`366`	`async Task RefreshRepositories()`
`@@ -496,12 +494,10 @@ void TrackState(object sender, PropertyChangedEventArgs e)`
`496`	`494`	`{`
`497`	`495`	`if (machine.PermittedTriggers.Contains(e.PropertyName))`
`498`	`496`	`{`
`499`		`-#if DEBUG`
`500`		`- log.Information("{PropertyName} title:{Title} busy:{IsBusy}",`
	`497`	`+ log.Debug("{PropertyName} title:{Title} busy:{IsBusy}",`
`501`	`498`	`e.PropertyName,`
`502`	`499`	`((ITeamExplorerSection)sender).Title,`
`503`	`500`	`((ITeamExplorerSection)sender).IsBusy);`
`504`		`-#endif`
`505`	`501`	`machine.Fire(e.PropertyName);`
`506`	`502`	`}`
`507`	`503`	`}`