From dc2f0055cc49a9a40c7546250b0a579b7c808f27 Mon Sep 17 00:00:00 2001
From: Patrick Kan <55383971+patrickkfkan@users.noreply.github.com>
Date: Fri, 19 Aug 2022 17:02:01 +0800
Subject: [PATCH] feat: improve parsing (#145)

* fix: err in `MusicDetailHeader` when no duration

* feat: get video duration from more places
---
 src/parser/classes/MusicDetailHeader.js       | 2 +-
 src/parser/classes/MusicResponsiveListItem.ts | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/parser/classes/MusicDetailHeader.js b/src/parser/classes/MusicDetailHeader.js
index 191c558f..bc34a00e 100644
--- a/src/parser/classes/MusicDetailHeader.js
+++ b/src/parser/classes/MusicDetailHeader.js
@@ -14,7 +14,7 @@ class MusicDetailHeader extends YTNode {
     this.second_subtitle = new Text(data.secondSubtitle);
     this.year = this.subtitle.runs.find((run) => (/^[12][0-9]{3}$/).test(run.text))?.text || null;
     this.song_count = this.second_subtitle.runs[0].text;
-    this.total_duration = this.second_subtitle.runs[2].text;
+    this.total_duration = this.second_subtitle.runs[2]?.text || '';
     this.thumbnails = Thumbnail.fromResponse(data.thumbnail.croppedSquareThumbnailRenderer.thumbnail);
     this.badges = Parser.parse(data.subtitleBadges);
 
diff --git a/src/parser/classes/MusicResponsiveListItem.ts b/src/parser/classes/MusicResponsiveListItem.ts
index 61181941..d2fbcf55 100644
--- a/src/parser/classes/MusicResponsiveListItem.ts
+++ b/src/parser/classes/MusicResponsiveListItem.ts
@@ -178,7 +178,8 @@ class MusicResponsiveListItem extends YTNode {
       }));
     }
 
-    const duration_text = this.#flex_columns[1].key('title').instanceof(Text).runs?.find((run) => (/^\d+$/).test(run.text.replace(/:/g, '')))?.text;
+    const duration_text = this.#flex_columns[1].key('title').instanceof(Text).runs?.find((run) => (/^\d+$/).test(run.text.replace(/:/g, '')))?.text ||
+      this.#fixed_columns[0]?.key('title').instanceof(Text).runs?.find((run) => (/^\d+$/).test(run.text.replace(/:/g, '')))?.text;
     duration_text && (this.duration = {
       text: duration_text,
       seconds: timeToSeconds(duration_text)