Skip to content

Commit

Permalink
Use if/else instead of return in search route
Browse files Browse the repository at this point in the history
  • Loading branch information
SamantazFox committed Jun 14, 2021
1 parent a2f5342 commit 3de92b3
Showing 1 changed file with 16 additions and 17 deletions.
33 changes: 16 additions & 17 deletions src/invidious/routes/search.cr
Original file line number Diff line number Diff line change
Expand Up @@ -45,27 +45,26 @@ class Invidious::Routes::Search < Invidious::Routes::BaseRoute
# Display the full page search box implemented in #1977
env.set "search", ""
templated "search_homepage", navbar_search: false
return
end
else
page = env.params.query["page"]?.try &.to_i?
page ||= 1

page = env.params.query["page"]?.try &.to_i?
page ||= 1
user = env.get? "user"

user = env.get? "user"
begin
search_query, count, videos, operators = process_search_query(query, page, user, region: region)
rescue ex
return error_template(500, ex)
end

begin
search_query, count, videos, operators = process_search_query(query, page, user, region: region)
rescue ex
return error_template(500, ex)
end
operator_hash = {} of String => String
operators.each do |operator|
key, value = operator.downcase.split(":")
operator_hash[key] = value
end

operator_hash = {} of String => String
operators.each do |operator|
key, value = operator.downcase.split(":")
operator_hash[key] = value
env.set "search", query
templated "search"
end

env.set "search", query
templated "search"
end
end

0 comments on commit 3de92b3

Please sign in to comment.