Delete user from default mvc application database

Question

I am trying to delete an already existing user from a database, which was created automatically when creating MVC application.
The database consists of tables:

AspNetUsers
AspNetUserRoles
AspNetUserLogins
AspNetUserClaims
AspNetRoles

In my code it looks like this:

var user = new ApplicationUser { UserName = model.email, Email = model.email };
var context = new ApplicationDbContext();
context.Users.Attach(user);
context.Users.Remove(user);
context.SaveChangesAsync();
return RedirectToAction("OperationSuccess", "Account");

I have also tried this:

var user = new ApplicationUser { UserName = model.email, Email = model.email };
var context = new ApplicationDbContext();
UserManager.DeleteAsync(user);

But it doesn't help at all. The application itselt does not break and does not show any errors, but the user is still in the database. How do I delete it?

What is the IdentityResult? *var identityResult = await UserManager.DeleteAsync(user);* There should be information in the Errors property: https://msdn.microsoft.com/en-us/library/microsoft.aspnet.identity.identityresult(v=vs.108).aspx — , May 28 '17 at 19:19

score 1 · Accepted Answer · answered May 28 '17 at 15:15

1

Try this code:

public async Task<IdentityResult> DeleteUser(string email)
{
    var user = UserManager.Users.FirstOrDefault(x => x.Email == email);
    if(user == null) return null;
    var result = await UserManager.DeleteAsync(user); //here result has two properties Errors and Succeeded.
    return result;
}

Also, your code is not working because you are creating the object yourself and assigning only two properties yourself in spite of fetching the data from database.

answered May 28 '17 at 15:15

Prateek Pandey

833
6
19

Thank you, but unfortunately, this does not make a change in the database. – Maciej Miśkiewicz May 28 '17 at 15:27
have you checked the result variable to check for errors? – Prateek Pandey May 28 '17 at 15:28
`- result Id = 415, Status = WaitingForActivation, Method = "{null}", Result = "{Not yet computed}" System.Threading.Tasks.Task` And exeption is null. @prateek-pandey – Maciej Miśkiewicz May 28 '17 at 15:46
Need to add result `var result = DeleteUser(email).Result` – Liran May 28 '17 at 16:04

score 0 · Answer 2 · answered May 29 '17 at 01:32

0

Hi I think you have some versioning problem and its seems that you need to give one extra paramater to the DeleteAsync method.

Kindly refer the below link, since they had same kind of issue and resolved it.

https://stackoverflow.com/a/24594440/3397630

Hope it may give you some idea for your solution too.

Thanks

Karthik

answered May 29 '17 at 01:32

Karthik Elumalai

1,574
1
11
12

What extra parameter? – May 29 '17 at 10:19

Jinto John · Answer 3 · 2019-02-22T07:00:24.230

Hope below code will help you to fix your problem

[HttpPost]

    public async Task<ActionResult> Delete(string userId)
    {
        // Check for for both ID and exit if not found
        if (String.IsNullEmpty(userId))
        {
            return new HttpStatusCodeResult(HttpStatusCode.BadRequest);
        }  
        var user = UserManager.Users.SingleOrDefault(u => u.Id == Userid);

// Look for user in the UserStore

        // If not found, exit
        if (user == null)
        {
            return HttpNotFound();
        }



            var results = await UserManager.DeleteAsync(user); // Remove user from UserStore

            // If the statement is a success

            if (results.Succeeded)
            {
                // Redirect to Users page
                return RedirectToAction("Index", "Users");
            }
            else
            {
                return new HttpStatusCodeResult(HttpStatusCode.BadRequest);
            }       
        }

Delete user from default mvc application database

3 Answers3