так согласно GuIValidatableObject.Validate()
должен вызываться, когда контроллер проверяет это модель (т.е. до ModelState.IsValid
), однако просто сделать модель реализации IValidatableObject
, кажется, не работает, потому что Validate(..)
не дозвонились ,ModelState.IsValid против IValidateableObject в MVC3
Кто-нибудь знает, есть ли что-то еще, что я должен подключить, чтобы заставить это работать?
EDIT:
Вот код, в соответствии с просьбой.
public class LoginModel : IValidatableObject
{
[Required]
[Description("Email Address")]
public string Email { get; set; }
[Required]
[Description("Password")]
[DataType(DataType.Password)]
public string Password { get; set; }
[DisplayName("Remember Me")]
public bool RememberMe { get; set; }
public int UserPk { get; set; }
public IEnumerable<ValidationResult> Validate(ValidationContext validationContext)
{
var result = DataContext.Fetch(db => {
var user = db.Users.FirstOrDefault(u => u.Email == Email);
if (user == null) return new ValidationResult("That email address doesn't exist.");
if (user.Password != User.CreateHash(Password, user.Salt)) return new ValidationResult("The password supplied is incorrect.");
UserPk = user.UserPk;
return null;
});
return new List<ValidationResult>(){ result };
}
}
Действие. (Я ничего особенного не делать в контроллере ...)
[HttpPost]
public ActionResult Login(LoginModel model)
{
if (ModelState.IsValid)
{
FormsAuthentication.SetAuthCookie(model.Email, model.RememberMe);
return Redirect(Request.UrlReferrer.AbsolutePath);
}
if (ControllerContext.IsChildAction || Request.IsAjaxRequest())
return View("LoginForm", model);
return View(model);
}
я поставил точку останова на первой линии LoginModel.Validate()
и не кажется, чтобы получить удар.
Вы код выглядит просто отлично. Точно так, как должно. Просто точка интереса, но у вас есть дублирующая модель? Я знаю, что у меня есть модель представления и модель db для каждого объекта. Может ли ваш контроллер ссылаться на неправильную модель? – Buildstarted
Кроме того, в качестве примечания стороны: вы обязательно должны вернуть только одну ошибку, если имя пользователя или пароль недействительны и не являются отдельными ошибками. Это просто для безопасности, так как я могу проверить каждое поле отдельно, чтобы найти имя пользователя, а затем работать с паролем для этого пользователя. Это не требуется, но это хорошая идея :) – Buildstarted
Вы можете использовать 'yield return DataContext ...' вместо того, чтобы возвращать новый список. Это будет красивее и быстрее. – pipedreambomb