問題
brakemanを使用時に == Warnings == として
Confidence: Weak Category: HTTP Verb Confusion Check: VerbConfusion Message: Potential HTTP verb confusion. `HEAD` is routed like `GET` but `request.get?` will return `false` Code: session[:forwarding_url] = request.original_url if request.get?
というエラーが表示されました。
Message: Potential HTTP verb confusion. `HEAD` is routed like `GET` but `request.get?` will return `false`
この部分を翻訳すると、
request.get?
は false
を返します。
とのことで、この部分を直せば問題はないだろうと思ったので
session[:forwarding_url] = request.original_url if request.get?
ここを
session[:forwarding_url] = request.original_url unless !request.get?
と書き換えたことによって、エラーは表示されなくなりました。
##質問
質問なのですが、この部分は書き直すべきなのでしょうか?
エラーは出ていないため、正しく表示されるのですが、Railsの慣習的に書き直すべきなのかどうかをご教授頂けますと幸いです。
至らない点があると思いますが、よろしくお願いいたします。
あなたの回答
tips
プレビュー