多対多で関連付けたモデルのデータ更新を1つのフォームで行う際にオートフィルさせたい

Question

実現したいこと

次のような関連付けを行ったデータの更新（update）を1つのフォーム（memoの更新フォーム）で行いたいです。
その際、memoに関連付けられたsourceモデルのurlを入力フォームにオートフィルさせたいです。
現状でも更新自体はできますが、フォームのurl入力欄が空欄になってしまいます。

実行環境

・Rails 6.0.0
・ruby 2.6.3p62 (2019-04-16 revision 67580) [x86_64-linux]
・Cloud 9

ソースコード

class MemosController < ApplicationController

#（中略）
 
  def new
    @memo = Memo.new
    @source = @memo.sources.build
  end
  
  def create
    @memo = Memo.new(memo_params.slice(:word, :content, :synonym, :tag_list))
    byebug
    if @memo.save
      memo_params[:url].reject(&:blank?).uniq.each do |url|
        @source = Source.find_or_create_by(url: url)
        @memo.sources << @source
      end
      flash[:notice] = "新しいMemoを作成しました！"
      redirect_to root_url
    else
      flash.now[:alert] = "無効な送信です"
      render :new
    end
  end

#（中略）

  #editとupdateを上手く実装できません
  def edit
    @memo = Memo.find(params[:id])
  end
  
  def update
    @memo = Memo.find(params[:id])
    if @memo.update(memo_params)
      redirect_to root_url
      flash[:notice] = "Memoを更新しました！"
    else
      flash.now[:alert] = "無効な送信です"
      render :edit
    end
  end

  private
    def memo_params
      params.require(:memo).permit(:word, :content, :synonym, :tag_list, url: [])
    end
end

<%= form_with model: @memo, local: true do |f| %>
  <div class="form-group">
    <%= f.label :word, "単語" %>
    <%= f.text_field :word, class: "form-control", %>
  </div>
  <div class="form-group">
    <%= f.label :synonym, "別名" %>
    <%= f.text_field :synonym, class: "form-control" %>
  </div>
  <div class="field form-group">
    <%= f.label :tag_list, "単語" %>
    <%= text_field_tag 'memo[tag_list]', @memo.tag_list.join(","), class: "form-control" %>
  </div>
  <div class="form-group">
    <%= f.label :content, "意味" %>
    <%= f.text_area :content, rows: "10", class: "form-control" %>
  </div>
  <div class="form-group"> <!--この部分にmemoと関連付けたurlをオートフィルさせたい-->
    <%= f.label :url, "単語" %>
    <%= f.url_field :url, name: "memo[url][]", class: "form-control" %>
    <%= f.url_field :url, name: "memo[url][]", class: "form-control url-form" %>
    <%= f.url_field :url, name: "memo[url][]", class: "form-control url-form" %>
  </div>
  <div class="form-group">
    <%= f.submit yield(:btn_message), class: "btn btn-success btn-lg btn-block" %>
  </div>
<% end %>

困っていること

上記内容でeditしようとすると、urlの入力欄に値がオートフィルされません。
原因はeditアクションのインスタンス変数@memoにキーがurlの値が入っていないことだと考えております。
なんとか更新フォームのurl入力欄に@memoと関連付けられたurlをオートフィルさせることは出来ないでしょうか。

試したこと

更新時にオートフィルされるのはeditアクションで作成された@memoに入っているハッシュを元に行われている？と考え、@memoに関連付けたurlを代入しようとしましたが、うまいくいきませんでした。

・関連付けから習得したurlの配列を@sourceに代入し、:urlをキーとして@memoに追加しようとするもエラー

def edit
  @memo = Memo.find(params[:id])
  @source = @memo.sources
  @memo[:url] = @source
end

ActiveModel::MissingAttributeError in MemosController#edit
can't write unknown attribute `url`

・配列として代入したいのだから、挿入先に空の配列を用意してみるもエラー

def edit
  @memo = Memo.find(params[:id])
  @source = @memo.sources
  @memo[:url][] = @source
end

NoMethodError in MemosController#edit
undefined method `[]=' for nil:NilClass

どなたか、ヒントだけでもいいので、ご回答お願い申し上げます。

Comments

[winterboum]

すでに関連付けられているsourceについてのみurlを出したいのでしょうか？
それとも、sourceを追加削除も考えているのでしょうか。
それにもよるのですが。。。

[_Taturon_]

前回の質問ではお世話になりました。
すでに関連付けられているsourceについてのみです。

一般的？なフォームですと、編集フォームへアクセス（/memos/:id/edit）した際にeditアクション（ @memo = Memo.find(params[:id])）によってモデルから値が引っ張り出され、その値がフォームにあらかじめ入力された状態になりますよね？（違ってたらごめんなさい）
これを関連付けた値でもやりたいのです

例えばmemo1にurl1とurl2が関連づけられて保存されていた場合に、memo1に対してeditアクションを実行してフォームに遷移した際に、Memoモデルに保存された値だけでなく、関連づけられたurl1,url2もフォームに最初から入力させた状態にしたいです

現状ですと、Memoモデルに保存した値しかeditのフォームに入力されません

説明が難しいです

Answer 1

urlはSourceのものですので、memoの編集時にその値を修正することはしない、と考えてよいですね？　他のmemoでもつかっている可能性があるのですから。
すると f.url_field と入力fieldにするのではなく

memo.sources.each do | source |
 source.url
end

と表示だけするのが妥当でしょう。
もしURLの修正も行いたいのでしたら

f.fields_for :sources do | fs |
  fs.url_field :url
end

ですが、これだと def update がややこしくなります。
入れなおしたurlが他のsourceに既に有ったらどうするのか
同じsourceを参照していたほかのmemoは実は元のurlが正しいということは絶対ないのか
などをどう扱うのか、を考え、かつそれを実現する必要が出てきます

Comments

[_Taturon_]

winterboumさん、ご回答ありがとうございます。
自分でもいろいろ調べた結果、おっしゃられている通り、fields_forで実装してみようと思います。

[_Taturon_]

いろいろ検討した結果、memoとurlを多対多では無く、一対多で実装することにいたしました。
理由としては、urlが重複する可能性が低いことと多対多で実装する技術力がないためです。
ただ、ここ数日間の学習でリレーションに関する知識はかなり強化されました。

２度にわたって回答して頂き誠にありがとうございました！