Multiple users invitation fix (#22)

VirtoCommerce · Aug 17, 2022 · 07b5f70 · 07b5f70
1 parent 3b82d77
commit 07b5f70
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/VirtoCommerce.ProfileExperienceApiModule.Data/Commands/InviteUserCommandHandler.cs b/src/VirtoCommerce.ProfileExperienceApiModule.Data/Commands/InviteUserCommandHandler.cs
@@ -47,23 +47,23 @@ public InviteUserCommandHandler(
 
         public virtual async Task<IdentityResultResponse> Handle(InviteUserCommand request, CancellationToken cancellationToken)
         {
-            using var userManager = _userManagerFactory();
-
             var result = new IdentityResultResponse
             {
                 Errors = new List<IdentityErrorInfo>(),
-                Succeeded = true,
+                Succeeded = false,
             };
 
             // PT-6083: reduce complexity
             foreach (var email in request.Emails)
             {
+                using var userManager = _userManagerFactory();
+
                 var contact = new Contact { FirstName = string.Empty, LastName = string.Empty, FullName = string.Empty, Organizations = new List<string> { request.OrganizationId } };
                 await _memberService.SaveChangesAsync(new Member[] { contact });
 
                 var user = new ApplicationUser { UserName = email, Email = email, MemberId = contact.Id, StoreId = request.StoreId };
                 var identityResult = await userManager.CreateAsync(user);
-
+                
                 if (identityResult.Succeeded)
                 {
                     var store = await _storeService.GetByIdAsync(user.StoreId);
@@ -87,7 +87,7 @@ public virtual async Task<IdentityResultResponse> Handle(InviteUserCommand reque
                 }
 
                 result.Errors.AddRange(identityResult.Errors.Select(x => x.MapToIdentityErrorInfo()));
-                result.Succeeded &= identityResult.Succeeded;
+                result.Succeeded |= identityResult.Succeeded;
 
                 if (!result.Succeeded)
                 {