Skip to content

Commit

Permalink
Ameba: Fix Lint/UselessAssign (#4795)
Browse files Browse the repository at this point in the history
Related to issue 2231
  • Loading branch information
SamantazFox committed Aug 11, 2024
2 parents 80ffc44 + fad0a4f commit 5e38ef5
Show file tree
Hide file tree
Showing 8 changed files with 6 additions and 12 deletions.
1 change: 0 additions & 1 deletion spec/invidious/search/iv_filters_spec.cr
Original file line number Diff line number Diff line change
Expand Up @@ -301,7 +301,6 @@ Spectator.describe Invidious::Search::Filters do

it "Encodes features filter (single)" do
Invidious::Search::Filters::Features.each do |value|
string = described_class.format_features(value)
filters = described_class.new(features: value)

expect("#{filters.to_iv_params}")
Expand Down
2 changes: 1 addition & 1 deletion src/invidious/channels/channels.cr
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ def fetch_channel(ucid, pull_all_videos : Bool)
id: video_id,
title: title,
published: published,
updated: Time.utc,
updated: updated,
ucid: ucid,
author: author,
length_seconds: length_seconds,
Expand Down
4 changes: 2 additions & 2 deletions src/invidious/frontend/misc.cr
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,9 @@ module Invidious::Frontend::Misc

if prefs.automatic_instance_redirect
current_page = env.get?("current_page").as(String)
redirect_url = "/redirect?referer=#{current_page}"
return "/redirect?referer=#{current_page}"
else
redirect_url = "https://redirect.invidious.io#{env.request.resource}"
return "https://redirect.invidious.io#{env.request.resource}"
end
end
end
2 changes: 1 addition & 1 deletion src/invidious/helpers/handlers.cr
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ class AuthHandler < Kemal::Handler
if token = env.request.headers["Authorization"]?
token = JSON.parse(URI.decode_www_form(token.lchop("Bearer ")))
session = URI.decode_www_form(token["session"].as_s)
scopes, expire, signature = validate_request(token, session, env.request, HMAC_KEY, nil)
scopes, _, _ = validate_request(token, session, env.request, HMAC_KEY, nil)

if email = Invidious::Database::SessionIDs.select_email(session)
user = Invidious::Database::Users.select!(email: email)
Expand Down
2 changes: 1 addition & 1 deletion src/invidious/user/imports.cr
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ struct Invidious::User
playlist = create_playlist(title, privacy, user)
Invidious::Database::Playlists.update_description(playlist.id, description)

videos = item["videos"]?.try &.as_a?.try &.each_with_index do |video_id, idx|
item["videos"]?.try &.as_a?.try &.each_with_index do |video_id, idx|
if idx > CONFIG.playlist_length_limit
raise InfoException.new("Playlist cannot have more than #{CONFIG.playlist_length_limit} videos")
end
Expand Down
4 changes: 0 additions & 4 deletions src/invidious/videos.cr
Original file line number Diff line number Diff line change
Expand Up @@ -394,10 +394,6 @@ end
def fetch_video(id, region)
info = extract_video_info(video_id: id)

allowed_regions = info
.dig?("microformat", "playerMicroformatRenderer", "availableCountries")
.try &.as_a.map &.as_s || [] of String

if reason = info["reason"]?
if reason == "Video unavailable"
raise NotFoundException.new(reason.as_s || "")
Expand Down
1 change: 0 additions & 1 deletion src/invidious/yt_backend/extractors.cr
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,6 @@ private module Parsers
end

live_now = false
paid = false
premium = false

premiere_timestamp = item_contents.dig?("upcomingEventData", "startTime").try { |t| Time.unix(t.as_s.to_i64) }
Expand Down
2 changes: 1 addition & 1 deletion src/invidious/yt_backend/extractors_utils.cr
Original file line number Diff line number Diff line change
Expand Up @@ -83,5 +83,5 @@ end

def extract_selected_tab(tabs)
# Extract the selected tab from the array of tabs Youtube returns
return selected_target = tabs.as_a.select(&.["tabRenderer"]?.try &.["selected"]?.try &.as_bool)[0]["tabRenderer"]
return tabs.as_a.select(&.["tabRenderer"]?.try &.["selected"]?.try &.as_bool)[0]["tabRenderer"]
end

0 comments on commit 5e38ef5

Please sign in to comment.