Allowing the User to be constructed without a CacheContainer

StanleyGoldman · StanleyGoldman · commit 849e2b4072dc · 2018-08-22T10:23:57.000-04:00
diff --git a/src/GitHub.Api/Git/Repository.cs b/src/GitHub.Api/Git/Repository.cs
@@ -423,9 +423,12 @@ public class User : IUser
 
         public User(ICacheContainer cacheContainer)
         {
-            this.cacheContainer = cacheContainer;
-            cacheContainer.CacheInvalidated += (type) => { if (type == CacheType.GitUser) GitUserCacheOnCacheInvalidated(); };
-            cacheContainer.CacheUpdated += (type, dt) => { if (type == CacheType.GitUser) CacheHasBeenUpdated(dt); };
+            if (cacheContainer != null)
+            {
+                this.cacheContainer = cacheContainer;
+                cacheContainer.CacheInvalidated += (type) => { if (type == CacheType.GitUser) GitUserCacheOnCacheInvalidated(); };
+                cacheContainer.CacheUpdated += (type, dt) => { if (type == CacheType.GitUser) CacheHasBeenUpdated(dt); };
+            }
         }
 
         public void CheckAndRaiseEventsIfCacheNewer(CacheType cacheType, CacheUpdateEvent cacheUpdateEvent) => cacheContainer.CheckAndRaiseEventsIfCacheNewer(CacheType.GitUser, cacheUpdateEvent);

Original file line number	Diff line number	Diff line change
`@@ -423,9 +423,12 @@ public class User : IUser`
`423`	`423`
`424`	`424`	`public User(ICacheContainer cacheContainer)`
`425`	`425`	`{`
`426`		`- this.cacheContainer = cacheContainer;`
`427`		`- cacheContainer.CacheInvalidated += (type) => { if (type == CacheType.GitUser) GitUserCacheOnCacheInvalidated(); };`
`428`		`- cacheContainer.CacheUpdated += (type, dt) => { if (type == CacheType.GitUser) CacheHasBeenUpdated(dt); };`
	`426`	`+ if (cacheContainer != null)`
	`427`	`+ {`
	`428`	`+ this.cacheContainer = cacheContainer;`
	`429`	`+ cacheContainer.CacheInvalidated += (type) => { if (type == CacheType.GitUser) GitUserCacheOnCacheInvalidated(); };`
	`430`	`+ cacheContainer.CacheUpdated += (type, dt) => { if (type == CacheType.GitUser) CacheHasBeenUpdated(dt); };`
	`431`	`+ }`
`429`	`432`	`}`
`430`	`433`
`431`	`434`	`public void CheckAndRaiseEventsIfCacheNewer(CacheType cacheType, CacheUpdateEvent cacheUpdateEvent) => cacheContainer.CheckAndRaiseEventsIfCacheNewer(CacheType.GitUser, cacheUpdateEvent);`