Clear up `resolve_link_as`

Swaps to case/when to highlight the 3 branches better and uses `presence_in`
to cut down on the else branch noise.

`resolve_link_as` added in: eb90b8bc86
Most recent other commit: 46bfd082b0

Made a decision to tweak this as core, don't send cosmetic PRs.
This commit is contained in:
Kasper Timm Hansen 2021-08-24 22:38:40 +02:00
parent c7647f8842
commit 915b9cdbb7
No known key found for this signature in database
GPG Key ID: 191153215EDA53D8
1 changed files with 6 additions and 8 deletions

View File

@ -530,14 +530,12 @@ module ActionView
end
def resolve_link_as(extname, mime_type)
if extname == "js"
"script"
elsif extname == "css"
"style"
elsif extname == "vtt"
"track"
elsif (type = mime_type.to_s.split("/")[0]) && type.in?(%w(audio video font image))
type
case extname
when "js" then "script"
when "css" then "style"
when "vtt" then "track"
else
mime_type.to_s.split("/").first.presence_in(%w(audio video font image))
end
end