railsで def createができません

Question

前提・実現したいこと

入力したモデルを保存したいです
cocoonを使っています。

エラー・困っていること

保存したいのですが、def createを通過してしまいます。
また、 binding.pryも入れていたのですが、def createに行ってくれないため止まってくれません。

ソースコード

コントローラー

class ItemsController < ApplicationController
    before_action :set_article, only:[:new, :show, :edit, :update, :destroy]

    def index
    end

    def new
        if user_signed_in?
        else
            flash[:alert] = "ログインしてください。"
            redirect_to article_path(id: @article.id)
        end
    end

    def create
        if @article.save
            flash[:success] = "アイテムが追加されました！"
            redirect_to new_item_path(id: @item.article_id)
        else
            flash[:alert] = "アイテムの追加に失敗しました。"
            redirect_to new_item_path(id: @item.article_id)
        end
    end

    def destroy
    end

    private

    def set_article
        @article = Article.find(params[:id])
    end

    def create_params
        params.require(:article).permit(:name, :content, :image, :user_id, items_attributes: [:id, :name, :image])
    end

    def correct_user
        @item = current_user.items.find_by(id: params[:id])
        redirect_to root_url if @item.nil?
    end
end

モデル

class Item < ApplicationRecord
  belongs_to :article, optional: true
end

ビュー

        <form>
            <h2アイテムを追加する</h2>
            <%= form_for(@article) do |f| %>
                <%= f.fields_for :items do |i| %>
                <div class="item">
                    <%= i.hidden_field :article_id, value: @article.id %>
                    <div class="field">
                        <label>アイテム名</label>
                        <%= i.text_field :name %>
                    </div>
                    <div class="field file-field">
                        <label>画像</label>
                        <%= i.file_field :image, :type => "file" %> 
                    </div>
                </div>
                <% end %>
                <div class="add-form">
                    <%= link_to_add_association '追加する', f, :items, partial: 'shared/add_form' %>
                </div>
               <div class="form-btn">
                    <%= f.submit "アイテムを追加する" %>
                </div>     
            <% end %>     
        </form>

Articleコントローラー

class ArticlesController < ApplicationController
    include SetupOgbImage
    
    before_action :set_article, only:[:show, :edit, :update, :destroy]
    before_action :correct_user, only: [:edit]

    def index
    end

    def new
        if user_signed_in?
          @article = current_user.articles.build
        else
          flash[:alert] = "ログインしてください。"
          redirect_to root_path
        end
    end

    def create
        @article = current_user.articles.create(create_params)
        if @article.save
          flash[:success] = "記事が作成されました！"
          redirect_to article_path(id: @article.id)
        else
          flash[:alert] = "記事の作成に失敗しました。"
          redirect_to new_article_path
        end
    end

    def show
        @item = @article.items.all
    end

    def edit
    end

    def update
    end

    def destroy
    end

    private

    def set_article
        @article = Article.find(params[:id])
    end

    def create_params
        params.require(:article).permit(:name, :content, :image, :user_id, items_attributes: [:id, :name, :image])
    end

    def correct_user
        @article = current_user.articles.find_by(id: params[:id])
        redirect_to root_url if @article.nil?
    end 
end

Comments

[GoToWorks]

articleのコントローラーってどうなっていますか？

[rails1]

articleコントローラーを追記させていただきました。

Answer 1

maisumakunさんもおっしゃっている通り、外側のformは余計ですので外してください。

あと気になったのはform_for(@article)だと、@articleが保存済みの場合は自動的にupdate用のフォームになってしまいます。
厳密にフォーム先を定義したいなら、Rails5ではform_withで定義することをお勧めします。
https://qiita.com/hmmrjn/items/24f3b8eade206ace17e2

Comments

[rails1]

> dialbirdさま

ありがとうございます。おっしゃる通り、updateに飛んでいたのですが、update actionが記載がなかったため動いていないということが分かりました。

Answer 2

1つ考えられることとして、<form>が2重になっていることがあります（form_forも<form>を生成します）。外側の<form>と</form>を消してみたらどうなりますでしょうか。

Comments

[rails1]

> maisumakunさま

ご回答いただきましてありがとうございます。
いただいたようにformを一つに統一してみました。
解決したみたいです。