EmailConfirmed and PhoneNumberConfirmed are set to false after updating a user

[gdlcf88]

When updating a user with identity-management, if user's EmailConfirmed and PhoneNumberConfirmed dont change, should them keep original value?

abp/modules/identity/src/Volo.Abp.Identity.Application/Volo/Abp/Identity/IdentityUserAppService.cs

Line 128 in 79cc798

private async Task UpdateUserByInput(IdentityUser user, IdentityUserCreateOrUpdateDtoBase input)

[hikalkan]

@maliming can you test this and fix if there is a bug please?

[maliming]

This is actually the design of the Identity.
https://github.com/aspnet/AspNetCore/blob/bfec2c14be/src/Identity/Extensions.Core/src/UserManager.cs#L1389

[gdlcf88]

What if dont set the un-changed Email/PhoneNumber value for keeping their confirmed status?

abp/modules/identity/src/Volo.Abp.Identity.Application/Volo/Abp/Identity/IdentityUserAppService.cs

Line 130 in 79cc798

(await _userManager.SetEmailAsync(user, input.Email)).CheckErrors();

[maliming]

You can override the related methods. I will take the time to figure out the design logic of identity.

[maliming]

Well, maybe we should not call the SetEmailAsync method if the email has not changed.