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

Update XmlTvListingsProvider.cs to fix the record series button missing on many programs #12398

Merged
merged 2 commits into from
Aug 20, 2024

Conversation

bpauquette
Copy link
Contributor

@bpauquette bpauquette commented Aug 5, 2024

Fixes a bug where certain programs like the "Good Morning America" no longer have the record series button.

Changes
Assume that if the program has an EpisodeNumber then it is part of a series.

Issues
The record series button is not displaying on many programs that are generated by zaptoxml.pl perl script.

https://github.com/jef/zap2xml

Fixes a bug where certain programs like the "Good Morning America" no longer have the record series button.
@bpauquette bpauquette marked this pull request as draft August 6, 2024 00:56
@bpauquette bpauquette marked this pull request as ready for review August 6, 2024 01:05
@bpauquette
Copy link
Contributor Author

Fixes issue #11976

@bpauquette
Copy link
Contributor Author

For reference the xmltv.dtd is here https://github.com/XMLTV/xmltv/blob/master/xmltv.dtd . I am not sure if it really speaks about an isSeries boolean. Seems like it must be derived.

@theguymadmax
Copy link
Contributor

This probably should have targeted the 10.9.z branch. Otherwise, you'll have some grumpy guy commenting every week what's taking so long ☺️.

Copy link
Member

@nielsvanvelzen nielsvanvelzen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a descriptive title to the pull request

@bpauquette bpauquette changed the title Update XmlTvListingsProvider.cs Update XmlTvListingsProvider.cs to fix the record series button missing on many programs Aug 10, 2024
@bpauquette
Copy link
Contributor Author

This probably should have targeted the 10.9.z branch. Otherwise, you'll have some grumpy guy commenting every week what's taking so long ☺️.

theguymadmax can see into the future. How did he know?

@Bond-009 Bond-009 merged commit 567adb8 into jellyfin:master Aug 20, 2024
15 checks passed
Bond-009 added a commit to Bond-009/jellyfin that referenced this pull request Aug 20, 2024
Update XmlTvListingsProvider.cs to fix the record series button missing on many programs
crobibero pushed a commit that referenced this pull request Aug 20, 2024
#12481)

Co-authored-by: grumpycat <bryan.pauquette@gmail.com>
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.

5 participants