diff options
| author | syeopite <70992037+syeopite@users.noreply.github.com> | 2021-09-28 15:23:36 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-09-28 15:23:36 +0000 |
| commit | 6df85718e6dac2faa9037fcf2283aa6b5ab819a3 (patch) | |
| tree | e4f99be55a43c27813f998593360172b50f4dd97 | |
| parent | 092b8a4e5220cbe7e6eed45d1c331d5596dc68bc (diff) | |
| download | invidious-6df85718e6dac2faa9037fcf2283aa6b5ab819a3.tar.gz invidious-6df85718e6dac2faa9037fcf2283aa6b5ab819a3.tar.bz2 invidious-6df85718e6dac2faa9037fcf2283aa6b5ab819a3.zip | |
Apply suggestions from code review
Co-authored-by: Samantaz Fox <coding@samantaz.fr>
| -rw-r--r-- | src/invidious/helpers/extractors.cr | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/invidious/helpers/extractors.cr b/src/invidious/helpers/extractors.cr index 32134cc9..0c645868 100644 --- a/src/invidious/helpers/extractors.cr +++ b/src/invidious/helpers/extractors.cr @@ -139,8 +139,8 @@ private module Parsers # When public subscriber count is disabled, the subscriberCountText isn't sent by InnerTube. # Always simpleText # TODO change default value to nil - subscriber_count = item_contents.dig?("subscriberCountText").try &.["simpleText"].try { \ - |s| short_text_to_number(s.as_s.split(" ")[0]) } || 0 + subscriber_count = item_contents.dig?("subscriberCountText", "simpleText") + .try { |s| short_text_to_number(s.as_s.split(" ")[0]) } || 0 auto_generated = !item_contents["videoCountText"]? ? true : false @@ -265,7 +265,8 @@ private module Parsers private def self.parse(item_contents, author_fallback) title = extract_text(item_contents["title"]?) || "" - url = item_contents["endpoint"]?.try &.dig("commandMetadata", "webCommandMetadata", "url").as_s + url = item_contents.dig?("endpoint", "commandMetadata", "webCommandMetadata", "url") + .try &.as_s # Sometimes a category can have badges. badges = [] of Tuple(String, String) # (Badge style, label) @@ -450,7 +451,7 @@ private module HelperExtractors # Returns a 0 when it's unable to do so def self.get_video_count(container : JSON::Any) : Int32 if box = container["videoCountText"]? - return extract_text(container["videoCountText"]?).try &.gsub(/\D/, "").to_i || 0 + return extract_text(box).try &.gsub(/\D/, "").to_i || 0 elsif box = container["videoCount"]? return box.as_s.to_i else |