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

Changed ' to ", to fix m3u/m3u8 list specific to Jellyfin loading m3u… #141

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

slippyC
Copy link

@slippyC slippyC commented Feb 8, 2024

… lists.

For some odd reason Jellyfin doesn't like ' around data it prefers ". Also changed channelID to channel-id. I see this more commonly and it also uses that id in the epg/xmltv files.

This should help deal with guide data and loading icons for channels without going in an actually mapping channels in Jellyfin.

… lists.

For some odd reason Jellyfin doesn't like ' around data it prefers ".  Also changed channelID to channel-id.  I see this more commonly and it also uses that id in the epg/xmltv files.

This should help deal with guide data and loading icons for channels without going in an actually mapping channels in Jellyfin.
@rocky4546
Copy link
Contributor

channelID is found in other files and will need to be addressed before applying this PR. The rest looks good.

@slippyC
Copy link
Author

slippyC commented Feb 10, 2024

Alrighty, I'll change it over. It won't effect any of the stuff I use. I have never seen it as ChannelID in another .m3u file. Anyway, will change it over.

@rocky4546
Copy link
Contributor

You missed what I said. I did not say to change what you did, but rather, you missed other spots. If we are going to assume that channelID is now channel-id, it needs to change in other files as stated above. Only changing half the spots makes no sense to me.

@slippyC
Copy link
Author

slippyC commented Feb 10, 2024

I see, I misunderstood what you were asking. I'll have to go through the files and figure it out then. If you want to commit that as a fix for current stable Jellyfin, then go for it. If you want to wait for me to go through the files, I don't know how long that will take; because I don't know all the references of where it is used(yes, I can do a project wide search; but still have to make sure how it's referenced).

All I do know is the current stable version of Jellyfin doesn't work right with the way it is currently(you will have to manually map). I was about to pull my hair out trying to figure out why the guide wasn't populating and why channel icons weren't showing up(even though the logs said there was guide data found). I was first looking at the EPG files to see if id's weren't correlating. Also tried to remove channel name duplicates, multiple names for same channel. I didn't think that that should have an impact on the two being tied together, since it really depends on the ids between m3u and xmltv files.

@slippyC
Copy link
Author

slippyC commented Feb 10, 2024

You missed what I said. I did not say to change what you did, but rather, you missed other spots. If we are going to assume that channelID is now channel-id, it needs to change in other files as stated above. Only changing half the spots makes no sense to me.

After last response I did have time to do a project wide search. This is only place that is referenced. I also did a search for some of the other m3u elements just to see if they would pop up, but it looks as if they are only referenced when writing to the m3u file/stream(in that part of code).

After checking for possible reference issues, which none were found.  Changing to more common channel-id tag.
@stale stale bot added the inactive associated with a stale issue label Mar 5, 2024
@rocky4546 rocky4546 added bug Something isn't working and removed inactive associated with a stale issue labels Mar 5, 2024
@cabernetwork cabernetwork deleted a comment from stale bot Mar 5, 2024
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

Successfully merging this pull request may close these issues.

2 participants