Rails 3. before_destroy Validierung um zu verhindern, dass das löschen übergeordnete Datensätze

Habe ich Sendungen und Rechnungen.

Rechnung gehört zu dem Versand

Sendung hat einen Rechnung

Wenn der Sendung eine Rechnung, dann wird der Versand sollte nicht mehr gelöscht werden. Ich brauche, um diese bis in den Modellen, denn ich bin mit ActiveAdmin.

Also ich habe das in der Sendung.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

Aber ich bekomme nur eine gelbe Meldung "Sendung kann nicht gelöscht werden," aber es war in der Tat gelöscht.

Wie kann ich verhindern, dass die Sendung gelöscht wird?

InformationsquelleAutor leonel | 2012-01-11
  1. 23

    Den before_destroy callback muss einen true/false-Wert, um zu bestimmen, ob oder nicht zu fahren Sie bis.

    Hinzufügen return false zu Ihrem check_for_invoice etwa so:

    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
    • Ah, dass ist richtig! Ich habe vergessen den Wert false zurück. Ich postete auch diesen Teil des Codes falsch if invoice.nil?, es soll unless invoice.nil? statt.
    • Ich war neugierig auf die Logik - aber dachte, es könnte nur etwas komisch, Sie waren dabei. Ich werde zu aktualisieren meine Antwort, dass es für die Nachwelt Willen.
    • Check diese Antwort auf eine ähnliche Frage für einen besseren Weg, dies zu tun.
    InformationsquelleAutor Paul Simpson
  2. 5

    Meine 2 Cent in den Versand.rb

    has_one :invoice, dependent: :restrict

    Ich denke, dass es dann funktioniert, sah ich diese Lösung in einem anderen thread. Ich bin tryng jetzt in meine Modelle.

    InformationsquelleAutor josias
  3. 3

    Aus der docs:

    Wenn ein before_* callback false zurück, alle die später Rückrufe und die damit verbundenen Maßnahmen werden aufgehoben.

    Also versuchen Sie dies:

    self.errors[:base] << "Cannot delete shipment while its invoice exists." and return false
    • du hast Recht, dass return ist meist nicht idiomatisch in Ruby, aber was ist, wenn die Zeile nicht die Letzte in der Methode (plus betrachten refactoring)? Oder was ist, wenn Sie später hinzufügen, andere Anweisungen in der Methode und vergessen, fügen Sie die Rückkehr? Ich denke, das sind die Gründe warum Sie das tun, sehen and return false verwendet in rails-Projekten...
    • Okay, das ist ein überzeugendes argument genug. Ich habe ein Rollback meine überarbeitung.
    InformationsquelleAutor maprihoda
  4. 1

    Für Rails 4:

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

    wird den trick tun. Wenn Sie möchten, eine Ausnahme statt eines Fehlers verwenden :restrict_with_exception. Sehen Sie mehr an die jeweiligen api-docs-Seite.

    Für Rails 3 (vielleicht auch früher) versuchen:

    class Shipment < ActiveRecord::Base
  has_one :invoice, dependent: :restrict
    InformationsquelleAutor nitsas
Schreibe einen Kommentar