videos: Fix extraction code according to tests

pull/3238/head
Samantaz Fox 2 years ago
parent 33150f5de3
commit e23ceb6ae9
No known key found for this signature in database
GPG Key ID: F42821059186176E

@ -666,10 +666,9 @@ def parse_video_info(video_id : String, player_response : Hash(String, JSON::Any
raise BrokenTubeException.new("twoColumnWatchNextResults") if !main_results raise BrokenTubeException.new("twoColumnWatchNextResults") if !main_results
primary_results = main_results.dig?("results", "results", "contents") # Primary results are not available on Music videos
# See: https://github.com/iv-org/invidious/pull/3238#issuecomment-1207193725
raise BrokenTubeException.new("results") if !primary_results if primary_results = main_results.dig?("results", "results", "contents")
video_primary_renderer = primary_results video_primary_renderer = primary_results
.as_a.find(&.["videoPrimaryInfoRenderer"]?) .as_a.find(&.["videoPrimaryInfoRenderer"]?)
.try &.["videoPrimaryInfoRenderer"] .try &.["videoPrimaryInfoRenderer"]
@ -680,6 +679,7 @@ def parse_video_info(video_id : String, player_response : Hash(String, JSON::Any
raise BrokenTubeException.new("videoPrimaryInfoRenderer") if !video_primary_renderer raise BrokenTubeException.new("videoPrimaryInfoRenderer") if !video_primary_renderer
raise BrokenTubeException.new("videoSecondaryInfoRenderer") if !video_secondary_renderer raise BrokenTubeException.new("videoSecondaryInfoRenderer") if !video_secondary_renderer
end
video_details = player_response.dig?("videoDetails") video_details = player_response.dig?("videoDetails")
microformat = player_response.dig?("microformat", "playerMicroformatRenderer") microformat = player_response.dig?("microformat", "playerMicroformatRenderer")
@ -691,8 +691,11 @@ def parse_video_info(video_id : String, player_response : Hash(String, JSON::Any
title = video_details["title"]?.try &.as_s title = video_details["title"]?.try &.as_s
# We have to try to extract viewCount from videoPrimaryInfoRenderer first,
# then from videoDetails, as the latter is "0" for livestreams (we want
# to get the amount of viewers watching).
views = video_primary_renderer views = video_primary_renderer
.dig?("viewCount", "videoViewCountRenderer", "viewCount", "runs", 0, "text") .try &.dig?("viewCount", "videoViewCountRenderer", "viewCount", "runs", 0, "text")
.try &.as_s.to_i64 .try &.as_s.to_i64
views ||= video_details["viewCount"]?.try &.as_s.to_i64 views ||= video_details["viewCount"]?.try &.as_s.to_i64
@ -825,7 +828,7 @@ def parse_video_info(video_id : String, player_response : Hash(String, JSON::Any
if live_now if live_now
video_type = VideoType::Livestream video_type = VideoType::Livestream
elsif premiere_timestamp.not_nil! elsif !premiere_timestamp.nil?
video_type = VideoType::Scheduled video_type = VideoType::Scheduled
published = premiere_timestamp || Time.utc published = premiere_timestamp || Time.utc
else else
@ -861,7 +864,7 @@ def parse_video_info(video_id : String, player_response : Hash(String, JSON::Any
"author" => JSON::Any.new(author || ""), "author" => JSON::Any.new(author || ""),
"ucid" => JSON::Any.new(ucid || ""), "ucid" => JSON::Any.new(ucid || ""),
"authorThumbnail" => JSON::Any.new(author_thumbnail.try &.as_s || ""), "authorThumbnail" => JSON::Any.new(author_thumbnail.try &.as_s || ""),
"authorVerified" => JSON::Any.new(author_verified), "authorVerified" => JSON::Any.new(author_verified || false),
"subCountText" => JSON::Any.new(subs_text || "-"), "subCountText" => JSON::Any.new(subs_text || "-"),
} }

Loading…
Cancel
Save