UpdateAsync does not save the data - asp.net

Question

I have edit page, which should to edit data of IdentityUser. On this page, I have a form with method="post" and also I have a controller, that have a method with [HttpPost] attribute. So, I try to use _userManager.UpdateAsync(user) but unfortunately it does not work. I have checked on succeeded updating user data, but I have no idea why data not updates. Below you will see the code of View and Controllers.
Edit View
#model BookStore.ViewModels.EditViewModel
#{
ViewData["Title"] = "Edit Info";
Layout = "~/Views/Shared/_Layout.cshtml";
}
<form method="post" asp-controller="Profile" asp-action="EditInfo">
<h3 class="registerTitle">Edit Data</h3>
<hr />
<div asp-validation-summary="ModelOnly" class="text-danger"></div>
<div class="form-group">
<div><label asp-for="Email"></label></div>
<input asp-for="Email" placeholder="example#mail.com" />
<span class="text-danger" asp-validation-for="Email"></span>
</div>
<!--Birthday inputs-->
<div class="form-group">
<div>
<label type="text" asp-for="birthDate"></label>
</div>
#Html.DropDownList("Month", Enumerable.Range(1, 12).Select(i => new SelectListItem { Value = i.ToString(), Text = System.Globalization.CultureInfo.InvariantCulture.DateTimeFormat.GetMonthName(i) }), "Select Month")
#Html.DropDownList("Day", Enumerable.Range(1, 31).Select(i => new SelectListItem { Value = i.ToString(), Text = i.ToString() }), "Select Day")
#Html.DropDownList("Year", Enumerable.Range(1900, 109).Select(i => new SelectListItem { Value = i.ToString(), Text = i.ToString() }), "Select Year")
</div>
<!--First Name input-->
<div class="form-group">
<div>
<label type="text" asp-for="FirstName"></label>
</div>
<input type="text" asp-for="FirstName" placeholder="First Name" />
<span class="text-danger" asp-validation-for="FirstName"></span>
</div>
<!--Last Name input-->
<div class="form-group">
<div>
<label type="text" asp-for="LastName"></label>
</div>
<input type="text" asp-for="LastName" placeholder="First Name" />
<span class="text-danger" asp-validation-for="LastName"></span>
</div>
<!--User Name input-->
<div class="form-group">
<div>
<label type="text" asp-for="UserName"></label>
</div>
<input type="text" asp-for="UserName" placeholder="Some_username" />
<span class="text-danger" asp-validation-for="UserName"></span>
</div>
<div class="row">
<div class="col-md"></div>
<div class="col-md"><input type="submit" value="Save"/></div>
<div class="col-md"></div>
</div>
</form>
ProfileController
[Route("Info/Edit/{username}")]
[HttpGet]
public async Task<IActionResult> Edit(string username)
{
var user = await _userManager.FindByNameAsync(username);
EditViewModel edit = new EditViewModel()
{
FirstName = user.FirstName,
LastName = user.LastName,
Month = user.birthDate.Month,
Year = user.birthDate.Year,
Day = user.birthDate.Day,
Email = user.Email,
UserName = user.UserName
};
return View(edit);
}
[HttpPost]
[ValidateAntiForgeryToken]
public async Task<IActionResult> EditInfo(EditViewModel editModel)
{
var user = await _userManager.FindByNameAsync(User.Identity.Name);
if (user == null) return BadRequest("user is not found");
user.FirstName = editModel.FirstName;
user.LastName = editModel.LastName;
user.UserName = editModel.UserName;
user.Year = editModel.birthDate.Year;
user.Month = editModel.birthDate.Month;
user.Day = editModel.birthDate.Day;
user.Email = editModel.Email;
IdentityResult result = await _userManager.UpdateAsync(user);
if (result.Succeeded)
{
return Content("Ok");
}
return View(editModel);
}
EditViewModel class
public class EditViewModel
{
[EmailAddress]
public string Email { get; set; }
[Display(Name = "Username")]
public string UserName { get; set; }
[Display(Name = "First Name")]
public string FirstName { get; set; }
[Display(Name = "Last Name")]
public string LastName { get; set; }
[Display(Name = "Date of birth")]
[DataType(DataType.Date)]
public DateTime birthDate
{
get
{
return new DateTime(Year,Month,Day);
}
}
public int Day { get; set; }
public int Month { get; set; }
public int Year { get; set; }
}

Answer 1

Explicitly add an antiforgery token to a form element
<form asp-action="EditInfo" asp-controller="Profile" method="post">
#Html.AntiForgeryToken()
<!-- ... -->
</form>
and fix the action code, return badrequest if user is null
var user = await _userManager.FindByNameAsync(editModel.UserName);
if(user == null) return BadRequest("user is not found");
user.FirstName = editModel.FirstName;
....
var result = await _userManager.UpdateAsync(user);
if (result.Succeeded)
{
return Content("Ok");
}
......
and I can see some bugs in the view too. For example
#Html.DropDownList("Day", Enumerable.Range(1, 31).Select(i => new SelectListItem { Value = i.ToString(), Text = i.ToString() }), "Select Day")
should be
#Html.DropDownListFor(model=> Model.Day, Enumerable.Range(1, 31).Select(i => new SelectListItem { Value = i.ToString(), Text = i.ToString() }), "Select Day")
I can see in the post action
user.Day = editModel.birthDate.Day;
it should be
user.Day = editModel.Day;
but i see this in Get action
editModel.Month = user.birthDate.Month,
I am amazed how you could compile it.