Сообщения об ошибках валидации Rails: отображение только одного сообщения об ошибке на поле

Question

Rails отображает все сообщения об ошибках валидации, связанные с данным полем. Если у меня есть три validates_XXXXX_of :email, и я оставляю поле пустым, я получаю три сообщения в списке ошибок.

Пример:

validates_presence_of :name
validates_presence_of :email
validates_presence_of :text

validates_length_of :name, :in => 6..30
validates_length_of :email, :in => 4..40
validates_length_of :text, :in => 4..200

validates_format_of :email, :with => /^([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})$/i<br/>

<%= error_messages_for :comment %> дает мне:

7 errors prohibited this comment from being saved

There were problems with the following fields:

Name can't be blank
Name is too short (minimum is 6 characters)
Email can't be blank
Email is too short (minimum is 4 characters)
Email is invalid
Text can't be blank
Text is too short (minimum is 4 characters)

Лучше отображать по одному сообщению за раз. Есть ли простой способ решить эту проблему? Это выглядит просто, если иметь условие типа: Если вы нашли ошибку для :email, прекратите проверку :email и перейдите к другому полю.

Answer 1

[Обновить] Январь / 2013 до Rails 3.2.x - обновить синтаксис; добавить спецификацию

Вдохновленный новыми проверочными методами в Rails 3.0 Я добавляю этот крошечный Validator. Я называю это ReduceValidator.

lib/reduce_validator.rb:

# show only one error message per field
#
class ReduceValidator < ActiveModel::EachValidator

  def validate_each(record, attribute, value)
    return until record.errors.messages.has_key?(attribute)
    record.errors[attribute].slice!(-1) until record.errors[attribute].size <= 1
  end
end

Моя модель выглядит как - обратите внимание на :reduce => true:

validates :title, :presence => true, :inclusion => { :in => %w[ Mr Mrs ] }, :reduce => true
validates :firstname, :presence => true, :length => { :within => 2..50 }, :format => { :without => /^\D{1}[.]/i }, :reduce => true
validates :lastname, :presence => true, :length => { :within => 2..50 }, :format => { :without => /^\D{1}[.]/i }, :reduce => true

Работает как шарм в моем текущем Rails Project. Преимущество заключается в том, что я установил валидатор только на несколько полей, а не на все.

spec/lib/reduce_validator_spec.rb

require 'spec_helper'

describe ReduceValidator do

  let(:reduce_validator) { ReduceValidator.new({ :attributes => {} }) }

  let(:item) { mock_model("Item") }
  subject { item }

  before(:each) do
    item.errors.add(:name, "message one")
    item.errors.add(:name, "message two")
  end

  it { should have(2).error_on(:name) }

  it "should reduce error messages" do
    reduce_validator.validate_each(item, :name, '')
    should have(1).error_on(:name)
  end

end

Answer 2

Imo проще:

<% @model.errors.each do |attr, msg| %>
  <%= "#{attr} #{msg}" if @model.errors[attr].first == msg %> 
<% end %>

Answer 3

Берт на RailsForum писал об этом некоторое время назад. Он написал код ниже, и я добавил несколько небольших настроек для его запуска на Rails-3.0.0-beta2.

Добавьте это в файл с именем app/helpers/errors_helper.rb и просто добавьте helper "errors" в свой контроллер.

module ErrorsHelper

  # see: lib/action_view/helpers/active_model_helper.rb
  def error_messages_for(*params)
        options = params.extract_options!.symbolize_keys

        objects = Array.wrap(options.delete(:object) || params).map do |object|
          object = instance_variable_get("@#{object}") unless object.respond_to?(:to_model)
          object = convert_to_model(object)

          if object.class.respond_to?(:model_name)
            options[:object_name] ||= object.class.model_name.human.downcase
          end

          object
        end

        objects.compact!
        count = objects.inject(0) {|sum, object| sum + object.errors.count }

        unless count.zero?
          html = {}
          [:id, :class].each do |key|
            if options.include?(key)
              value = options[key]
              html[key] = value unless value.blank?
            else
              html[key] = 'errorExplanation'
            end
          end
          options[:object_name] ||= params.first

          I18n.with_options :locale => options[:locale], :scope => [:errors, :template] do |locale|
            header_message = if options.include?(:header_message)
              options[:header_message]
            else
              locale.t :header, :count => count, :model => options[:object_name].to_s.gsub('_', ' ')
            end

            message = options.include?(:message) ? options[:message] : locale.t(:body)

            error_messages = objects.sum do |object|
              object.errors.on(:name)
              full_flat_messages(object).map do |msg|
                content_tag(:li, ERB::Util.html_escape(msg))
              end
            end.join.html_safe

            contents = ''
            contents << content_tag(options[:header_tag] || :h2, header_message) unless header_message.blank?
            contents << content_tag(:p, message) unless message.blank?
            contents << content_tag(:ul, error_messages)

            content_tag(:div, contents.html_safe, html)
          end
        else
          ''
        end
  end

  ####################
  #
  # added to make the errors display in a single line per field
  #
  ####################
  def full_flat_messages(object)
    full_messages = []

    object.errors.each_key do |attr|
      msg_part=msg=''
      object.errors[attr].each do |message|
        next unless message
        if attr == "base"
          full_messages << message
        else
          msg=object.class.human_attribute_name(attr)
          msg_part+= I18n.t('activerecord.errors.format.separator', :default => ' ') + (msg_part=="" ? '': ' & ' ) + message
        end
      end
      full_messages << "#{msg} #{msg_part}" if msg!=""
    end
    full_messages
  end

