Rails - 間違ったURLパラメータを認識するフォームの条件文

Question

私は、添付ファイルとユーザの間の多様な関係を持っています。&チームは、ユーザとチームの両方が添付ファイルをアップロードできるようにします。 ： "ファイルの追加"ボタンをクリックすると：team_idまたは：user_idがパラメータとして渡されます。次に、attachments＃newフォームの隠しフィールドは、attribute_updateを適切なモデルに適用できるように、要求がチームまたはユーザーから来ているかどうかをコントローラに通知します。しかし実際には、チームの要求はユーザーのサブアクションとして配信され、その逆もあります。何が間違っている可能性がありますかに関する任意のアイデア？

class Attachment < ActiveRecord::Base 
    belongs_to :upload, polymorphic: true 
end 

class User < ActiveRecord::Base 
    has_many :attachments, as: :upload 
end 

class Team < ActiveRecord::Base 
    has_many :attachments, as: :upload 
end 

チーム位

<%= link_to "Add Files", new_attachment_path(:team_id => @team.id), class: "btn btn-md" %> 

ユーザー＃は＃新しい

<%= link_to "Add Files", new_attachment_path(:user_id => current_user), class: "btn btn-md" %> 

添付ファイルを表示するには、show

<% provide(:title, 'Add file') %> 

<div class="row"> 
    <div class="col-md-6 col-md-offset-3"> 
    <%= form_for(@attachment) do |f| %> 
     <%= render 'shared/error_messages', object: f.object %> 

     <%= f.file_field :file %> 

     <% if request.path == new_attachment_path(params[:team_id]) %> 
     <%= hidden_field_tag(:subaction, 'Team') %> 
     <% elsif request.path == new_attachment_path(params[:user_id]) %> 
     <%= hidden_field_tag(:subaction, 'User') %> 
     <% end %> 

     <%= f.submit "Add Files", class: "btn btn-primary" %> 
    <% end %> 
    </div> 
</div> 

アタッチメントコントローラ

def create 
    @attachment = Attachment.create(attachment_params) 
    @team = Team.find_by(params[:team_id]) 
    @user = User.find_by(params[:user_id]) 
    if @attachment.save 
     if params[:subaction] == 'Team' 
     @attachment.update_attribute(:upload, @team) 
     flash[:success] = "Team file uploaded!" 
     redirect_to @team 
     elsif params[:subaction] == 'User' 
     @attachment.update_attribute(:upload, @user) 
     flash[:success] = "User file uploaded!" 
     redirect_to current_user 
     end 
    else 
     render 'new' 
    end 
end 

Answer 1

コントローラif文を正しく機能させるために、team_idまたはuser_idの代わりに 'User'または 'Team'という文字列を渡すようにurl paramを変更しました。

位

<%= link_to "Add Files", new_attachment_path(:upload_type => 'Team'), class: "btn btn-md" %> 

ユーザー＃が

<%= link_to "Add Files", new_attachment_path(:upload_type => 'User'), class: "btn btn-md" %> 

添付ファイルを表示するには、showチーム＃新しい

<% provide(:title, 'Add file') %> 

<div class="row"> 
    <div class="col-md-6 col-md-offset-3"> 
    <%= form_for(@attachment) do |f| %> 
     <%= render 'shared/error_messages', object: f.object %> 

     <%= f.file_field :file %> 

     <%= hidden_field_tag(:subaction, params[:upload_type]) %> 

     <%= f.submit "Add Files", class: "btn btn-primary" %> 
    <% end %> 
    </div> 
</div> 

Answer 2

ifの文がattachments#newにあると思います。 params [：team_id]とparams [：user_id]の両方が整数を返すので、ifステートメントはリクエストがチームやユーザーから来ているかどうかを本当に判断できません。提案として、あなたは、このような@team_idとして変数にあなたのparams[:team_id]を格納し、このような何かにあなたのif文変更することができます：概説として、あなたのモデル中の多型の関連付けを使用して、この更を改善することができるかもしれしかし

<% unless @team_id.nil? %> 
    <%= hidden_field_tag(:subaction, 'Team') %> 
<% else%> 
    <%= hidden_field_tag(:subaction, 'User') %> 
<% end %> 

をhttps://launchschool.com/blog/understanding-polymorphic-associations-in-rails