"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_name
、action_name
とかでシンプルにArray#include?
とかcase/when
とか==
とかで比較したほうがわかりやすそうな気がしています。
例えば、コントローラ側でどういうアクションが対象かを定義しておいて、
class LocationsConroller < ApplicationController
HOGE_ACTIONS = %w[index new]
end
ビューで判定するとかですね。
- if LocationsConroller::HOGE_ACTIONS.include?(action_name)
定数定義やロジックをLocationsHelper
に切り出してもいいと思います。