Merge pull request #57 from Azure-Samples/tibre/cacheMethodSwapped

jmprieur · web-flow · commit bf3bcce3030a · 2019-05-31T00:16:47.000+02:00
Fixed swapped cache methods
diff --git a/TaskWebApp/Utils/MSALPerUserSessionTokenCache.cs b/TaskWebApp/Utils/MSALPerUserSessionTokenCache.cs
@@ -178,7 +178,11 @@ private void UserTokenCacheBeforeWriteNotification(TokenCacheNotificationArgs ar
 		/// <param name="args">Contains parameters used by the MSAL call accessing the cache.</param>
 		private void UserTokenCacheAfterAccessNotification(TokenCacheNotificationArgs args)
 		{
-			this.LoadUserTokenCacheFromSession();
+			// if the access operation resulted in a cache update
+			if (args.HasStateChanged)
+			{
+				this.PersistUserTokenCache();
+			}
 		}
 
 		/// <summary>
@@ -187,11 +191,7 @@ private void UserTokenCacheAfterAccessNotification(TokenCacheNotificationArgs ar
 		/// <param name="args">Contains parameters used by the MSAL call accessing the cache.</param>
 		private void UserTokenCacheBeforeAccessNotification(TokenCacheNotificationArgs args)
 		{
-			// if the access operation resulted in a cache update
-			if (args.HasStateChanged)
-			{
-				this.PersistUserTokenCache();
-			}
+			this.LoadUserTokenCacheFromSession();
 		}
 
 		/// <summary>

Original file line number	Diff line number	Diff line change
`@@ -178,7 +178,11 @@ private void UserTokenCacheBeforeWriteNotification(TokenCacheNotificationArgs ar`
`178`	`178`	`/// <param name="args">Contains parameters used by the MSAL call accessing the cache.</param>`
`179`	`179`	`private void UserTokenCacheAfterAccessNotification(TokenCacheNotificationArgs args)`
`180`	`180`	`{`
`181`		`- this.LoadUserTokenCacheFromSession();`
	`181`	`+ // if the access operation resulted in a cache update`
	`182`	`+ if (args.HasStateChanged)`
	`183`	`+ {`
	`184`	`+ this.PersistUserTokenCache();`
	`185`	`+ }`
`182`	`186`	`}`
`183`	`187`
`184`	`188`	`/// <summary>`
`@@ -187,11 +191,7 @@ private void UserTokenCacheAfterAccessNotification(TokenCacheNotificationArgs ar`
`187`	`191`	`/// <param name="args">Contains parameters used by the MSAL call accessing the cache.</param>`
`188`	`192`	`private void UserTokenCacheBeforeAccessNotification(TokenCacheNotificationArgs args)`
`189`	`193`	`{`
`190`		`- // if the access operation resulted in a cache update`
`191`		`- if (args.HasStateChanged)`
`192`		`- {`
`193`		`- this.PersistUserTokenCache();`
`194`		`- }`
	`194`	`+ this.LoadUserTokenCacheFromSession();`
`195`	`195`	`}`
`196`	`196`
`197`	`197`	`/// <summary>`