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

Clean up EIT fixes for ProSiebenSat.1 and Vodafone Germany #993

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

citronalco
Copy link
Contributor

  • Update EIT fixes for ProSiebenSat.1 (Europe): EIT of ProSiebenSat.1 has changed years ago, old fixes do not work any more. ProSiebenSat.1 still dumps all kind of metadata into the subtitle field from where it can be retrieved.

  • Remove most fixes for Kabel Deutschland: EPG of Vodafone's (formerly Kabel Deutschland) DVB-C is much better today, most fixes are superflous now. Due to frequency/channel changes, the old fixes did not work any more anyway. Only use updated fixes for ProSiebenSat.1

  • Remove fixups for German DVB-T: Service got disabled in 2019.

Checklist

@kmdewaal kmdewaal self-assigned this Dec 9, 2024
@kmdewaal
Copy link
Contributor

kmdewaal commented Dec 9, 2024

Looks like our regression tests are not completely happy:

FAIL! : TestEITFixups::testDEPro7Sat1() Compared values are not the same
Actual (event->m_subtitle) : "Folgentitel,"
Expected (QString("Folgentitel")): "Folgentitel"
Loc: [test_eitfixups.cpp(551)]

This test can be executed with "make test".
Please investigate!

* Update EIT fixes for ProSiebenSat.1 (Europe):
EIT of ProSiebenSat.1 has changed years ago, old fixes do not work any more.
ProSiebenSat.1 still dumps all kind of metadata into the subtitle field
from where it can be retrieved.

* Remove most fixes for Kabel Deutschland:
EPG of Vodafone's (formerly Kabel Deutschland) DVB-C is much better today,
most fixes are superflous now. Due to frequency/channel changes, the old
fixes did not work any more anyway. Only use updated fixes for ProSiebenSat.1

* Remove fixups for German DVB-T:
Service got disabled in 2019.
@citronalco
Copy link
Contributor Author

I did not even recognize that there are tests....

Fixed.

@kmdewaal
Copy link
Contributor

Thanks!

@kmdewaal kmdewaal merged commit 3bb9c29 into MythTV:master Dec 11, 2024
8 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants