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

DeleteAllInputs must not delete iptv_channel #987

Merged
merged 1 commit into from
Dec 7, 2024

Conversation

kmdewaal
Copy link
Contributor

@kmdewaal kmdewaal commented Dec 4, 2024

The CardUtil function DeleteAllInputs does delete the content of all four tables related to inputs but it does also delete the content of table iptv_channel.
This is not correct; table iptv_channel contains the URLs of the channels that are IPTV channels.
This bug has been introduced as part of commit d4c0f13. The entries in table iptv_channel can only be removed when the corresponding channel has been deleted.
This is done as part of a periodic housekeeping task.

Refs #936

The CardUtil function DeleteAllInputs does delete the
content of all four tables related to inputs but it does
also delete the content of table iptv_channel.
This is not correct; table iptv_channel contains the URLs
of the channels that are IPTV channels.
This bug has been introduced as part of commit d4c0f13.
The entries in table iptv_channel can only be removed
when the corresponding channel has been deleted.
This is done as part of a periodic housekeeping task.

Refs MythTV#936
@kmdewaal kmdewaal requested a review from paul-h December 5, 2024 22:10
@kmdewaal kmdewaal merged commit c91be53 into MythTV:master Dec 7, 2024
8 of 10 checks passed
@kmdewaal kmdewaal deleted the cardutil_iptv branch December 7, 2024 15:35
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.

1 participant