0

現在、下記のようなコードになっており、

current_page?(controller: 'foo', action: 'bar')

今回は、actionを増やしたい(違うactionでも動かせる)と考えています。
しかし条件演算子でかいてもうまくいかないです。
下記のように書いても、片方のactionしか動かないです

- if current_page?(controller: 'locations', action: "new" || "index")

どのようにかけばうまく動作させることができるのか教えていただきたいです。
よろしくお願い致します。

2 件の回答 2

1

"new" || "index"は常に"new"にしかならないので、current_page?(controller: 'locations', action: "new")と書いたのと同じなのが、意図どおりに動かない原因だと思います。

current_page?の実装を見ると途中でurl_forで変換されてしまうようなので1度に判定するのは難しそうです。

愚直な案でよければ以下のようにやれば動くかと思います。

案1. case/whenで1つずつ判定する

- case
- when current_page?(controller: 'locations', action: "new")
- when current_page?(controller: 'locations', action: "index")

案2. Array#any?を使う

- if %w[new index].any? {|action| current_page?(controller: 'locations', action: action) }

実際の使われ方とか周辺コードがわからないのであれですが……個人的には、current_page?って必要以上に複雑な感じがするのでcontroller_nameaction_nameとかでシンプルにArray#include?とかcase/whenとか==とかで比較したほうがわかりやすそうな気がしています。

例えば、コントローラ側でどういうアクションが対象かを定義しておいて、

class LocationsConroller < ApplicationController
  HOGE_ACTIONS = %w[index new]
end

ビューで判定するとかですね。

- if LocationsConroller::HOGE_ACTIONS.include?(action_name)

定数定義やロジックをLocationsHelperに切り出してもいいと思います。

0
0

おまけでちょいやりすぎな別案を載せておきます。

いっそのことcurrent_page?の挙動を変えてしまえーという暴挙です。

# ヘルパー
module ApplicationHelper
  def current_page?(options)
    if options.kind_of?(Hash) && options[:actions]
      options[:actions].any? {|action| super(options.merge(action: action)) }
    else
      super
    end
  end
end

# ビュー
- if current_page?(controller: 'locations', actions: %w[new index])

current_page?に渡す引数がactionではなくactionsなのに注意してください。

4
  • コメントありがとうございます。下の強行策にでたのですが、routing errorとなってしまいます。
    – p-1037
    Commented 2016年9月6日 9:18
  • options[:actions].any? {|action| super(options.merge(action: action)) } がどうやらエラー原因のようなのですがこれは、どういった記述都なっているのでしょうか??
    – p-1037
    Commented 2016年9月6日 9:20
  • 検証してないので間違っている可能性もありますが、エラー行の上にactions = options.delete(:actions)を追加して、エラー行のoptions[:actions]actionsにしてみてください。それでもダメなら、もう少し詳しい情報をいただけるとありがたいです。
    – kg8m
    Commented 2016年9月6日 10:58
  • コードの意味については、わからないのはどの部分でしょうか。それによって回答も変わってきます。
    – kg8m
    Commented 2016年9月6日 11:02

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。