Rails コントローラでパラメータ追加

コントローラーでアップデートする前に、ビューで取得していないupdate_user_idのパラメータを追加したいです。

こちらを参考にしてみましたがうまくできませんでした。
https://stackoverflow.com/questions/5014804/rails-change-value-of-parameter-in-controller/5014844

どうぞよろしくお願いいたします。

  def update

    company_params[:updated_user_id] = current_user.id #deviseを利用
    
    respond_to do |format|
      if @company.update(company_params)
        format.html { redirect_to  company_path, notice: 'Company was successfully updated.' }
        format.json { render :show, status: :ok, location: @company }
      else
        format.html { render :edit }
        format.json { render json: @company.errors, status: :unprocessable_entity }
      end
    end
  end

    def company_params
      params.require(:company).permit(:company_name,:updated_user_id)
    end

###追記
下記で実現できましたが、できればもう少しスマートにparamsを変更できればと思います。
引き続きどうぞよろしくお願いいたします。

~~company_params[:updated_user_id] = current_user.id #deviseを利用~~
↓
@company.updated_user_id = current_user.id #deviseを利用

ruby_math

2019/04/17 04:51

これ以上スマートな方法でのユーザーIDのパラメータ送信は思いつきませんね。個人的には一番スマートな方法であると思いますし、要件は満たしていると考えます。

回答3件

ベストアンサー

Hash#mergeを利用して

ruby
1if @company.update(company_params.merge(update_user_id: current_user.id))

などはどうでしょうか？

投稿2019/04/17 06:52

asm

総合スコア15145

@company.update(company_params) を実行する直前で @company.updated_user_id = current_user.id を実行するようにしてみてください。

提示のコードが動かない原因は company_params が呼ぶ度に新しいオブジェクトを返すためです。 company_params[:updated_user_id] = current_user.id の実行後に company_params を呼んでも updated_user_id の値は入っていません。company_params で値をまとめたい場合は、その点を考慮してみてください。

hiddenフィールドを使う場合の懸念はおっしゃる通りだと思います。

投稿2019/04/17 04:57

退会済みユーザー

総合スコア0

回答失礼致します。

参考にされたサイトをみました。
少し記述方法が違いますね。

ワタシ的にはView側にhidden_fieldをおいたほうが簡単だと思います。

erb
1<%# なんとなくform_withで書いていますが御自身の仕様に合わせてください %>
2<%= form_with model: @conpany do |f| %>
3
4<%#  @updated_user_id変数に現在ログインされているユーザーのIDを代入する %>
5<% @updated_user_id = current_user.id %>
6
7<%# 隠しフォーム @updated_user_idをvalueとして持たせてあげることで現在ログインされているユーザーのIDが送られる %>
8<%= f.hidden_field :updated_user_id,  value:"#{@updated_user_id}" %>
9
10<% end %>

やりたいこととしては、現在ログインされているユーザーのIDをupdated_user_idに代入したいと言うことでよろしいでしょうか？

上記を試してみてください。

=======================================================================

確かにHidden_fieldは危険ですね、以下を試してみてください。

rb
1  def update
2
3    respond_to do |format|
4      if @company = Company.find(params[:id])
5         @company.updated_user_id = current_user.id
6         @company.update(company_params)
7        format.html { redirect_to  company_path, notice: 'Company was successfully updated.' }
8        format.json { render :show, status: :ok, location: @company }
9      else
10        format.html { render :edit }
11        format.json { render json: @company.errors, status: :unprocessable_entity }
12      end
13    end
14  end