종속성 => : destroy 및 before_filter

Question

계정 모델 (has_many : users)과 사용자 모델 (belongs_to : account, : dependent => : destroy)의 두 모델이 문제가 있습니다. 내가 삭제되는 마스터 사용자를 보호하기 위해 노력하고있어

def protect_master_user 
    unless User.find_all_by_account_id_and_is_master(self.account_id, true).count > 1 
    false 
    end 
end 

:

내 사용자 모델은 다음있다. 상위 (계정)가 삭제 된 경우이를 어떻게 무시할 수 있습니까? 나는 시도 : 종속 => : 파괴 및 : 아무 소용이 삭제하십시오.

EDIT : 고정 코드.

Answer 1

has_many : users, : dependent => : delete_all 또는 계정의 추가 사용자 정의 콜백이 있습니다. : delete_all 메소드는 더 간단하지만 권고하지는 않는다. 왜냐하면 다른 콜백이 사용자 레코드에서 일어나지 않기 때문이다.

올바른 해결책은 사용자의 콜백과 함께 작동하는 Account의 custom before_destroy 콜백입니다.

class Account < ActiveRecord::Base 
    has_many :users 

    before_destroy :destroy_users 
    protected 
    def destroy_users 
     users.each do |user| 
     u.account_id = nil 
     u.destroy 
     end 
    end 

end 

class User < ActiveRecord::Base 
    belongs_to :account 

    before_destroy :protect_master_user 

    protected 
    def protect_master_user 
     unless account_id.nil? ||! master || 
     User.find_all_by_account_id_and_master(self.account_id, true).count > 1 
     errors.add_to_base "Cannot remove master user." 
     return false 
     end 
    end 
end 

account.id가 nil 인 경우를 제외하고는 단락 회로를 사용하고 destroy는 계속됩니다. 사용자가 마스터 사용자가 아닌 경우에도 마찬가지입니다. 파괴되는 객체가 마스터 사용자가 아닌 경우에도 둘 이상의 마스터 사용자가 있는지 확인해야하는 이유는 무엇입니까?

다시 말해서 삭제 대신 삭제를 사용할 수 있습니다. 그러나 당신이 가지고 있거나 미래에있을 모든 * _destroy 콜백을 건너 뜁니다.

Answer 2

나는 아주 똑같은 질문과 수수께끼를 최근에 가지고 있었고이 문제를 다루는 가장 좋은 방법은 컨트롤러에서 처리하는 것이었다. 사용자가 시스템 그것을하고있다 :

class UsersController < ApplicationController 
    def destroy 
     user = current_account.users.find(params[:id].to_i) 
     if allow_removal_of_user?(user) && user.destroy 
     redirect_to users_path, :notice => "User sucessfully deleted" 
     else 
     flash[:error] = user.errors.empty? ? "Error" : user.errors.full_messages.to_sentence 
     render :edit 
     end 
    end 

    private 
    def allow_removal_of_user?(user) 
     if user == current_user 
     errors.add(:user_removal, "Can't delete yourself") 
     false 
     elsif user.only_user? || (user.master_users_for_account.count == 1 && user.master_role?) 
     errors.add(:user_removal, "Can't delete last Master user") 
     false 
     else 
     true 
     end 
    end 
    end 

희망이 누군가를 돕는다!