edit4ever / script.module.zap2epg

zap2epg - EPG grabber for USA/Canada
GNU General Public License v3.0
41 stars 25 forks source link

Feature Request: Add option to remove prohibited characters from series title #13

Closed RKCRLR closed 5 years ago

RKCRLR commented 5 years ago

Tvheadend has an option to use Windows safe filenames when recordings are made however this does not extend to the sub-directories that are created for scheduled recordings. This causes Autorecs to fail when a prohibited character is in the series title (i.e., the ":" in "NCIS: New Orleans" or "NCIS: Los Angeles").
The workaround is to manually remove the prohibited character in folder to be created when creating the timer or in the Autorecs tab.
Tvheadend is planning to address this in version 4.4 but it would be nice to have an option to remove prohibited characters from the series title in the mean time.

edit4ever commented 5 years ago

I took a look at the tvheadend code and it should be an easy fix for them to add it. Since I don't expect it will take that long - I don't want to permanently add this feature to zap2epg. However, to get you through for now - I created a version of zap2epg with the feature added that you can use.

Here you go: https://www.dropbox.com/s/z689mmjn28lg0uo/script.module.zap2epg-1.2.0-safetitle.zip?dl=0

RKCRLR commented 5 years ago

Thanks! I’ll give this a try and see how it works.

Sent from Mail for Windows 10