end

Answer 4

Я написал пользовательский помощник

def display_error(field)
    if @user.errors[field].any?
        raw @user.errors[field].first+"<br>"
    end
end

, а затем я использую его в поле текста ниже так:

<%= display_error(:password) %>

Answer 5

Как насчет этого @event.errors[:title].first?

Answer 6

Аналогично ответу olovwia :

<% @errors.keys.each do |attr| %>
 <%= "#{attr.capitalize} #{@errors[attr].first}."%>
<% end %>"

Answer 7

Я использую этот код для выпуска Ruby on Rails 3.0, который я вставил в lib/core_ext/rails/active_model/errors.rb:

module ActiveModel
  class Errors
    def full_message_per_field
      messages_per_field = []
      handled_attributes = []

      each do |attribute, messages|
        next if handled_attributes.include? attribute
        messages = Array.wrap(messages)
        next if messages.empty?

        if attribute == :base
          messages_per_field << messages.first
        else
          attr_name = attribute.to_s.gsub('.', '_').humanize
          attr_name = @base.class.human_attribute_name(attribute, :default => attr_name)
          options = { :default => "%{attribute} %{message}", :attribute => attr_name }

          messages_per_field << I18n.t(:"errors.format", options.merge(:message => messages.first))
        end

        handled_attributes << attribute
      end

      messages_per_field
    end
  end
end

По сути, это тот же код, что и ActiveModel::Errors#full_messages, но он не будет отображать более одной ошибки на атрибут. Обязательно запрашивайте файл (скажем, в инициализаторе), и теперь вы можете позвонить @model.errors.full_message_per_field do |message| ...

Answer 8

Я бы отображал все сообщения об ошибках в одной строке и в формате предложения. Вы не хотите, чтобы пользователь исправлял одну ошибку и заканчивал тем, что имел другую ошибку, о которой он не знал после отправки. Рассказывая им все правила, вы сохраните их клики. С учетом сказанного, вот как я это сделаю:

flash_message_now("error", 
   @album.errors.keys.map { |k| "#{Album.human_attribute_name(k)} #{@album.errors[k].to_sentence}"}.to_sentence
)

с flash_message_now, определенным в ApplicationController (вы можете добавить его в помощник)

def flash_message_now(type, text)
    flash.now[type] ||= []
    flash.now[type] << text
  end

Answer 9

Добавление метода в ActiveModel :: Класс ошибок

module ActiveModel
  class Errors
    def full_unique_messages
      unique_messages = messages.map { |attribute, list_of_messages| [attribute, list_of_messages.first] }
      unique_messages.map { |attribute_message_pair| full_message *attribute_message_pair }
    end
  end
end

Добавьте его в файл, например lib/core_ext/rails/active_model/errors.rb. Создайте файл config/initializers/core_ext.rb и добавьте к нему require "core_ext/rails/active_model/errors.rb".

Answer 10

Мой патч обезьяны ActiveModel::Errors класса lib/core_ext/rails/active_model/errors.rb (я использую этот код для выпуска Ruby on Rails 5.0):

module ActiveModel
  class Errors

    # don't add an attribute's error message to details
    # if it already contains at least one message

    alias_method :old_add, :add

    def add(attribute, message = :invalid, options = {})
      if details[attribute.to_sym].size.zero?
        old_add(attribute, message, options)
      end
    end

  end
end

Создайте файл config/initializers/core_ext.rb и добавьте к нему требование core_ext/rails/active_model/errors.rb.