Rieles 3. validación before_destroy para evitar la eliminación de registros principales

Question

Tengo envíos y facturas.

factura pertenece al embarque
embarque tiene una factura

Si el envío tiene una factura, a continuación, el envío no debe ser capaz de ser eliminado. Necesito configurar esto en los modelos porque estoy usando ActiveAdmin.

Así que hice esto en shipment.rb

has_one :invoice 
before_destroy :check_for_invoice 

private 

def check_for_invoice 
    unless invoice.nil? 
    self.errors[:base] << "Cannot delete shipment while its invoice exists." 
    end 
end 

Pero acabo de recibir un mensaje de color amarillo que dice "El envío no se puede borrar" pero de hecho fue eliminado.

¿Cómo puedo evitar que se elimine el envío?

Answer 1

La devolución de llamada before_destroy necesita un valor verdadero/falso para determinar si debe o no proceeed.

Añadir un return false a su check_for_invoice así:

has_one :invoice 
before_destroy :check_for_invoice 

private 

def check_for_invoice 
    unless invoice.nil?  
    self.errors[:base] << "Cannot delete shipment while its invoice exists." 
    return false 
    end 
end 

Answer 2

Desde el docs:

Si un before_ * devolución de llamada devuelve falso, todas las devoluciones de llamada más tarde y la acción asociada se cancelan.

Así que trate esto:

self.errors[:base] << "Cannot delete shipment while its invoice exists." and return false 

Answer 3

Mis 2 centavos en shipment.rb

has_one :invoice, dependent: :restrict 

Creo que va a trabajar, vi esta solución en otro hilo. Estoy probando ahora en mis modelos.

Answer 4

para los carriles 4:

class Shipment < ActiveRecord::Base 
    has_one :invoice, dependent: :restrict_with_error 

hará el truco. Si desea una excepción en lugar de un error, use :restrict_with_exception. Ver más en the relevant api docs page.

Para Rails 3 (quizás antes también) Proveedores:

class Shipment < ActiveRecord::Base 
    has_one :invoice, dependent: :restrict