ruby on rails - valida_unicidad_de en rieles de modelos anidados destruidos

ruby-on-rails nested-forms

Question

Tengo un modelo de proyecto que acepta atributos anidados para la tarea.

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

La validación de unicidad en el modelo de tareas genera problemas al actualizar el proyecto.

En la edición del proyecto, elimino una tarea T1 y luego agrego una nueva tarea con el mismo nombre T1, la validación de exclusividad restringe el guardado de Proyecto.

params hash mirar algo así como

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

La validación en la tarea se realiza antes de destruir la tarea anterior. Por lo tanto, la validación falla. ¿Alguna idea de cómo validar de tal manera que no considere que la tarea se destruya?

Answer 1

Andrew France creó un parche en este thread , donde la validación se realiza en la memoria.

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

Answer 2

Como lo entiendo, el enfoque de Reiner sobre la validación en la memoria no sería práctico en mi caso, ya que tengo muchos "libros", 500 K y creciendo. Eso sería un gran éxito si quieres traer todo a la memoria.

La solución que se me ocurrió es:

Coloque la condición de unicidad en la base de datos (lo que encontré es siempre una buena idea, ya que en mi experiencia Rails no siempre hace un buen trabajo aquí) agregando lo siguiente a su archivo de migración en db / migrate /:

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

En el controlador, coloque los atributos de guardar o actualizar dentro de una transacción y rescatar la excepción de la base de datos. P.ej,

 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

fin

Answer 3

La respuesta de Rainer Blessing es buena. Pero es mejor cuando podemos marcar qué tareas están duplicadas.

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

Answer 4

Para Rails 4.0.1, este problema está marcado como corregido por esta solicitud de extracción, https://github.com/rails/rails/pull/10417

Si tiene una tabla con un índice de campo único y marca un registro para su destrucción, y crea un nuevo registro con el mismo valor que el campo único, entonces cuando llame a guardar, se generará un error de índice único a nivel de base de datos.

Personalmente, esto todavía no funciona para mí, así que no creo que esté completamente arreglado todavía.

Answer 5

Ref this

¿Por qué no usas: alcance

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

esto creará una tarea única para cada proyecto.