销毁和删除函数的问题 Rails

Question

我决定将我的应用程序上的销毁功能更改为仅将状态字段切换为 false（索引和显示操作仅显示状态为 true 的结果）。

好吧，我已经改变了这一点，一开始它可以工作，但现在它只是在完成提交之前“回滚”......

控制器销毁：

def destroy
@customer = Customer.find(params[:id])
@contact = @customer.contacts
@adress = @customer.adresses

if @customer.delete(current_user)
  @contact.each do |f|
    f.delete(current_user)
  end
  @adress.each do |g|
    @telephone = g.telephones
    @telephone.each do |t|
      t.delete(current_user)
    end
    g.delete(current_user)
  end
  flash[:success] = "Cliente excluído"
else
  flash[:danger] = "Erro! O cliente não foi excluído."
end
redirect_to customers_path
end

模型删除：

def delete(user_id)
update_attributes(status: false, changed_by: user_id, deleted_at: Time.now, updated_at: Time.now)
end

除此之外，我将 :destroy 方法仅限于管理员（这是正确的方法吗？为什么不删除？）

谢谢！

Answer 1

您的回滚问题可能与验证失败有关。你可以通过将update_attributes更改为update_attributes!来找出这是什么，当验证失败时会抛出一个错误，你可以看到哪里出了问题。

除此之外，还有一些方法可以改进你的代码

首先，您可以将dependent: :destroy 添加到关系中，而不是手动删除客户的联系人和地址。

这样工作：

class Customer < ActiveRecord::Base
  has_many :contacts, dependent: :destroy
  has_many :adresses, dependent: :destroy
end

class Adress < ActiveRecord::Base
  has_many :telephones, dependent: :destroy
end

当您在记录上调用.destroy 时，它将在任何依赖于它们的关系上调用.destroy： :destroy。 .delete 不会做这个通过，这是他们之间的主要区别。

通过将其添加到上述关系中，您可以将控制器更改为：

def destroy
  @customer = Customer.find(params[:id])
  if @customer.destroy
    flash[:success] = "Cliente excluído"
  else
    flash[:danger] = "Erro! O cliente não foi excluído."
  end
  redirect_to customers_path
end

这会破坏客户、地址和地址的电话。

非永久删除

要按照您的意愿进行软删除，您应该使用Paranoia gem。 它允许您使用acts_as_paranoid 分配一个类，因此当您调用.delete 或.destroy 时，它并没有真正被删除，它只是被隐藏并且可以稍后恢复。

它与 Activerecord 完美集成，因此您无需执行 Customer.where(status: true) 之类的操作即可获取未删除的记录，默认情况下它不会包含已删除的记录。尽管您可以根据需要提出要求。

gem 的 Github 页面很好地解释了如何使用它。

默认情况下，它不会保存执行删除的用户，只会保存执行删除的时间。如果你想这样做，你应该创建一个包装器，在分配执行删除的用户后调用 delete。

这样的事情应该可以工作：

def destroy_by(user)
  update_attributes!(changed_by: user)
  destroy
end

然后您可以将控制器中的@customer.destroy 更改为@customer.destroy_by(current_user)

无需覆盖或限制删除或销毁。