From: edit4ever Sent: Monday, March 4, 2019 9:26 AM To: edit4ever/script.module.zap2epg Cc: RKCRLR; Author Subject: Re: [edit4ever/script.module.zap2epg] Feature Request: Add option toremove prohibited characters from series title (#13)

I took a look at the tvheadend code and it should be an easy fix for them to add it. Since I don't expect it will take that long - I don't want to permanently add this feature to zap2epg. However, to get you through for now - I created a version of zap2epg with the feature added that you can use. Here you go: https://www.dropbox.com/s/z689mmjn28lg0uo/script.module.zap2epg-1.2.0-safetitle.zip?dl=0 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

RKCRLR commented 3 years ago

Any possibility you can add this option in version 2.X? I'm contemplating going to Kodi 19 and this option would be nice to retain. I submitted a bug report to Tvheadend and they said they were going to fix it in version 4.4. However I got a notice the other day that it has now been postponed until version 4.6.

On Mon, Mar 4, 2019 at 11:25 AM Sterling Rorden sterling.rorden@gmail.com wrote:

Thanks! I’ll give this a try and see how it works.

Sent from Mail https://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: edit4ever notifications@github.com Sent: Monday, March 4, 2019 9:26 AM To: edit4ever/script.module.zap2epg script.module.zap2epg@noreply.github.com Cc: RKCRLR sterling.rorden@gmail.com; Author author@noreply.github.com Subject: Re: [edit4ever/script.module.zap2epg] Feature Request: Add option toremove prohibited characters from series title (#13)

I took a look at the tvheadend code and it should be an easy fix for them to add it. Since I don't expect it will take that long - I don't want to permanently add this feature to zap2epg. However, to get you through for now - I created a version of zap2epg with the feature added that you can use.

Here you go:

https://www.dropbox.com/s/z689mmjn28lg0uo/script.module.zap2epg-1.2.0-safetitle.zip?dl=0

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/edit4ever/script.module.zap2epg/issues/13#issuecomment-469340041, or mute the thread https://github.com/notifications/unsubscribe-auth/AiN-ZGsi2x7HeodWXP_EK7tTZNkEvlh1ks5vTVcxgaJpZM4batcU .[image: https://github.com/notifications/beacon/AiN-ZA-vOWRX5C9xN5AvIR1hLCVleEpaks5vTVcxgaJpZM4batcU.gif]

edit4ever commented 3 years ago

Give this version a try and see if it works for you - only good on Kodi 19. https://www.dropbox.com/s/ra21ufezsevuwnu/script.module.zap2epg-2.0.0-safetitle.zip?dl=0

RKCRLR commented 3 years ago

Thanks, I’ve installed it. I’ll wait for the next guide update to see if it works.

Sent from Mail for Windows 10

From: edit4ever Sent: Wednesday, February 24, 2021 10:37 AM To: edit4ever/script.module.zap2epg Cc: RKCRLR; Author Subject: Re: [edit4ever/script.module.zap2epg] Feature Request: Add option toremove prohibited characters from series title (#13)

Give this version a try and see if it works for you - only good on Kodi 19. https://www.dropbox.com/s/ra21ufezsevuwnu/script.module.zap2epg-2.0.0-safetitle.zip?dl=0 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

RKCRLR commented 3 years ago

I tried manually updating the guide data and am getting an error. I reinstalled version 2.0.0 and still get the error. I also get an error when I open Configure Channel List when I exit even if I just back out.
Strange thing is that safetitle appeared to run OK last night when scheduled.
I’ve attached my Kodi logfile and the zap2epg logfile.

Sent from Mail for Windows 10

From: Sterling Sent: Wednesday, February 24, 2021 12:40 PM To: edit4ever/script.module.zap2epg; edit4ever/script.module.zap2epg Cc: Author Subject: RE: [edit4ever/script.module.zap2epg] Feature Request: Add optiontoremove prohibited characters from series title (#13)

Thanks, I’ve installed it.  I’ll wait for the next guide update to see if it works. 

Sent from Mail for Windows 10

From: edit4ever Sent: Wednesday, February 24, 2021 10:37 AM To: edit4ever/script.module.zap2epg Cc: RKCRLR; Author Subject: Re: [edit4ever/script.module.zap2epg] Feature Request: Add option toremove prohibited characters from series title (#13)

Give this version a try and see if it works for you - only good on Kodi 19. https://www.dropbox.com/s/ra21ufezsevuwnu/script.module.zap2epg-2.0.0-safetitle.zip?dl=0 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

edit4ever commented 3 years ago

no attachments. :-(

RKCRLR commented 3 years ago

I don’t know what it is about Outlook Mail that makes it sometimes strip off attachments.

I’ll try again from Outlook 2007.

Let me know if this doesn’t work.

From: edit4ever [mailto:notifications@github.com] Sent: Thursday, February 25, 2021 9:01 AM To: edit4ever/script.module.zap2epg Cc: RKCRLR; Author Subject: Re: [edit4ever/script.module.zap2epg] Feature Request: Add option to remove prohibited characters from series title (#13)

no attachments. :-(

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/edit4ever/script.module.zap2epg/issues/13#issuecomment-786052705 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIRX4ZHFJ2DEAI4UHRGGZT3TAZ653ANCNFSM4G3K24KA . https://github.com/notifications/beacon/AIRX4ZAHQDAZIRSDLLSUBRLTAZ653A5CNFSM4G3K24KKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOF3NDMYI.gif

edit4ever commented 3 years ago

Nope – try sending from your normal email directly to my gmail account.

Jay Maloney

From: RKCRLR Sent: Thursday, February 25, 2021 9:20 AM To: edit4ever/script.module.zap2epg Cc: edit4ever; State change Subject: Re: [edit4ever/script.module.zap2epg] Feature Request: Add option toremove prohibited characters from series title (#13)

I don’t know what it is about Outlook Mail that makes it sometimes strip off attachments.

I’ll try again from Outlook 2007.

Let me know if this doesn’t work.

From: edit4ever [mailto:notifications@github.com] Sent: Thursday, February 25, 2021 9:01 AM To: edit4ever/script.module.zap2epg Cc: RKCRLR; Author Subject: Re: [edit4ever/script.module.zap2epg] Feature Request: Add option to remove prohibited characters from series title (#13)

no attachments. :-(

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/edit4ever/script.module.zap2epg/issues/13#issuecomment-786052705 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIRX4ZHFJ2DEAI4UHRGGZT3TAZ653ANCNFSM4G3K24KA . https://github.com/notifications/beacon/AIRX4ZAHQDAZIRSDLLSUBRLTAZ653A5CNFSM4G3K24KKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOF3NDMYI.gif

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.

RKCRLR commented 3 years ago

Is your gmail jay.maloney@gmail.com?

From: edit4ever [mailto:notifications@github.com] Sent: Thursday, February 25, 2021 9:47 AM To: edit4ever/script.module.zap2epg Cc: RKCRLR; Author Subject: Re: [edit4ever/script.module.zap2epg] Feature Request: Add option to remove prohibited characters from series title (#13)

Nope – try sending from your normal email directly to my gmail account.

Jay Maloney

From: RKCRLR Sent: Thursday, February 25, 2021 9:20 AM To: edit4ever/script.module.zap2epg Cc: edit4ever; State change Subject: Re: [edit4ever/script.module.zap2epg] Feature Request: Add option toremove prohibited characters from series title (#13)

I don’t know what it is about Outlook Mail that makes it sometimes strip off attachments.

I’ll try again from Outlook 2007.

Let me know if this doesn’t work.

From: edit4ever [mailto:notifications@github.com] Sent: Thursday, February 25, 2021 9:01 AM To: edit4ever/script.module.zap2epg Cc: RKCRLR; Author Subject: Re: [edit4ever/script.module.zap2epg] Feature Request: Add option to remove prohibited characters from series title (#13)

no attachments. :-(

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/edit4ever/script.module.zap2epg/issues/13#issuecomment-786052705 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIRX4ZHFJ2DEAI4UHRGGZT3TAZ653ANCNFSM4G3K24KA . https://github.com/notifications/beacon/AIRX4ZAHQDAZIRSDLLSUBRLTAZ653A5CNFSM4G3K24KKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOF3NDMYI.gif

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/edit4ever/script.module.zap2epg/issues/13#issuecomment-786083430 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIRX4ZEAZ6YOQVNHKXSAUALTA2EIBANCNFSM4G3K24KA . https://github.com/notifications/beacon/AIRX4ZBPC2RDSPKDWGPDCHDTA2EIBA5CNFSM4G3K24KKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOF3NK4ZQ.gif

edit4ever commented 3 years ago

edit4ever@gmail.com