ModelState.AddModelError not displaying error in view - ASP.NET

Question

I have been banging my head against the wall trying to solve this issue.

I have this view controller

public class LoginController : Controller
    {
        public ActionResult Index(LoginClass model)
        {
            return View(model);
        }

        [HttpPost]
        public ActionResult Login(LoginClass model, string ReturnUrl)
        {

            if (!this.ModelState.IsValid)
            {
                return this.View(model);
            }

            if (ModelState.IsValid)
            {
                if (Membership.ValidateUser(model.UserName, model.Password))
                {
                    FormsAuthentication.SetAuthCookie(model.UserName, model.RememberMe);
                    if (Url.IsLocalUrl(ReturnUrl) && ReturnUrl.Length > 1 && ReturnUrl.StartsWith("/")
                        && !ReturnUrl.StartsWith("//") && !ReturnUrl.StartsWith("/\\"))
                    {
                       return Redirect(ReturnUrl);
                    }
                    else
                    {
                        return RedirectToAction("Index", "Home");
                    }
                }
                else
                {
                    ModelState.AddModelError(string.Empty, "The user name or password provided is incorrect");
                }


            }

            return RedirectToAction("Index", "Login", model);

        }

        public ActionResult Logout()
        {
            FormsAuthentication.SignOut();

            return RedirectToAction("Index", "Home");
        }

    }

and here is my view:

<section id="login">
    <div class="container">
        <form action="~/Login/Login" id="Login" method="post">
            <div class="row">
                <div class="col-md-12">
                    <p>
                        <label for="username">Username</label>
                        <input type="text" id="username" name="username" class="form-control" />
                    </p>
                </div>
            </div>
            <div class="row">
                <div class="col-md-12">
                    <p>
                        <label for="password">Password</label>
                        <input type="password" id="password" name="password" class="form-control" />
                    </p>
                </div>
            </div>
            <div class="row">
                <div class="col-md-12">
                    <p>
                        <input type="submit" id="submit" name="submit" value="Login" class="btn btn-default" />
                    </p>
                </div>
            </div>
            <div class="row">
                <div class="col-md-12">
                    <p>
                        <label for="password">Remember Me?</label>
                        <input type="checkbox" id="chkPersist" name="chkPersist" />
                    </p>
                </div>
            </div>
            <div class="row">
                <div class="col-md-12">
                    <p>
                        @Html.ValidationSummary()
                    </p>
                </div>
            </div>
        </form>
    </div>
</section>

My issue is that my error message is not appearing when I enter the wrong username and password. Why is not displaying?

what happens when you do @Html.ValidationSummary(false) – cableload Apr 05 '16 at 19:27 — cableload, Apr 05 '16 at 19:27
Same result, no error message appears – user979331 Apr 05 '16 at 19:48 — user979331, Apr 05 '16 at 19:48

cableload · Answer 1 · 2016-04-05T19:53:25.733

1

You need to add a placeholder by using the Html Helper function to display the message.

 <div>@Html.ValidationMesssage("KeyName")</div>

to your view so that you can display the validation message.

The KeyName comes from controller

  ModelState.AddModelError("KeyName", "The user name or password provided is incorrect");

Also you may need to ensure client side validation is enabled in your web.config.(usually they are)

 <appSettings>
   <add key="ClientValidationEnabled" value="true" />
   <add key="UnobtrusiveJavaScriptEnabled" value="true" />
 </appSettings>

edited Apr 05 '16 at 19:53

answered Apr 05 '16 at 19:22

cableload

4,215
5
36
62

what happens when you do @Html.ValidationSummary(false) – cableload Apr 05 '16 at 19:31
same result...nothing – user979331 Apr 05 '16 at 19:48
i guess another thing to check is whether jquery is available on that page? – cableload Apr 05 '16 at 19:51
jquery is available on the page and the items in my web config are there. – user979331 Apr 05 '16 at 20:01
i am not seeing anything wrong with your code. Can you try moving your @Html.ValidationSummary code line as given in the example here http://geekswithblogs.net/stun/archive/2011/01/28/aspnet-mvc-3-client-side-validation-summary-with-jquery-validation-unobtrusive-javascript.aspx – cableload Apr 05 '16 at 20:11

score 1 · Accepted Answer · edited May 23 '17 at 12:07

The problem is with the last line in your Login method. You are calling RedirectToAction instead of View. This has the consequence that you lose all your view specific state including the model state and validation errors that you built up in your Login Action. You can change your Login method like so (I simplified it a little), really the only change is replacing RedirectToAction("Index", "Login", model) with View(model) on the last line.

If you do want to redirect in the event of authentication failure and you did want to use RedirectToAction then see my other answer I posted here.

[HttpPost]
public ActionResult Login(LoginClass model, string ReturnUrl)
{
    if (!this.ModelState.IsValid)
        return this.View(model);

    if (Membership.ValidateUser(model.UserName, model.Password))
    {
        FormsAuthentication.SetAuthCookie(model.UserName, model.RememberMe);

        if (Url.IsLocalUrl(ReturnUrl) && ReturnUrl.Length > 1 && ReturnUrl.StartsWith("/")
            && !ReturnUrl.StartsWith("//") && !ReturnUrl.StartsWith("/\\"))
        {
           return Redirect(ReturnUrl);
        }
        else
        {
            return RedirectToAction("Index", "Home");
        }
    }
    ModelState.AddModelError(string.Empty, "The user name or password provided is incorrect");
    return this.View(model); // return to the same view to show your error message

    // return RedirectToAction("Index", "Login", model); // do not redirect
}

ModelState.AddModelError not displaying error in view - ASP.NET

2 Answers2