質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.46%
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

リダイレクト

プログラムの入力元や出力先を通常とは別の場所に転送させることをリダイレクトと呼びます。

Q&A

解決済

1回答

2749閲覧

投稿者以外のユーザーが、投稿者のプロトタイプ編集ページに遷移できないようにしたい

LYFT

総合スコア1

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

リダイレクト

プログラムの入力元や出力先を通常とは別の場所に転送させることをリダイレクトと呼びます。

0グッド

0クリップ

投稿2021/06/04 13:47

前提・実現したいこと

prototypesコントローラーにおいて、投稿者以外がURLを直接入力しeditアクションに
アクセスしたら、トップページにリダイレクトするように設定したい。

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

エラーメッセージ

NoMethodError in PrototypesController#show undefined method `user_id' for nil:NilClass

コード一覧

prototypes_controller.rb

class PrototypesController < ApplicationController before_action :authenticate_user!, expect: [:index, :show] before_action :move_to_index, expect: :index def index @prototypes = Prototype.includes(:user).order("created_at DESC") end def new @prototype = Prototype.new end def create @prototype = Prototype.new(prototype_params) if @prototype.save redirect_to root_path else render :new end end def show @prototype = Prototype.find(params[:id]) @comment = Comment.new @comments = @prototype.comments.includes(:user) end def edit @prototype = Prototype.find(params[:id]) unless @prototype.user_id == current_user.id redirect_to action: :index end end def update @prototype = Prototype.find(params[:id]) if @prototype.update(prototype_params) redirect_to prototype_path else render :edit end end def destroy @prototype = Prototype.find(params[:id]) if @prototype.destroy redirect_to root_path end end def move_to_index unless user_signed_in? && current_user.id == @prototype.user_id redirect_to action: :index end end private def prototype_params params.require(:prototype).permit(:title, :catch_copy, :concept, :image).merge(user_id: current_user.id) end end

routes.rb

Rails.application.routes.draw do devise_for :users root to: "prototypes#index" resources :prototypes, only: [:new,:create,:show,:edit,:update,:destroy] do resources :comments, only: :create end resources :users, only: :show end

試したこと

prototypes_controller.rb記載のdef move_to_indexの変数を
色々と変えて試しましたが、どうすればトップページに遷移するのか、
答えを見つけられませんでした。

是非、アドバイスをいただき目的を達成したく、
ご指導の程宜しくお願い致します。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

unless user_signed_in? && current_user.id == @prototype.user_id

@prototypeはおそらく、この時点では代入されていませんのでnilです。

ruby

1 def move_to_index 2 @prototype = Prototype.find(params[:id]) 3 unless user_signed_in? && current_user.id == @prototype.user_id 4 redirect_to action: :index 5 end 6 end

before_action :authenticate_user!, expect: [:index, :show]
before_action :move_to_index, expect: :index

本件とは関係ありませんがexpectではなくexceptです。

投稿2021/06/04 17:32

asm

総合スコア15147

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

winterboum

2021/06/04 23:55

いろんな actionで共通していますから before_action で set_prototype という手もありますね
asm

2021/06/05 00:46

set_系before_actionは害悪だという人もいますし各人が判断することかなと思います。
LYFT

2021/06/05 03:44

お二人ともご対応いただきありがとうございました! 無事解決致しました。 違うところのスペルミスまで発見していただけるとは感動です!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.46%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問