Why phone number update is done through userManager

[snax4a]

Hello,

I am wondering why in the userService.UpdateAsync method we are doing the following:
https://github.com/fullstackhero/dotnet-webapi-boilerplate/blob/main/src/Infrastructure/Identity/UserService.CreateUpdate.cs#L172

user.FirstName = request.FirstName;
user.LastName = request.LastName;
user.PhoneNumber = request.PhoneNumber;
string phoneNumber = await _userManager.GetPhoneNumberAsync(user);
if (request.PhoneNumber != phoneNumber)
{
    await _userManager.SetPhoneNumberAsync(user, request.PhoneNumber);
}

Why the phone number is retrieved and set through userManager?
The field update is done like other fields user.PhoneNumber = request.PhoneNumber; so why do we need this usermanager get and set methods here?

[DevJr14]

@snax4a This logic it ensures that you don't update same phone number if given that they were previous saved under that user.

[snax4a]

The if statement does that but before that, the field is updated using user.PhoneNumber = request.PhoneNumber so this logic makes no sense.
I think these lines should be removed

string phoneNumber = await _userManager.GetPhoneNumberAsync(user);
if (request.PhoneNumber != phoneNumber)
{
    await _userManager.SetPhoneNumberAsync(user, request.PhoneNumber);
}

[DevJr14]

Oh i get what you mean, for me personally i don't have problem using _userManager api method.

[snax4a]

@fretje can you take a look, please?

[fretje]

Sorry I wouldn't know why that's there... Maybe @iammukeshm knows? But I guess you're right yes... I suppose you've tested your theory? ;-)