editをしても情報が引き継がれていないのを解決したい

Question

こんばんわ。現在1：nで組んだn側のアソシエーションにedit機能を付けたいです。

色々試してみみて、editにアクセスする事までは出来たのですが、元のformの内容がedit引き継がれません。そのまま登録するとupdateではなくcreateされ、新たにeachが追加されてしまう状態です。

ご教示よろしくお願い致します。

 <table class="company">
  <tbody>
    <% if @company.blocks.any? %>
     <% @company.blocks.each do |block| %>
    <tr>
      　<td><%= block.name %></td>
        <td><%= block.progress %></td>
        <td><%= block.start %></td>
        <td><%= block.supply %></td>
        <td><%= block.comment %></td>
        <td><%= link_to '編集', edit_mypage_company_block_path(@company, block), class: :"btn btn-primary"%>
            <%= link_to '削除',
                       mypage_company_block_path(company_id: @company.id, id: block.id),
                       method: :delete,
                       class: :"btn btn-primary",
                       data: { confirm: '本当に削除しますか？'} %></td>
                       <% end %>
                      <% end %>
      </tr>
    <%= form_for ([:mypage, @company, @company.blocks.build]) do |f| %>
    <tr>
      <td><%= f.select(:name, [
        ["トライアル"],
                           ]) %></td>
      <td><%= f.select(:progress, [
        ["1.申込フォーム到着","1.申込フォーム到着"]
                           ]) %></td>
      <td><%= f.date_field :start %></td>
      <td><%= f.date_field :supply %></td>
      <td width="30%" colspan="2"><%= f.text_field :comment %></td>
    </tr>
    <tr>
      <td colspan ="7"><%= f.submit '登録' %></td>
    </tr>
  </tbody>
 </table>
 <% end %>

<table class="company">
 <tbody>

<%= form_for ([:mypage, @company, @company.blocks.build]) do |f| %>
  <tr>
    <td><%= f.select(:name, [
      ["一般トライアルコース"]
                         ]) %></td>
    <td><%= f.select(:progress, [
      ["1.申込フォーム到着","1.申込フォーム到着"],
                         ]) %></td>
    <td><%= f.date_field :start %></td>
    <td><%= f.date_field :supply %></td>
  </tr>
  <tr>
      <th colspan="4">コメント</th>
  </tr>
  <tr>
    <td colspan="4"><%= f.text_area :comment %></td>
  </tr>
  <tr>
    <td colspan ="7"><%= f.submit '登録' %></td>
  </tr>
</tbody>
</table>
<% end %>

class Mypage::BlocksController < Mypage::BaseController
  #before_action :load_company
#  before_action :load_block, only: [:edit,:update,:show,:destroy]

  #  def load_company
  #    @company = Company.find(params[:company_id])
  #  end

  #  def load_block
  #    @block = Block.find(params[:id])
  #  end

    def create
      @block = Block.build
      #@company = Company.find(params[:company_id])
      if @company.blocks.create(block_params)
      redirect_to mypage_company_path(@company)
      end
    end

    def edit
      @company = Company.find(params[:company_id])
      #@block = Block.find(params[:id])
    end

    def update
      @block = Block.find(params[:id])
      #@block = @company.blocks.find(params[:id])
      if @block.update(block_params)
        redirect_to mypage_company_path(@company)
      else
        render 'edit'
      end
    end

    def destroy
      @company = Company.find(params[:company_id])
      @block = @company.blocks.find(params[:id])
      @block.destroy
  		redirect_to mypage_company_path(@company)
    end

  private
    def block_params
      params.require(:block).permit(
        :name,
        :system,
        :progress,
        :start,
        :supply,
        :comment
        )
    end

  end

Answer 1

<%= form_for ([:mypage, @company, @company.blocks.build]) do |f| %>

この時点で、@company.blocks.buildと空のインスタンスを生成してるので、自動的にcreateアクションに飛んでしまうかと思います。

なので、form_forに入る@companyが単一データの場合(@company=Company.find(1)とか)であればupdateアクションに遷移してくれると思います。

また、コントローラ部分のdef updateに関しては

@company.update(message_params)

で良さそうですが、@blockはform_forで使用していませんし、何かの名残でしょうか？

Comments

[KOO_]

ご回答ありがとうございます。
form内は@blockのものになります。
その場合どうすれば良いでしょうか。。。？

[bamboo-nova]

companyではなくblockで、特にパラメータの指定などがなければ
<%= form_for (@block) do |f| %>
で良いと思います。

[KOO_]

お世話になっております。
ActionView::Template::Error (First argument in form cannot contain nil or be empty):
11: </tr>
12: <%= form_for ([:mypage, @block]) do |f| %>
13: <tr>
14: <td><%= f.select(:name, [
15: ["一般トライアルコース"],
app/views/mypage/blocks/edit.html.erb:12:in `_app_views_mypage_blocks_edit_html_erb___4054124701742398534_70352037819860'

その場合以下のようにエラーになってしまいます。。。

[bamboo-nova]

@blockが生成されていないから、form_forのヘルパメソッドが動かせなくなっているようですね。
基本、form_forの引数にするオブジェクトはnilではいけないのですが、
コントローラのメソッドで、 @company = Company.find(params[:company_id])としてますね。これだと、@blockは作られていないことになるので、こちらを削除して@block = Block.find(params[:id])などとする必要があるのではないでしょうか？(:idは適当に推測して入れました)

先ほども言ったのですが、form_forで.buildをしていれば空のインスタンスを生成しているのでcreateアクションとしてコントローラに渡されてしまうので、updateアクションに渡すには単一データのインスタンスが入っている必要があります(要は、@blockが空でなく値が代入されている必要があります)