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

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

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

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

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

Q&A

解決済

1回答

256閲覧

除外しているアクションにもprivateメソッドが適用されてしまう

imamoto_browser

総合スコア1161

Ruby

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

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

0グッド

0クリップ

投稿2018/04/25 17:03

controllerで適用を除外しているアクションにもprivateメソッドが適用されてしまいます。
onlyもexceptも試したのですが、そのprivateメソッドのエラーが出てしまいます。

そもそもなんですが、独自名のアクションって追加すべきではないのでしょうか。(下記の場合、def user_detail)

#環境
rails4.2.1

#やりたいこと
show.html.erbのリンクからuser_detailにエラーなく遷移したい。

#エラー内容

ActiveRecord::RecordNotFound in ProjectsController#show

Couldn't find Project with 'id'=user_detail [WHERE "projects"."deleted_at" IS NULL]

ruby

1 def set_project 2 @project = Project.find(params[:id]) 3 end 4end

#ソースコード
controller

ruby

1class ProjectsController < AuthorizedController 2 PER = 5 3 #before_action :set_project, only: [:show,:edit, :update, :destroy] ,except: [:user_detail] 4 before_action :set_project, except: :user_detail 5 6 def show 7 #render plain: params[:id].inspect 8 @user = current_user 9 @r = Project.joins("LEFT OUTER JOIN reviews ON projects.id = reviews.project_id LEFT OUTER JOIN user_items ON user_items.user_id = reviews.user_id").where("reviews.project_id = ?" , params[:id]).select("reviews.* , user_items.*") 10 11 end 12 def user_detail 13 14 #render plain: params[:id].inspect 15 16 @ra = Project.joins("LEFT OUTER JOIN reviews ON projects.id = reviews.project_id LEFT OUTER JOIN user_items ON user_items.user_id = reviews.user_id").where("reviews.user_id = ?" , params[:id]).select("projects.*, reviews.* , user_items.*") 17 18 @r = Project.joins("LEFT OUTER JOIN reviews ON projects.id = reviews.project_id LEFT OUTER JOIN user_items ON user_items.user_id = reviews.user_id").where("reviews.project_id = ?" , params[:id]).select("reviews.* , user_items.*") 19 20 end 21 22 private 23 24 def set_project 25 @project = Project.find(params[:id]) 26 end 27

view

ruby

1 <% @r.each do |userlist| %> 2 <%= link_to userlist.user, projects_user_detail_path ,id: userlist.user_id %> 3 : 4 <%= add_star %> 5 <b><%= userlist.star %></b> 6 <%= userlist.review %> 7 <br> 8 <% end %> 9

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

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

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

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

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

guest

回答1

0

ベストアンサー

状況としては、/projects/user_detailが、showアクションに流れてしまっています。routes.rbで、resourcesより前に/projects/user_detailへのrouteを追加してください。

そもそもなんですが、独自名のアクションって追加すべきではないのでしょうか。

Railsの創始者であるDHH氏はその見解です

投稿2018/04/25 22:46

maisumakun

総合スコア145183

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

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

imamoto_browser

2018/04/26 06:13

いつも回答ありがとうございます。 routeをresourceの前に書いてみましたが、やはりbefore_actionに行ってしまうようです。貼っていただいたリンクの内容に賛同できる部分が多く、今回はUserDetailsControllerを作ることにしました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.49%

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

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

質問する

関連した質問