Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Subtitle icons not parsing #1109

Open
5 tasks done
Luna712 opened this issue May 29, 2024 · 8 comments
Open
5 tasks done

Subtitle icons not parsing #1109

Luna712 opened this issue May 29, 2024 · 8 comments
Labels
bug Something isn't working

Comments

@Luna712
Copy link
Contributor

Luna712 commented May 29, 2024

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

  • I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
  • I have written a short but informative title.
  • I have updated the app to pre-release version Latest.
  • If related to a provider, I have checked the site and it works, but not the app.
  • I will fill out all of the requested information in this form.
@Luna712 Luna712 added the bug Something isn't working label May 29, 2024
@KingLucius
Copy link
Contributor

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

@Luna712
Copy link
Contributor Author

Luna712 commented May 29, 2024

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
Copy link
Contributor Author

Luna712 commented May 29, 2024

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

@KingLucius
Copy link
Contributor

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

@Luna712
Copy link
Contributor Author

Luna712 commented May 30, 2024

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
Copy link
Contributor Author

Luna712 commented May 30, 2024

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.

@KingLucius
Copy link
Contributor

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
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants