From 39fb99636eac3482222ab1d5f0b410b46093f296 Mon Sep 17 00:00:00 2001 From: Robin Herbots Date: Wed, 30 Nov 2016 13:13:57 +0100 Subject: [PATCH] The extra call to update is redundant as an update is already called by SetUserProperty Solves issue IdentityManager/IdentityManager/#133 in identitymanager --- source/AspNetIdentity/AspNetIdentityManagerService.cs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/source/AspNetIdentity/AspNetIdentityManagerService.cs b/source/AspNetIdentity/AspNetIdentityManagerService.cs index 0dc6b4c..d8ed352 100644 --- a/source/AspNetIdentity/AspNetIdentityManagerService.cs +++ b/source/AspNetIdentity/AspNetIdentityManagerService.cs @@ -539,11 +539,11 @@ public virtual async Task SetUserPropertyAsync(string sub return propResult; } - var result = await userManager.UpdateAsync(user); - if (!result.Succeeded) - { - return new IdentityManagerResult(result.Errors.ToArray()); - } + //var result = await userManager.UpdateAsync(user); + //if (!result.Succeeded) + //{ + // return new IdentityManagerResult(result.Errors.ToArray()); + //} return IdentityManagerResult.Success; }