Fixing bug and removing useless log messages

StanleyGoldman · StanleyGoldman · commit 616ce615cfb5 · 2017-06-22T07:57:15.000-04:00
diff --git a/src/GitHub.Api/Metrics/UsageModel.cs b/src/GitHub.Api/Metrics/UsageModel.cs
@@ -17,7 +17,7 @@ public class Usage
 
     public class UsageModel
     {
-        private List<Usage> reports;
+        private List<Usage> reports = new List<Usage>();
 
         public IList<Usage> Reports
         {
diff --git a/src/GitHub.Api/Metrics/UsageTracker.cs b/src/GitHub.Api/Metrics/UsageTracker.cs
@@ -51,15 +51,6 @@ private UsageStore LoadUsage()
                 if (json != null)
                 {
                     result = SimpleJson.DeserializeObject<UsageStore>(json);
-                    if (result.Model == null)
-                    {
-                        Logger.Warning("Model is Null");
-                    }
-
-                    if (result.Model.Reports == null)
-                    {
-                        Logger.Warning("Reports is Null");
-                    }
                 }
                 else
                 {

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ public class Usage`
`17`	`17`
`18`	`18`	`public class UsageModel`
`19`	`19`	`{`
`20`		`- private List<Usage> reports;`
	`20`	`+ private List<Usage> reports = new List<Usage>();`
`21`	`21`
`22`	`22`	`public IList<Usage> Reports`
`23`	`23`	`{`
Original file line number	Diff line number	Diff line change
`@@ -51,15 +51,6 @@ private UsageStore LoadUsage()`
`51`	`51`	`if (json != null)`
`52`	`52`	`{`
`53`	`53`	`result = SimpleJson.DeserializeObject<UsageStore>(json);`
`54`		`- if (result.Model == null)`
`55`		`- {`
`56`		`- Logger.Warning("Model is Null");`
`57`		`- }`
`58`		`-`
`59`		`- if (result.Model.Reports == null)`
`60`		`- {`
`61`		`- Logger.Warning("Reports is Null");`
`62`		`- }`
`63`	`54`	`}`
`64`	`55`	`else`
`65`	`56`	`{`