Как "проверять" на уничтожение в рельсах

При уничтожении резервного ресурса я хочу гарантировать несколько вещей, прежде чем я разрешу операцию уничтожения продолжить? В принципе, я хочу, чтобы остановить операцию уничтожения, если я заметил, что это приведет к тому, что база данных окажется недопустимой? Нет никаких обратных вызовов проверки на операцию уничтожения, поэтому как можно "подтвердить", следует ли принять операцию уничтожения?

Ответ 1

Вы можете создать исключение, которое вы затем поймаете. Rails wraps удаляет транзакции, что помогает.

Например:

class Booking < ActiveRecord::Base
  has_many   :booking_payments
  ....
  def destroy
    raise "Cannot delete booking with payments" unless booking_payments.count == 0
    # ... ok, go ahead and destroy
    super
  end
end

В качестве альтернативы вы можете использовать обратный вызов before_destroy. Этот обратный вызов обычно используется для уничтожения зависимых записей, но вы можете исключить исключение или добавить ошибку.

def before_destroy
  return true if booking_payments.count == 0
  errors.add :base, "Cannot delete booking with payments"
  # or errors.add_to_base in Rails 2
  false
  # Rails 5
  throw(:abort)
end

myBooking.destroy теперь вернет false, а myBooking.errors будет заполнено при возврате.

Ответ 2

просто примечание:

Для рельсов 3

class Booking < ActiveRecord::Base

before_destroy :booking_with_payments?

private

def booking_with_payments?
        errors.add(:base, "Cannot delete booking with payments") unless booking_payments.count == 0

        errors.blank? #return false, to not destroy the element, otherwise, it will delete.
end

Ответ 3

Это то, что я сделал с Rails 5:

before_destroy do
  cannot_delete_with_qrcodes
  throw(:abort) if errors.present?
end

def cannot_delete_with_qrcodes
  errors.add(:base, 'Cannot delete shop with qrcodes') if qrcodes.any?
end

Ответ 4

У ассоциаций ActiveRecord has_many и has_one разрешает зависимую опцию, которая гарантирует, что связанные строки таблицы удаляются при удалении, но обычно это означает, что ваша база данных чиста, а не предотвращает ее недействительность.

Ответ 5

Вы можете обернуть действие destroy в выражении "if" в контроллере:

def destroy # in controller context
  if (model.valid_destroy?)
    model.destroy # if in model context, use `super`
  end
end

Где valid_destroy? это метод в вашем классе модели, который возвращает true, если выполняются условия для уничтожения записи.

Наличие такого метода также позволит вам запретить отображение опции удаления пользователю, что улучшит работу пользователя, так как пользователь не сможет выполнить незаконную операцию.

Ответ 6

В итоге я использовал код для создания переопределения can_destroy на activerecord: https://gist.github.com/andhapp/1761098

class ActiveRecord::Base
  def can_destroy?
    self.class.reflect_on_all_associations.all? do |assoc|
      assoc.options[:dependent] != :restrict || (assoc.macro == :has_one && self.send(assoc.name).nil?) || (assoc.macro == :has_many && self.send(assoc.name).empty?)
    end
  end
end

Это имеет дополнительное преимущество: сделать тривиальным, чтобы скрыть/показать кнопку удаления на ui

Ответ 7

Вы также можете использовать обратный вызов before_destroy для создания исключения.

Ответ 8

У меня есть эти классы или модели

class Enterprise < AR::Base
   has_many :products
   before_destroy :enterprise_with_products?

   private

   def empresas_with_portafolios?
      self.portafolios.empty?  
   end
end

class Product < AR::Base
   belongs_to :enterprises
end

Теперь, когда вы удаляете предприятие, этот процесс проверяет, есть ли продукты, связанные с предприятиями Примечание. Вы должны написать это в верхней части класса, чтобы сначала проверить его.

Ответ 9

Использовать проверку контекста ActiveRecord в Rails 5.

class ApplicationRecord < ActiveRecord::Base
  before_destroy do
    throw :abort if invalid?(:destroy)
  end
end
class Ticket < ApplicationRecord
  validate :validate_expires_on, on: :destroy

  def validate_expires_on
    errors.add :expires_on if expires_on > Time.now
  end
end