DBに保存できるようにしたい

Question

DBに保存が少し前まではできていたが、現在保存ボタンを押してもDBに保存できなくなっているのを解決したい

Rails.application.routes.draw do
  root to: 'cards#index' #トップページへ遷移
  resources :cards do#7つのアクションの省略
    collection do
      get 'search'
    end
    resources :categories, only: [:show]
  end
  

end

ルーティングはこのようにしています。

class CardsController < ApplicationController
  before_action :set_card, only: [:edit, :show]
  before_action :goback, only: [:create, :update, :destroy]
  def index
    @cards = Card.all
  end

  def new
    @card = Card.new
  end

  def create
    Card.create(cards_params)
    #redirect_to '/cards'
  end

  def edit
    #@card = Card.find(params[:id])
  end

  def update
    card = Card.find(params[:id])
    card.update(cards_params)
    #redirect_to '/cards'
  end

  def destroy
    card = Card.find(params[:id])
    card.destroy
    #redirect_to '/cards'
  end

  def show
    #@card = Card.find(params[:id])
  end

  def search
    @cards = Card.search(params[:keyword])
    @keyword = params[:keyword]
  end

  private

  def cards_params
    params.require(:card).permit(:language, :title, :releasedate, :price, :author, :memo)
  end

  def set_card
    @card = Card.find(params[:id])
  end

  def goback
    redirect_to '/cards'
  end
end

コントローラーはこのようにしています。
cretaeアクションでカラムを保存できるようにしています

class Card < ApplicationRecord
  belongs_to :category, optional: true
  def self.search(search) #self.クラスメソッド
    if search != ""
      Card.where('language LIKE(?) or title LIKE(?)', "%#{search}%","%#{search}%").order('id DESC') #言語、タイトルで検索することが可能
    else
      order('Id DESC')
    end
  end
end

class Category < ApplicationRecord
  has_many :cards
end

モデルはこの二つでアソシエーションを付けています。

.Add
  .Add__lists
    %h1 登録する本のデータの追加
    .Add__lists__list
      = form_for @card, url: cards_path, method: :post do |f|
        .Add__lists__list__group
          = f.label :言語
          = f.text_field :language, autofocus: true, placeholder: "例)言語を記入", class: "input-default"
        .Add__lists__list__group
          = f.label :タイトル
          = f.text_field :title, autofocus: true, placeholder: "例)タイトル記入", class: "input-default"
        .Add__lists__list__group
          = f.label :著者
          = f.text_field :author, autofocus: true, placeholder: "例)著者を記入", class: "input-default"
        .Add__lists__list__group
          = f.label :価格
          = f.text_field :price, autofocus: true, placeholder: "例)価格を記入", class: "input-default"
        .Add__lists__list__group
          = f.label :出版日
          = f.text_field :releasedate, autofocus: true, placeholder: "例)出版日を記入", class: "input-default"
        .Add__lists__list__group
          = f.label :コメント
          = f.text_area :memo, autofocus: true, placeholder: "例)コメントを記入", class: "input-default"
        .Add__lists__list__group
          = f.submit "保存", class: "save-btn"

= render partial: "top"

form_forで保存できるようにしています。保存ボタンを押すとトップページに遷移します。

binding.pryをcretaeアクションのところにかけても、エラーなく、保存ボタンを押すとトップに戻ります。

どなたか教えていただけませんか。宜しくお願いいたいします。

Answer 1

before_action :goback, only: [:create, :update, :destroy]

def goback
redirect_to '/cards'
end

ここでリダイレクトしているからです。

Comments

[退会済みユーザー]

ありがとうございます！

保存ボタンを押したらそのままトップページにいくようにするにはどのようにしたらいいでしょうか？

[rhiroe]

> 保存ボタンを押すとトップに戻ります。

現在がそのようになっているのですよね？

[退会済みユーザー]

redirect_toを削除すると、保存できるようになり、削除したのでトップに戻らないようになります。

gobackでトップになるようにしているためです

[rhiroe]

保存した後にredirectするようにすれば良いと思います。

[退会済みユーザー]

```ryby:/controller.rb
def cretae
def create
Card.create(cards_params)
#redirect_to '/cards'
end
```
before_actionのcretaeを外せばいいということですね