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

Case Sensitive tvg-id #32

Closed
ap-rose opened this issue Oct 7, 2019 · 6 comments
Closed

Case Sensitive tvg-id #32

ap-rose opened this issue Oct 7, 2019 · 6 comments
Labels
feature request A new feature request

Comments

@ap-rose
Copy link

ap-rose commented Oct 7, 2019

Could you please add an argument to be able to match tvg-id regardless of case.

I am having issues where my EPG contains data that is not being kept due to id being case sensitive.

All EPG providers change the format of their ID making it hard to parse properly.

@bebo-dot-dev
Copy link
Owner

I haven't experienced this issue myself but I'll take a look to see if I can reproduce the problem and come up with something to cope with it.

@bebo-dot-dev bebo-dot-dev added the feature request A new feature request label Oct 7, 2019
@ap-rose
Copy link
Author

ap-rose commented Oct 7, 2019

The issue I am having is that if the m3u contain the tvg-id as lowercase (tlc.ca) it will not be matched to my data found in my epg which is defined in uppercase. (TLC.ca) and vice versa.

Is there a way too loosen the match so it is not case sensitive.

@bebo-dot-dev
Copy link
Owner

Yep I get the idea. I've created a branch and applied a candidate change:

https://github.com/jjssoftware/m3u-epg-editor/tree/epg-tvg-id-casing

Give this version a try and let me know how it goes:

https://github.com/jjssoftware/m3u-epg-editor/blob/epg-tvg-id-casing/m3u-epg-editor.py

@ap-rose
Copy link
Author

ap-rose commented Oct 7, 2019

The fix you submitted did the trick, Thank you so much for prompt response.

Just know your software was of great use to me and continuing to use it.

@bebo-dot-dev
Copy link
Owner

You're welcome, thanks for the update, I'll merge the change.

@bebo-dot-dev
Copy link
Owner

Merged to master in #33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request A new feature request
Projects
None yet
Development

No branches or pull requests

2 participants