form_with editアクションにて複数のモデルから参照したい

Question

viewファイル

<%= form_with( model: [@main, @detail], url: page_path, local: true) do |f| %>
        <%# render 'shared/error_messages', model: f.object %>
      <%# 投稿画像 %>
          <div class="img-upload">
            <div class="weight-bold-text">
              出品画像
            <span class="indispensable">必須</span>
            </div>
            <div class="click-upload">
              クリックしてファイルをアップロード
            </div>
              <%= f.file_field :image %>
          </div>
      <%# 場所と場所の詳細 %>
          <div class="weight-bold-text">
            場所
            <span class="indispensable">必須</span>
          </div>
  
          <%= f.text_area :name, class:"single-input-primary", placeholder:"場所の名前（必須 20文字まで)", maxlength:"20" %>
  
          <div class="weight-bold-text">
            <div class="info-text">
              場所の詳細
            <span class="indispensable">必須</span>
            </div>
          </div>
  
          <%= f.text_area :info, class:"single-input-primary", placeholder:"どんな場面なのか詳細を書いてみよう。(200文字まで)" ,rows:"7" ,maxlength:"200" %>

          <div class="category_country">
            <%= f.collection_select(:category_id, Category.all, :id, :name, {}, {}) %>
            <%= f.collection_select(:country_id, Country.all, :id, :name, {}, {}) %>
          </div>

controller

class PagesController < ApplicationController

  before_action :authenticate_user!, except: [:index, :show]
  before_action :main_detail_find, only: [:show, :edit, :update, :destroy]

  def index
    @pages = Main.includes(:user).order("id DESC")
  end

  def show
  end

  def edit
  end

  def update
  end

  def destroy
    if current_user.id == @main.user_id
      @main.destroy
      redirect_to root_path
    else
      redirect_to root_path
    end
  end

  private

    def main_detail_find
      @main = Main.find(params[:id])
      @detail = Detail.find(@main.id)
    end

end

main

class CreateMains < ActiveRecord::Migration[6.0]
  def change
    create_table :mains do |t|
      t.string :name,              null: false
      t.text :info,              null: false

      t.references :user, null: false, foreign_key: true
    end
  end
end

detail

class CreateDetails < ActiveRecord::Migration[6.0]
  def change
    create_table :details do |t|

      t.integer :category_id, null: false
      t.integer :country_id, null: false

      t.references :main, null: false, foreign_key: true
    end
  end
end

上の画像のように@detailの情報しか反映されません。
@mainからも反映されるようにしたいです。
何かいい方法ないでしょうか？
--------追記----------
Activestorageを使用しているのですが保存されている画像についても反映されません

Answer 1

「上の画像のように@detailの情報しか反映されません。」と言われても、どのじょうほうがmainでなにがdetailか我々にはわかりません。

で、
form_with( model: [@main, @detail], url: page_path, local: true) do |f| %書くと、f は @detailと結びつきます。@mainは単にURL生成に使われるだけです。
そのURLはurl: optionで上書きされるので、結果このviewでは@mainは何の仕事もしていません。
text_field_tag を使って @mainの値を管理するか
form_with model: @main とし、@detailの方は fields_for を使って記述するかしてください

追記
suginokitech86 さんのこの設計はとても違和感があります。
PagesController は モデル Page を扱うという命名です。　Railsの規約では。
なのに モデル Main を扱っています。だから view での form_with でややこしい設定が必要になります。
敢えてモデルとは違うcontrollerを使うということもありますが、この場合その意味がありますか？

form_withでも model: [@main, @detail] と階層化する意味があるのか？
この記述は @detailを触るときの設定です。@mainも必要なら　model: @main とし、@detail は field_forでやるのが Railsの普通だろうと思います。

PagesController をやめて MainsController にし、form_with model: @main にするというRailsの普通に合わせるともっと楽につくれると思います。作り直しを薦めます。

2021/02/07 08:39

Comments

[osugi86]

winterboumさんありがとうごいます。
@mainと@detailについてはdbごと載せます

[osugi86]

winterboumさん再度ご指摘の方ありがとうございます。
Pagescontrollerと命名した理由としましては投稿したデータ一覧を表示させるために作成いたしました。
Mainscontrollerは既にメイン画面にて使用してしまっていました。
見直しながら編集をしていきたいとおもっております。

[winterboum]

Mainを一覧なら　MainsController の index ですね。
config/routes.rb で resources :mans ってすると自動で６つurlが用意されますが、それは特別な理由がなければ利用したほうが良いです。

[osugi86]

返信ありがとうございます。
今現状はMain/indexがメイン画面、Page/indexが投稿一覧表示となっております。
resourcesを使わない理由はPageではnew、createのルーティングを使用しないためでした。

[winterboum]

ですから、
Main/showをメイン画面にしてほしい。
Mainならnew,createを使うでしょう。Main#indexは一覧。
indexってそういう意味でしょう。

Answer 2

仕様として「メソッドの最後に記述した式が戻り値になる」というのがあるからでは。複数にしたいならその複数をまとめたオブジェクト・配列にするしかないと思います。

Comments

[osugi86]

m.ts10806さんいつもありがとうございます。
いちよう@main2=@main+@detailというように@main2を使用してform_withのmodel指定にしようかなと思ったのですがcontroller側の記述方法が思いつきません
@main = Main.find(params[:id])
@detail = Detail.find(@main.id)この記述を変えようと思っています。

[m.ts10806]

いつも？何かしましたっけ。。
質問1つも解決されてない方なので、むしろ批判的な気持ちしか私にはないですが。
あと細かいですが「とりあえずは」という意味で使いたいのでしたら「いちおう」です。「一応」

>@main2=@main+@detail
これで想定の形になりますか？デバッグしてみてください。

てっきり
[@main,@detail]
とか
{main:@name,detail:@detail}
のように明確にわけるものと思っていました。

[osugi86]

m.ts10806さん回答ありがとうございます。
いつもという意味に関してですが自分が質問したときに質問方法をご指摘いただいていたのでそう言った表現にしました。
あとはteratailsという場所については質問を必ず回答しなければいけないのでしょうか？
批判の意味をお願いいたします。

[m.ts10806]

teratailsではなくteratailです。

>質問を必ず回答しなければいけないのでしょうか？
いえ。そういう意味ではなく、質問履歴を見たら
https://teratail.com/users/suginokitech86#question
1つも解決されてなかったので（私が見た時点。この質問だけ解決されている）
「指摘や回答がついている状態で投げっぱなしにしている」という意味で「解決する気ないんだなこの人」という批判です。
解決する気なしと判断されれば、アドバイスしたとしても無駄になると感じて、関わらないようにする回答者は少なくないです。
プログラミングの本質は問題解決。問題を放置したままの人に、成長が見込めないのは回答者側はよく知っているので、「未解決質問があるまま次々と質問を投稿する人」には批判的な感情しかありません。

[osugi86]

返信ありがとうございます。
質問したことに関して自力で解決したら解決済などといったものがあるのでしょうか？
あるとするならそれをしたいです。

[m.ts10806]

ヘルプ読んでください。
https://teratail.com/help#resolve-myself

[osugi86]

返信ありがとうございます。
ベストアンサーにて解決済みなのですね。
ちなみに質問内容の削除などはできるのですか？

[m.ts10806]

ヘルプ読んでください。

[osugi86]

見てみます。ありがとうございました。