Ruby on Rails Associationを構築して2つの関連する関連を割り当てよう

Question

Userモデル、Buildingモデル、およびMaintenanceRequestモデルがあります。

ユーザhas_many :maintenance_requestsですが、belongs_to :buildingです。

メンテナンスは、私は、保守要求を作成し、その後、新しいを送信する方法を把握しようとしているbelongs_to :building、およびbelongs_to: user

を要求します。私がやりたいのは何

は次のとおりです。

@maintenance_request = current_user.building.maintenance_requests.build(permitted_mr_params) 
=> #<MaintenanceRequest id: nil, user_id: 1, building_id: 1> 

そしてそれは、親団体だに設定されたユーザーと建物と新しいメンテナンス要求を持っています。私がしなければならない何

：

@maintenance_request = current_user.maintenance_requests.build(permitted_mr_params) 
@maintenance_request.building = current_user.building 

私は、ユーザーの建物のベースにその建物を設定するためのメンテナンス要求を取得するとよいでしょう。

明らかに、私はこれを回避することができますが、私は構文的な砂糖を本当に感謝します。

Answer 1

設定されます

第2引数スコープを呼び出し可能（procまたはlambda）として渡して、特定のレコードセットを取得したり、関連するコレクションにアクセスしたときに生成されたクエリをカスタマイズすることができます。

すなわち

class User < ActiveRecord::Base 
    has_many :maintenance_requests, ->(user){building: user.building}, through: :users 
end 

次にご希望の1行は、あなたがcancancanを使用している場合は、あなたの能力ファイルに

can :create, MaintenanceRequest, user: @user.id, building: @user.building_idをハッシュ条件を追加することができ、またcurrent_user.building.maintenance_requests.build(permitted_mr_params)

を "ただ働き" 必要があります

Answer 2

私の意見では、あなたが提案するアプローチは問題ないと思います。それはもう1行のコードですが、コントローラの複雑さを増やすものではありません。

別のオプションは、あなたのリクエストパラメータで、user_idとbuilding_idをマージすることです：

permitted_mr_params.merge(user_id: current_user.id, building_id: current_user.building_id) 
@maintenance_request = MaintenanceRequest.create(permitted_mr_params) 

それとも、あなたは大量の割り当てを懸念していないなら、あなたのフォームで隠しフィールドとしてuser_idとbuilding_idを設定しました。しかし、私はパラメムをホワイトリストに載せなければならないので、大きなメリットはありません。

Answer 3

私のアプローチは、それがすでに利用者を通してそれに属しているので、

maintenance_request belongs_to :building 

をスキップすることです。代わりに、ユーザーが建物全体に移動することができますので、あなたは完全にmaintenance_request

UPDATE と明示的な建物の関連付けを省略することができ、建物のクラスで

、また

class Building 
    has_many :users 
    has_many :maintenance_requests, through: :users 

    #more stuff 
end 

の方法

class MaintenanceRequest 
    belongs_to :user 

    def building 
     user.building 
    end 

    #more class stuff 
end 

を定義することができますコールバックで自動動作を設定することができます。あなたがそれを行うようにジョブが実行されますが、あなたがユーザーのためにmaintenance_requestを作成するときに、よりRailseyの方法で

ので

class MaintenanceRequest 
    #stuff 
    before_create { 
     building=user.building 
    } 
end 

、建物はhas_manyドキュメントから、それに応じて