Validates_uniqueness_of в разрушенных вложенных модельных рельсах

У меня есть модель Project, которая принимает вложенные атрибуты для Task.

class Project < ActiveRecord::Base  
  has_many :tasks

  accepts_nested_attributes_for :tasks, :allow_destroy => :true

end

class Task < ActiveRecord::Base  
validates_uniqueness_of :name end

Валидация уникальности в модели задачи дает проблему при обновлении проекта.

При редактировании проекта я удаляю задачу T1, а затем добавляю новую задачу с тем же именем T1, проверка уникальности ограничивает сохранение проекта.

params hash выглядит примерно как

task_attributes => { {"id" =>
"1","name" => "T1", "_destroy" =>
"1"},{"name" => "T1"}}

Проверка выполнения задачи выполняется до уничтожения старой задачи. Следовательно, проверка не выполняется. Любая идея, как проверить, чтобы она не рассматривала задачу, которую нужно уничтожить?

Ответ 1

Andrew France создал патч в этом thread, где проверка выполняется в памяти.

class Author
  has_many :books

  # Could easily be made a validation-style class method of course
  validate :validate_unique_books

  def validate_unique_books
    validate_uniqueness_of_in_memory(
      books, [:title, :isbn], 'Duplicate book.')
  end
end

module ActiveRecord
  class Base
    # Validate that the the objects in +collection+ are unique
    # when compared against all their non-blank +attrs+. If not
    # add +message+ to the base errors.
    def validate_uniqueness_of_in_memory(collection, attrs, message)
      hashes = collection.inject({}) do |hash, record|
        key = attrs.map {|a| record.send(a).to_s }.join
        if key.blank? || record.marked_for_destruction?
          key = record.object_id
        end
        hash[key] = record unless hash[key]
        hash
      end
      if collection.length > hashes.length
        self.errors.add_to_base(message)
      end
    end
  end
end

Ответ 2

Как я понимаю, подход Рейнера к проверке в памяти не был бы практичным в моем случае, поскольку у меня много "книг", 500 тыс. и растет. Это было бы большим хитом, если вы хотите вложить все в память.

Решение, с которым я столкнулся, состоит в следующем:

Поместите условие уникальности в базу данных (которое я нашел, это всегда хорошая идея, так как по моему опыту Rails не всегда делает хорошую работу здесь), добавив следующее в ваш файл миграции в db/migrate/:

  add_index :tasks [ :project_id, :name ], :unique => true

В контроллере поместите сохранение или update_attributes внутри транзакции и устраните исключение базы данных. Например.

 def update
   @project = Project.find(params[:id])
   begin
     transaction do       
       if @project.update_attributes(params[:project])
          redirect_to(project_path(@project))
       else
         render(:action => :edit)
       end
     end
   rescue
     ... we have an exception; make sure is a DB uniqueness violation
     ... go down params[:project] to see which item is the problem
     ... and add error to base
     render( :action => :edit )
   end
 end

конец

Ответ 3

Для Rails 4.0.1 эта проблема отмечена как исправленная этим запросом на растяжение, https://github.com/rails/rails/pull/10417

Если у вас есть таблица с уникальным индексом поля, и вы отмечаете запись для уничтожения, и вы создаете новую запись с тем же значением, что и уникальное поле, то когда вы вызываете save, уникальный индекс базы данных ошибка будет сброшена.

Лично это все еще не работает для меня, поэтому я не думаю, что он полностью исправлен.

Ответ 4

Ответ на благословение Райнера хорош. Но лучше, когда мы можем отметить, какие задачи дублируются.

class Project < ActiveRecord::Base
  has_many :tasks, inverse_of: :project

  accepts_nested_attributes_for :tasks, :allow_destroy => :true
end

class Task < ActiveRecord::Base
  belongs_to :project

  validates_each :name do |record, attr, value|
    record.errors.add attr, :taken if record.project.tasks.map(&:name).count(value) > 1
  end
end

Ответ 5

Ref this

Почему вы не используете: scope

class Task < ActiveRecord::Base
  validates_uniqueness_of :name, :scope=>'project_id' 
end

это создаст уникальную задачу для каждого проекта.