cómo evitar guardar registros vacíos en un formulario de carriles anidados

Question

Estoy usando la gema nested_form para mi relación AddressBook. Cuando los espacios en blanco de usuario fuera el valor de un Addr existente, que quieren eliminar esa Addr en lugar de guardar con un espacio en blanco value

class Person < ActiveRecord::Base 
    has_many :addrs, dependent: :destroy 
    attr_accessible :name, :addrs_attributes 
    accepts_nested_attributes_for :addrs, reject_if: :addr_blank, allow_destroy: true 

    def addr_blank(a) 
    valid? && a[:id].blank? && a[:value].blank? 
    end 

class Addr < ActiveRecord::Base 
    belongs_to :person 
    attr_accessible :kind, :label, :value, :person_id 

Mi método :reject_if funciona bien pero no me da todo lo que necesito

1. valid? guarda mi Addrs en blanco alrededor a través de la validación
2. a[:id].blank? evita rechazos cuando los espacios en blanco de usuarios fuera y registro existente

Ahora, necesito eliminar (en lugar de guardar) un Addr existente cuando el usuario blanquea el value. Además, estoy exponiendo Personas y Agregados a través de una API RESTful. Veo dos opciones posibles:

1. proceso Deja la params hash para añadir el mágico _destroy=1 parámetro. IOW, emula la actividad del usuario al presionar el botón Eliminar.
2. Encapsule esto dentro del modelo Addr de modo que una actualización con un value en blanco se considere efectivamente una eliminación.

Basado en el consejo aquí es como yo implementé:

people_controller.rb

def update 
    @person = Person.find(params[:id]) 
    @person.destroy_blank_addrs(params[:person]) 
    respond_to do |format| 
    ... 

person.rb

def destroy_blank_addrs(person_params) 
    if valid? && person_params[:addrs_attributes] 
    person_params[:addrs_attributes].each do |addr_params_array| 
     addr_params= addr_params_array[1] 
     addr_params[:_destroy] = '1' if !addr_params[:id].blank? && addr_params[:value].blank? 
    end 
    end 
end 

Answer 1

Una tercera alternativa sería agregar una devolución de llamada before_save en Perso n que eliminará todas las direcciones que estén en blanco. Esta idea tiene algún mérito, pero probablemente no vaya con eso.

De las dos opciones que presenta, no iré con el post-procesamiento de los parámetros. Funcionará, pero es demasiado trabajo. Además, el código del controlador se volverá un poco más desordenado y creo firmemente en un controlador muy delgado.

La opción más fácil, en mi opinión, es eliminar las direcciones en blanco después de guardar. Puede agregar Person#remove_blank_addresses() y luego llamar al guardar correctamente. No necesita pasar los parámetros; puede simplemente iterar las direcciones y eliminar las en blanco. Tiene la desventaja de crear direcciones vacías y luego destruirlas, pero de todos modos las necesitarías para actualizar personas.

Si hablamos de la solución más limpia (en mi opinión), introduciría una tercera clase que manejaría toda esa lógica y haría que el controlador la delegara. El controlador sería tan fácil de probar de forma aislada y luego puede escribir una especificación de modelo que verifique todos los detalles esenciales. Es un poco más trabajo y no puedo pensar en un buen nombre en este momento (PersonUpdater?), Pero podría ser una idea que vale la pena pensar.

Answer 2

accepts_nested_attributes_for :addrs, 
    allow_destroy: true, 
    :reject_if => proc { |att| att[:name].blank? && attr[:description].blank? } 

Answer 3

accepts_nested_attributes_for :addrs, 
    allow_destroy: true, 
    reject_if: -> { |attr| [name, description].any? &:blank? } 

Answer 4

accepts_nested_attributes_for :addrs, 
    allow_destroy: true, 
    reject_if: :all_blank