自定义验证器: :创建不在轨道上运行的应用程序

Question

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

卷.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的记录。我只想在创建新卷时运行它，以便我仍然可以更新现有卷。但是，将：：create添加到CustomVolumeValidator会导致验证器停止工作。

我已经阅读了许多其他关于类似问题的条目，但没有找到解决方案。我很确定我错过了一些关于何时创建、验证和保存不同属性的信息，但我没有过多地使用自定义验证，而且我迷路了。

这是其他相关代码。

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

Answer 1

如果您的目标是让所有Volume记录都具有唯一的text_digest，则最好使用一个简单的:uniqueness验证器(以及关联的数据库唯一索引)。

但是，现有代码不起作用的原因是：

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

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

你可以通过将first_or_create替换为create来解决，但同样，使用唯一的索引和验证器(如果你愿意，可以使用自定义消息)会好得多。