レコードの複製について

Question

レコードを複製したいのですが、パラメータを引っ張ってきてくれません。

試したこと

def update_new
    @retrieve_list = RetrieveList.new
    @retrieve_list = RetrieveList.find(params[:id]).dup
    @retrieve_list.save
  end

これを試しにSHOW画面に書いてみました。

  def show
    @retrieve_list = RetrieveList.new
    @retrieve_list = RetrieveList.find(params[:id]).dup
    @retrieve_list.save
  end

これだとLink_toでIDを引っ張ってきてくれました。
しかしshow画面に行くたびに複製され、えらいことになります。
理想は前者でIDを引っ張ってきてくれれば成功しそうなのですが…。

これらのヒントで何かご教示できる方いましたら、宜しくお願いいたします。

Comments

[maisumakun]

update_newをどのように呼び出していますか？

[makox0105]

EDITと同じようにSHOW画面にボタン作って、
<%= link_to "複製",
update_new_retrieve_lists_url(@retrieve_list), :class => 'btn btn-default' %>
といいうような感じです。

Answer 1

link_toで単にページ遷移するだけではパラメーターは送られませんので、params.require(:retrieve_list)が存在せずエラーとなります。

Comments

[maisumakun]

単純に複製したいだけなら、IDから引いてきたモデルオブジェクトをdupすればコピーできますので、それをsaveしてください。

[makox0105]

ご回答ありがとうございます。イメージ的には別名で保存のニュアンスに近いです。なのでページを引っ張ってきてそれを違うIDで保存といった具合です。dupでもできますか？

[maisumakun]

> ページを引っ張ってきて

これは「ページから値を供給して」という意味でしょうか?

[makox0105]

はい、そうなります。

[maisumakun]

でしたら、その「値を供給する」フォームも作成する必要があります（逆にdupは不要です）。

[makox0105]

一応別のフォームがupdate_newなんです。Editとは分けています。

[makox0105]

ちなみにfindで特定のページを引っ張ってくることはできました。SHOWで選択したレコードを持ってくることができないんです。

[maisumakun]

> 一応別のフォームがupdate_newなんです

では、link_toでGETするのではフォームが無意味になります。submitボタンの配置に切り替えてみましょう。

[makox0105]

submitでどう書くと良いのでしょうか？何から何まで聞いてしまってすみません。

[maisumakun]

フォーム全体を書いていただけますか？

[makox0105]

フォーム追記しました。SHOW画面の一番下の複製ボタンがうまくいかないです。

[maisumakun]

えっと、データを送信するためのフォーム要素は1つもないのですか？

[makox0105]

一応送信するフォームがupdate_newになるかと。

[maisumakun]

改めて確認ですが、やりたいことは次のどちらですか？

・「中身の全データ」をブラウザからサーバに送信して、それで新しい項目を作る（この場合、update_newという特殊なアクションすら不要で、単なるcreateをそのまま使ってもいいかもしれません）
・ブラウザから送信するのは「コピーする指示とID」だけ（この場合、params.require以下の処理はまるまる不要です）

[makox0105]

前者になるかと思います。

[maisumakun]

では、まず「中身の全データ」を送信するためのフォーム要素をform_forの内側に用意してください。ユーザーに表示させたくないなら<input type="hidden">で可能です。

[makox0105]

はい

[makox0105]

それからどしましょう？

[makox0105]

すみません、ちょっと思ったのですが、IDさえ引っ張れれば大丈夫な気がしました。findでparams[:id]では引っ張れないみたいで…。

[maisumakun]

update_newへのルーティングはどのように定義していますか？

[makox0105]

resources :retrieve_lists do
get 'update_new', on: :collection
post 'update_new_post', on: :collection
end
です。

[maisumakun]

おそらく、IDの値はparams[:id]ではなくparams[:retrieve_list_id]に来ているかと思います。

[makox0105]

それも試しました。
Couldn't find RetrieveList with 'id'=
になってしまいます。。。

[maisumakun]

rake routesで確認するとルーティングはどのようになっていますか？

[makox0105]

Prefix Verb URI Pattern Controller#Action
update_new_retrieve_lists GET /retrieve_lists/update_new(.:format) retrieve_lists#update_new
update_new_post_retrieve_lists POST /retrieve_lists/update_new_post(.:format) retrieve_lists#update_new_post
retrieve_lists GET /retrieve_lists(.:format) retrieve_lists#index
POST /retrieve_lists(.:format) retrieve_lists#create
new_retrieve_list GET /retrieve_lists/new(.:format) retrieve_lists#new
edit_retrieve_list GET /retrieve_lists/:id/edit(.:format) retrieve_lists#edit
retrieve_list GET /retrieve_lists/:id(.:format) retrieve_lists#show
PATCH /retrieve_lists/:id(.:format) retrieve_lists#update
PUT /retrieve_lists/:id(.:format) retrieve_lists#update
DELETE /retrieve_lists/:id(.:format) retrieve_lists#destroy

こんな感じです。

[maisumakun]

そもそもupdate_newのルーティングがidを取る形になっていないですね。

[makox0105]

なるほど。どうしたらidが取れるのですか？

[maisumakun]

routes.rbにはどのように書いていますか？

[makox0105]

resources :retrieve_lists do
get 'update_new', on: :collection
post 'update_new_post', on: :collection
end

です。

[makox0105]

on: :collectionをon: :meｍberにしたらできたっぽいです。