Fix logic around only sending stats once per day.

Steven Kirk · Steven Kirk · commit b68d764ac3bc · 2016-06-23T14:09:22.000+02:00
diff --git a/src/GitHub.Exports/Services/UsageTracker.cs b/src/GitHub.Exports/Services/UsageTracker.cs
@@ -173,14 +173,13 @@ async void TimerTick(object sender, EventArgs e)
                 var includeMonthly = lastDate.Month != currentDate.Month;
 
                 // Only send stats once a day.
-                if (lastDate != currentDate)
+                if (lastDate.Date != currentDate.Date)
                 {
                     await SendUsage(includeWeekly, includeMonthly);
+                    ClearCounters(includeWeekly, includeMonthly);
+                    usage.LastUpdated = DateTimeOffset.Now.UtcDateTime;
+                    SaveUsage(usage);
                 }
-
-                ClearCounters(includeWeekly, includeMonthly);
-                usage.LastUpdated = DateTimeOffset.Now.UtcDateTime;
-                SaveUsage(usage);
             }
             catch //(Exception ex)
             {

Original file line number	Diff line number	Diff line change
`@@ -173,14 +173,13 @@ async void TimerTick(object sender, EventArgs e)`
`173`	`173`	`var includeMonthly = lastDate.Month != currentDate.Month;`
`174`	`174`
`175`	`175`	`// Only send stats once a day.`
`176`		`- if (lastDate != currentDate)`
	`176`	`+ if (lastDate.Date != currentDate.Date)`
`177`	`177`	`{`
`178`	`178`	`await SendUsage(includeWeekly, includeMonthly);`
	`179`	`+ ClearCounters(includeWeekly, includeMonthly);`
	`180`	`+ usage.LastUpdated = DateTimeOffset.Now.UtcDateTime;`
	`181`	`+ SaveUsage(usage);`
`179`	`182`	`}`
`180`		`-`
`181`		`- ClearCounters(includeWeekly, includeMonthly);`
`182`		`- usage.LastUpdated = DateTimeOffset.Now.UtcDateTime;`
`183`		`- SaveUsage(usage);`
`184`	`183`	`}`
`185`	`184`	`catch //(Exception ex)`
`186`	`185`	`{`