requireを設定するとエラーが出る

Question

前提

コントローラーでrequireを設定するとエラーが出ます。

実現したいこと

投稿した画像にタグ付けをしたいです。

発生している問題・エラーメッセージ

ActionController::ParameterMissing in DesignsController#create
param is missing or the value is empty: post_form

該当のソースコード

[app>controllers>designs_controller.rb]

class DesignsController < ApplicationController
  before_action :authenticate_user!, only:[:edit, :new, :destroy]
  before_action :move_to_index, except: [:index, :show]
  before_action :set_design, only: [:show, :edit]

  def index
    @designs = Design.includes(:user).order("created_at DESC")
  end

  def new
    @designs = Design.new
  end

  def create
    @designs = Design.create(design_params)
    if 
      @designs.save
      redirect_to root_path
      else
        render :new
    end
  end

  def show

  end

  def destroy
    design = Design.find(params[:id])
    if design.destroy
      redirect_to root_path
    end
  end

  def update
    design = Design.find(params[:id])
    if
    design.update(design_params)
      redirect_to design_path(design.id)
    else
      redirect_to request.referer
    end
  end

  def edit
    unless user_signed_in? && current_user.id == @design.user_id
      redirect_to action: :index
    end
  end

  def search
    @designs = Design.search(params[:keyword]).order("created_at DESC")
  end

  private

  def design_params
    params.require(:post_form).permit(:title, :file_name, :image).merge(user_id: current_user.id)
  end

  def set_design
    @design = Design.find(params[:id])
  end

  def move_to_index
    unless user_signed_in?
      redirect_to action: :index
    end
  end

end

[app>models>post_form.rb]

class PostForm
  include ActiveModel::Model

  attr_accessor(
    :title, :file_name, :image,
    :id, :created_at, :updated_at,
    :tag_name
   )

   with_options presence: true do
    validates :title
    validates :file_name
    validates :image
  end

  def save
    designs = Design.create(text: text, image: image)
    tag = Tag.where(tag_name: tag_name).first_or_initialize
    tag.save
    PostTagRelation.create(design_id: design.id, tag_id: tag.id)
  end

[app>models>post_tag_relation.rb]
class PostTagRelation < ApplicationRecord
  belongs_to :design
  belongs_to :tag
end

[app>models>design.rb]

class Design < ApplicationRecord
  belongs_to :user
  has_many :post_tag_relations
  has_many :tags, through: :post_tag_relations
  has_one_attached :image

  validates :title, presence: true
  validates :file_name, presence: true
  validates :image, presence: true

  def self.search(search)
    if search != ""
      Design.where('title LIKE(?)', "%#{search}%")
    else
      Design.all
    end
  end



end

[app>views>show.html.erb]

<main class="main">
  <div class="inner">
    <div class="prototype__wrapper">
      <p class="prototype__hedding">
        <%= @design.title %>
      </p>
      <% if user_signed_in? && current_user.id == @design.user_id %>
      <div class="prototype__detail"><br>
          <p class="detail__title">ファイル場所</p>
          <p class="detail__message">
            <%= @design.file_name %>
          </p>
        </div>
      <% end %>

      <span class="tag_style"><%= @design.tags.first&.tag_name %></span>

      <div class="prototype__image">
        <%= image_tag @design.image %>
      </div>
    </div>
  </div>
</main>

[app>views>designs>_form.html.erb]
<%= form_with model: @design,  url: url, method: method, id: 'new_post', local: true do |f| %>

  <div class="field">
    <%= f.label :title, "タイトル" %><br />
    <%= f.text_field :title, id:"design_title" %>
  </div>

  <div class="field">
    <%= f.label :file_name, "ファイル名" %><br />
    <%= f.text_area :file_name, class: :form__text, id:"design_file_name" %>
  </div>

  <div class="field">
    <%= f.label :image, "画像" %><br />
    <%= f.file_field :image, id:"design_image" %>
  </div>

 <div class="tag-field">
    <%= f.text_field :tag_name, placeholder: 'add tags' %>
  </div>

  <div class="actions">
    <%= f.submit "保存する", class: :form__btn %>
  </div>
<% end %>

試したこと

requireを削除すると正常に表示されるのですが
今度はタグの保存・表示ができなくなります。

補足情報（FW/ツールのバージョンなど）

初心者でして、初歩的な質問かと思いますが、何卒宜しくお願いいたします。

Answer 1

PostForm の save メソッドを拝見したところ、 create しか対応してないようですので、update の考慮は不要ってことで良いですかね？
とりあえず create だけ考えて回答します。

--

普通に Rails のモデルを使ってフォームを作る場合は、モデル名から「パラメータ名」や「送信先」を推測してくれるんですが、今回は独自のフォームクラスを作られているようなので、オプションを使って設定を上書きする必要があります。
あと、 model オプションの指定も変なので、これも修正が必要です。

