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

Limit channels for searching repetitions #18

Open
FireFlyVDR opened this issue Jul 2, 2024 · 2 comments
Open

Limit channels for searching repetitions #18

FireFlyVDR opened this issue Jul 2, 2024 · 2 comments

Comments

@FireFlyVDR
Copy link

Pressing '1' in the EPG detail for an event shows all repetitions of that event, but for channels with many regional channels this results in a long list with effectivley the same events.
Example: WDR with WDR HD Bonn, WDR HD Dortmund, WDR HD Essen etc. and also their SD variants WDR Bonn, WDR Dortmund, WDR Essen etc.
With the patch below the search is limited to the first n channels like in the Whats-On menu. The same config variable is used - maybe the varable should be renamed to reflect the additional behaviour. Please also check for any side effects.

--- epgsearchext.c.orig 2021-12-10 21:20:29.000000000 +0100
+++ epgsearchext.c      2024-07-02 12:57:13.927086983 +0200
@@ -1021,6 +1021,12 @@
                 continue;
             }

+            if (useChannel == 0) {
+                if (EPGSearchConfig.maxChannelMenuNow > 0 && channel->Number() > EPGSearchConfig.maxChannelMenuNow) {
+                    Schedule = Schedules->Next(Schedule);
+                    continue;
+                }
+            }
             if (useChannel == 1 && channelMin && channelMax) {
                 if (channelMin->Number() > channel->Number() || channelMax->Number() < channel->Number()) {
                     Schedule = Schedules->Next(Schedule);
@tomjfr
Copy link
Collaborator

tomjfr commented Jul 9, 2024

The side effect is that any searchtimer would also be limited to maxChannelMenuNow. I would accept the patch if this could be avoided.

@FireFlyVDR
Copy link
Author

Unfortunately I'm not familiar enough with epgsearch to write such a patch. Therefore please take this as an enhancement request. Thanks

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

No branches or pull requests

2 participants