Rails 3. before_destroy validation per evitare l'eliminazione dei record padre

Question

Ho delle spedizioni e fatture.

fattura appartiene alla spedizione
spedizione ha una fattura

Se la spedizione ha una fattura, quindi la spedizione non dovrebbe essere in grado di essere eliminato. Devo configurarlo nei modelli perché sto usando ActiveAdmin.

Quindi ho fatto questo in 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 

Ma ottengo solo un messaggio di giallo con scritto "La spedizione non può essere eliminato", ma è stato di fatto cancellato.

Come si impedisce la cancellazione della spedizione?

Answer 1

Il before_destroy callback ha bisogno di un valore true/false per determinare se o non proceeed.

Aggiungi un return false al check_for_invoice in questo modo:

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

Dal docs:

Se un before_ * callback restituisce false, tutti i richiami più tardi e l'azione associata vengono cancellati.

Quindi provare questo:

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

Answer 3

I miei 2 centesimi in shipment.rb

has_one :invoice, dependent: :restrict 

Penso che sarà il lavoro, ho visto questa soluzione in un altro thread. Sono in trincea ora nei miei modelli.

Answer 4

Per Rails 4:

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

farà il trucco. Se si desidera un'eccezione anziché un errore, utilizzare :restrict_with_exception. Vedi di più su the relevant api docs page.

Per Rails 3 (forse prima anche) Ricerche:

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