１・【model】 PostForm クラスを指定するのではなく、 PostForm クラスのインスタンスを指定します。
２・【送信先】 DesignsController#create に送信したいので、 url: designs_path, method: :post と書きます。
３・【パラメータ名】 require の名前に合わせて scope: :post_form と書きます。

[app>views>designs>_form.html.erb]

-  <%= form_with model:PostForm,  url: new_design_path, local: true do |f| %>
+  <%= form_with model: @post_form, url: design_path, method: :post, scope: :post_form, local: true do |f| %>

※※※ 事前に @post_form を作る必要があるので、コントローラー側にも修正が必要です。後述します ※※※

コントローラー側で PostForm を使うように変更します。

[app>controllers>designs_controller.rb]

  def new
-   @designs = Design.new
+   @post_form = PostForm.new
  end

  def create
-   @designs = Design.create(design_params)
+   @post_form = PostForm.new(design_params)

-   if @designs.save
+   if @post_form.save
      redirect_to root_path
    else
      render :new
    end
  end

あと、パラメータの名前が view と controller で異なっているので、これも修正してください。
（誤 tag → 正 tag_name）

  def design_params
-   params.require(:post_form).permit(:title, :file_name, :tag, :image).merge(user_id: current_user.id)
+   params.require(:post_form).permit(:title, :file_name, :tag_name, :image).merge(user_id: current_user.id)
  end

PostForm の処理もちょっと変なので直しましょう。

class PostForm
  include ActiveModel::Model

  attr_accessor(
    :title, :file_name, :image,
-   :id, :created_at, :updated_at,
+   :id, :user_id,
+   # ↑ created_at と updated_at は画面から受け取るパラメータではないので不要だと思います。
+   # ↑ 逆に user_id は design_params に含まれているので必要だと思います。
    :tag_name
  )

  with_options presence: true do
    validates :title
    validates :file_name
    validates :image
  end

  def save
+   # ↓ 複数モデルを扱うのでトランザクションが必須です。
+   ActiveRecord::Base.transaction do

-     designs = Design.create(text: text, image: image)
+     design = Design.create!(title: title, text: text, image: image, user_id: user_id)
+     # ↑ title に必須バリデーションを設定しているのに、保存時に使ってないのはおかしいと思います。
+     # ↑ user_id も必要ではないですか？
+     # ↑ トランザクションのために「 ! 」を付けます。
+     # ↑ 単一のレコードなのに変数名が複数形なのはおかしいです。

      tag = Tag.where(tag_name: tag_name).first_or_initialize
-     tag.save
+     tag.save!
+     # ↑ トランザクションのために「 ! 」を付けます。

-     PostTagRelation.create(design_id: design.id, tag_id: tag.id)
+     PostTagRelation.create!(design_id: design.id, tag_id: tag.id)
+     # ↑ トランザクションのために「 ! 」を付けます。

+   end

+   return true
+   # ↑ 「 save! 」や「 create! 」が一つもエラーにならなかったら true を返す。

+ rescue
+   false
+   # ↑ 「 save! 」や「 create! 」が一つでもエラーになったら false を返す。

  end

--

多分これで最低限 create が動くと思います。お試しください。

「一つの design に複数の tag を付ける」とか「update にも対応させる」とかやりたい場合は別途修正が必要です。

Answer 2

[app>controllers>designs_controller.rb]
def design_params
    params.require(:post_form).permit(:title, :file_name, :tag, :image).merge(user_id: current_user.id)
  end

[app>views>designs>_form.html.erb]
<%= form_with model:PostForm,  url: new_design_path, local: true do |f| %>

上記でエラーは出なくなりましたがタグが保存されず、表示もされません…。

Comments

[shinoharat]

送信先が new アクションになったから、「design_params」が呼び出されなくなってエラーが消えただけですね。

保存したいなら create アクションに飛ばす必要があります。

[tamtamtime]

ご回答、コメントありがとうございます！！
勉強になる事も多々あり、本当にありがとうございます！！

PostFormを修正したのですが、以下のようなエラーが出ました。

NameError in DesignsController#create

undefined local variable or method `text' for #<PostForm:0x00007fd374f5d3a0> Did you mean? test
Extracted source (around line #20):

18 def save
19 design = Design.create!(title: title, text: text, image: image, user_id: user_id)
20 tag = Tag.where(tag_name: tag_name).first_or_initialize
21 tag.save!
22 PostTagRelation.create!(design_id: design.id, tag_id: tag.id)

「text」を「file_name」に変更してもエラーになるのですが、どこかおかしいのでしょうか？
度々恐れ入ります、ご教示のほど何卒お願いいたします。

[shinoharat]

すみません、見落としていました。
対応としては

> 「text」を「file_name」に変更

で合ってると思います。
保存処理を

design = Design.create!(title: title, file_name: file_name, image: image, user_id: user_id)

に変えて実行し、表示されるエラーメッセージを教えてください。

[tamtamtime]

ありがとうございます！タグ保存されました！
本当にありがとうございます！！