FreeTube Incorrect labelling on live premier videos

Behavior of the program Live premiere videos that is not live yet labeled as Live on freetube

Expected behavior Live premiere videos labeled as such e.g:

To Reproduce Please add all steps to reproduce the behavior: 2. Click on 'Subscription" 4. See error

Screenshots:

Full screenshot:

**Desktop **

OS: Windows 10
OS Version: 1809
FreeTube version: 0.9.1
FreeTube build: [Nightly or Release]

Additional Information Youtube URL: https://youtu.be/VYu3PvRPZR8

Oct 31 '20 21:10 EggY0lk

Interesting, This only seems to happen if there are "viewers" waiting in the premiere room. Here are both situations in one picture.

Nov 01 '20 19:11 DuaneJaspers

I think, it would be best to leave the watching count in, but change the 'Live' to Premiere as @SwaggyHerosity showed in the screenshot. Then it is clear that it is a premiere but you also see how many people are waiting for it to start

Nov 02 '20 10:11 GilgusMaximus

Other than indicating that it is a premiere rather than a live video, it would be neat if the scheduled date and time are also included in the subscriptions page/search page (on the thumbnail or beside the view count), if possible.

May 30 '21 16:05 computeristic

Workaround: use Invidious API instead of Local API (Settings>General Settings>Preferred API Backend)

Aug 27 '22 06:08 gamer191

I spent a bit looking into this issue. Even though this sounds easy to do, but as I've come to find out, it's not even related to this repo.

I first looked at the ft-list-video to check how the labels were rendered. I ended up in the function parseVideoData to see there are two values that may be of interest: isLive and isUpcoming. These values come from the prop data. Logging these on a video currently premiering, I found that isLive was true.

So, next I went to the Channel view to check how the data was parsed. The mounted hook calls getChannelVideosLocal and in this function there is not much parsing. The request is done via the yt-channel-info library, so the parsing must have been done there.

With that, I went to the library. A bit of digging, and I found the parseChannelVideoResponse function is the one called to parse data. Within this function, parseVideo is called. This is where the magic happens. I believe this is the offending line. Upcoming videos also track the view count, therefore the video.shortViewCountText is a number and not undefined.

I didn't spend longer checking out the function, because I'm way passed the scope of this repo and by now it has been hours of digging through code I'd never seen in my life :laughing:

So this does not seem to be fixable in this app, and has to be fixed in the library itself. Since this was my first time seeing much of this code, it is possible that I've missed something. Although, I don't believe I have.

TL;DR: This is an issue with the yt-channel-info and not with the app itself. The library doesn't take into account that Upcoming/Premier videos may also have a live count, and therefore the parameter isLive is true, which causes this mislabelling.