自定义验证器位于：：create不在Rails应用程序上运行

如何解决自定义验证器位于：：create不在Rails应用程序上运行

我有一个用于创建卷的Rails应用程序，并使用ActiveModel :: Validator编写了两个自定义验证器。

volume.rb：

    class Volume < ActiveRecord::Base
      include UrlSafeCode
      include PgSearch::Model
      include ActiveModel::Validations
      validates :user_id,presence: true
      validates_with Validators::VolumeValidator
      validates_with Validators::CreateVolumeValidator,on: :create

    def self.digest text
      Digest::SHA256.hexdigest(text)
    end

    def text=(new_text)
      new_text.rstrip!
      new_text.downcase!
      self.text_digest = Volume.digest(new_text)
      super(new_text)
    end

我的问题：CreateVolumeValidator检查数据库中是否已存在具有相同text_digest的记录。我只想在创建新卷时运行此程序，以便仍可以更新现有卷。但是，在CustomVolumeValidator上添加：create会导致验证器停止工作。

我已经阅读了许多其他有关类似问题的文章，但没有找到解决方案。我很确定我丢失了有关何时创建，验证和保存不同属性的信息，但是我对定制验证的工作还不够多，我迷路了。

这是其他相关代码。

volumes_controller.rb

  def new
    @volume = Volume.new
  end

  def create
    our_params = params
      .permit(:text,:description)

    if params[:text].nil?
      render :retry
      return
    end

    text = params[:text].read.to_s
    text_digest = Volume.digest(text)   

    @description = our_params[:description]

    begin
      @volume = Volume.where(text_digest: text_digest)
        .first_or_create(text: text,user: current_user,description: our_params[:description])
    rescue ActiveRecord::RecordNotUnique
      retry
    end

    if @volume.invalid?
      render :retry
      return
    end

    render :create

  end

  def edit
    get_volume
  end

  def update

    get_volume
    unless @volume
      render nothing: true,status: :not_found
      return
    end

    @volume.update(params.require(:volume).permit(:text,:description))

    if @volume.save
      redirect_to volume_path(@volume.code)
    else
      flash[:notice] = @volume.errors.full_messages.join('\n')
      render :edit
    end

  end

  def get_volume
    @volume = Volume.where(code: params.require(:code)).first
  end

create_volume_validator.rb

class Validators::CreateVolumeValidator < ActiveModel::Validator
    def validate(volume)
        existing_volume = Volume.where(text_digest: volume.text_digest).first
        if existing_volume 
          existing_volume_link = "<a href='#{Rails.application.routes.url_helpers.volume_path(existing_volume.code)}'>here</a>."
          volume.errors.add :base,("This volume is already part of the referral archive and is available " + existing_volume_link).html_safe
        end
    end
end

解决方法

如果您的目标是使所有Volume记录都具有唯一的text_digest，那么最好使用简单的:uniqueness验证程序（以及关联的数据库唯一索引）。

但是，现有代码无法正常工作的原因是：

Volume.where(text_digest: text_digest).first_or_create(...)

这将返回具有匹配的Volume的第一个text_digest或创建一个新的{p}。但这意味着如果发生冲突，则不会创建任何对象，因此您的（on: :create）验证将不会运行。相反，它只是将@volume设置为现有对象，根据定义，该对象是有效的。如果没有匹配的记录，它会执行调用您的验证程序，但是没有什么要验证的，因为您已经证明不存在text_digest冲突。

您可以通过将first_or_create替换为create来解决，但再次，您可以使用唯一的索引和验证器（如果需要，可以使用自定义消息）来获得更好的收益。