Small Error in AccountController ConfirmEmail

[eggbeard]

I think the logic for ConfirmEmail got messed up when changing the null check of string to a > 0 check for an int.

It should be something like:

if(userId > 0 && code != null){
//Do success stuff
}
//Do Error stuff

[xivSolutions]

Yup. I must have missed that one.

Wanna fix it and shoot me a PR?

[fouimette]

(While waiting for @eggbeard to submit a PR)...

So it should look like this?

if (userId > 0 && code != null)
{
    //success
    var result = await UserManager.ConfirmEmailAsync(userId, code);
    return View(result.Succeeded ? "ConfirmEmail" : "Error");            
}
else
{
    //error
    return View("Error");
}

[xivSolutions]

@fouimette Looks right - I can;t check it right now. If you wanna check it, make sure it works, and shoot a PR I'll merge. Much appreciated, deep in day-job-related BS right now...