ErrorMessage игнорируется в атрибуте DataAnnotations DataType.

У меня есть модель, использующая DataAnnotations. Что-то типа

public class Appointment {
    [Required(ErrorMessage="Please enter your name")]
    public string Name { get; set; }

    [Required(ErrorMessage="Please enter your appointment date?")] 
    [DataType(DataType.Date, ErrorMessage="Appointment date is not a date")]
    public DateTime AppointmentDate { get; set; }
}

Атрибуты «Обязательные» соответствуют значению в ErrorMessage; то есть, если я не введу значение, я получаю сообщение «Пожалуйста, введите». Однако, если я введу строку в поле DateTime, я получаю стандартное системное сообщение об ошибке «Значение 'blah' недействительно для AppointmentDate».

Я отлаживал код ASP.NET MVC, и кажется, что в случае FormatException он не выбирает правильное отображаемое имя из propertyMetadata. Либо это, либо мне не хватает чего-то явно очевидного: /

Кто-нибудь сталкивался с этой проблемой? Это я или просто бета (я использую ASP.NET MVC 2 Beta)?


error-handling asp.net-mvc data-annotations
person Felix    schedule 23.11.2009    source источник

Ответы (2)


arrow_upward
2
arrow_downward

Атрибут DataType в MVC1 не делает того, что вы ожидаете, похоже, что он не работает и в MVC2. Лучше всего иметь строковое свойство, представляющее дату, проверьте его действительность там.

Вот небольшой отрывок из проекта (с использованием DataAnnotations и xVal):

private List<ErrorInfo> _errors;
        private List<ErrorInfo> Errors
        {
            get
            {
                if (_errors == null)
                    _errors = new List<ErrorInfo>();
                return _errors;
            }
            //set { _errors = value; }
        }

private string _startDateTimeString;

        /// <summary>
        /// A string reprsentation of the StartDateTime, used for validation purposes in the views.
        /// </summary>
        /// <remarks>
        /// If the user passes something like 45/45/80 it would be a valid mm/dd/yy format, but an invalid date,
        /// which would cause an InvalidOperationException to be thrown by UpdateModel(). By assigning dates to string properties
        /// we can check not only the format, but the actual validity of dates.
        /// </remarks>
        public string StartDateTimeString
        {
            get
            {
                return _startDateTimeString;
            }
            set
            {
                // Check whether the start date passed from the controller is a valid date.
                DateTime startDateTime;
                bool validStartDate = DateTime.TryParse(value, out startDateTime);
                if (validStartDate)
                {
                    StartDateTime = startDateTime;
                }
                else
                {
                    Errors.Add(new ErrorInfo("StartDateTime", "Provide a valid date for the start time."));
                }

                _startDateTimeString = value;
            }
        }

        partial void OnValidate(ChangeAction action)
        {
            if (action != ChangeAction.Delete)
            {
                Errors.AddRange(DataAnnotationsValidationRunner.GetErrors(this));

                if (StartDateTimeString != null)
                {
                    DateTime startDateTime;
                    if (!DateTime.TryParse(StartDateTimeString, out startDateTime))
                    {
                        Errors.Add(new ErrorInfo("StartDateTime", "Provide a valid date for the start time."));
                    }
                }

                if (Errors.Any())
                    throw new RulesException(Errors);
            }
        }
    }

Имеет смысл иметь чек в обоих местах в нашем проекте, но я просто хочу показать вам концепцию.

person Pawel Krakowiak    schedule 24.11.2009

arrow_upward
1
arrow_downward

Сегодня я столкнулся с этой проблемой, и я хотел поделиться другим решением ...

[Required(ErrorMessage="Please enter your appointment date?")] 
//[DataType(DataType.Date, ErrorMessage="Appointment date is not a date")]
[Range(typeof(DateTime), "1/1/1880", "1/1/2200", ErrorMessage = "...")]
public string AppointmentDate { get; set; }

Вам нужно будет настроить свой код для работы со строкой вместо даты и времени (по-видимому, легко, если это часть вашей модели представления), но сообщение об ошибке работает так, как ожидалось, и строка гарантированно будет действительной датой (возможно, с время).

person Mayo    schedule 08.10.2010