cloudstream icon indicating copy to clipboard operation
cloudstream copied to clipboard

Subtitle icons not parsing

Open Luna712 opened this issue 1 year ago • 10 comments

Steps to reproduce

Find a video with Hearing Impaired default subtitles

Expected behavior

Icon to display properly

Actual behavior

Screenshot_20240530_054310_CloudStream Beta

Cloudstream version and commit hash

4.3.2 dff5602

Android version

Android 14

Logcat

No response

Other details

I noticed this after updating to dff5602 and logging into SubDL account. I am not sure if it happened before but I assumed it had something to do with that.

Acknowledgements

  • [X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
  • [X] I have written a short but informative title.
  • [X] I have updated the app to pre-release version Latest.
  • [X] If related to a provider, I have checked the site and it works, but not the app.
  • [X] I will fill out all of the requested information in this form.

Luna712 avatar May 29 '24 21:05 Luna712

Can you elaborate, where this screenshot appears? As for SubDL it is totally via api and no scraping at all and the screenshots shows html, so I doubt it's something related but no harm to double-check

KingLucius avatar May 29 '24 22:05 KingLucius

Sorry I should have checked first I suppose this could potentially also be a provider issue. It was not related to update. It happens on stable release as well. I just opened a random series from the sorastream home page and it happened.

Luna712 avatar May 29 '24 22:05 Luna712

It doesn't happen when searching online subtitles only the auto subtitles though.

Luna712 avatar May 29 '24 22:05 Luna712

So it's for sure one of sub providers of SS

KingLucius avatar May 30 '24 00:05 KingLucius

So it's for sure one of sub providers of SS

@KingLucius

I think this has to do with opensubtitles, if you search subtitles on opensubtitles.com you get icons in there also like the fa-check which means "Subtitles from trusted source"

I think CloudStream needs to handle parsing this HTML properly. Probably stripping the check marks but could probably replacing the hearing impaired HTML icon with the icon used in CloudStream within subtitles search.

Looks like the same type of icon would also happen with SubDL and maybe addic7ed as well but I'm unsure if those even supports auto subtitles.

Also on another note, SubDL seems unreliable a bit, if you open subtitles on SubDL it shows a lot more then when you Search Online Subtitles from within CloudStream, it seems to omit any with Hearing Impaired at all and others (maybe omitting HI is just a coincidence but it seems only about 5% that actually exist in SubDL properly show up when searching in CloudStream)

Luna712 avatar May 30 '24 16:05 Luna712

Wait I'm just now learning how subtitles even work. This potentially isn't even a CS issue but rather extension issue. I'm unsure. If it is I apologize.

Luna712 avatar May 30 '24 16:05 Luna712

Opensubtitles api provider uses api only, no parsing at all Also SoraStream collect subs from his own sources

SubDL & OpenSubtitles load only on "load online subtitles"

For SubDL issue, i will review. They changes the api a lot as they are always working on it currently

KingLucius avatar May 30 '24 16:05 KingLucius

You are right, SubDL api has an filtering issue It feeds only the non hearing impaired subtitles Reported to main dev there

KingLucius avatar May 31 '24 16:05 KingLucius

this issue still exists in the latest version Screenshot_20240727-233213

janowakps avatar Jul 27 '24 21:07 janowakps

this issue still exists in the latest version Screenshot_20240727-233213

Provider issue not app issue

KingLucius avatar Jul 28 '24 06:07 KingLucius

extension issue

C10udburst avatar Sep 17 '24 09:09 C10udburst

Can we have a Regular expression to remove out the HTML part of the list items?

elvistony avatar Dec 01 '24 18:12 elvistony