[live] [merged] Kanalnummernlimit wird nicht immer beachtet

Hallo zusammen,

ich nutze selbst ein paar Patches, die vielleicht für den einen oder anderen von euch von Interesse sein könnten. Markus hat vorgeschlagen, solche Patches hier im Forum in jeweils einem Thread kurz vorzustellen und euer Votum einzuholen, ob sie ins Git sollen:

• 0004b Limited channel selection to configured channel limit.zip

Wenn man in den Einstellungen bspw. die Anzeige bis Kanal Nr. 5 einstellt, werden in der Kanalauswahlliste der Programmübersicht immer alle benannten Kanäle ausgegeben:

Der Patch limitiert dies auf das eingestellte Kanalnummernlimit, in diesem Beispiel also Kanal 5:

Prinzipiell beachtet die Zeitleiste zwar ebenfalls das Kanalnummernlimit, aber dummerweise hat die Zeitleiste ein anderes Problem: Wenn man für die Zeitleiste noch keine Kanalgruppen einstellt hat, dann trägt die Zeitleiste initial die Kanäle bis zum Kanalnummernlimit als neue Einstellung für die Kanalgruppen ein. Wenn man danach das Kanalnummernlimit ändert, hat dies keine Auswirkung mehr, da die vom alten Limit abgeleiteten Kanalgruppen ja noch immer gelten.

Der Patch löst dies so, dass die Kanalgruppen von der Zeitleiste nur dann gesetzt werden, wenn das Feld für die Kanalgruppen leer ist. Sobald dort eine Eingabe ohne Kanalnummer (sprich: ohne eine Ziffer) erfolgt, also bspw. nur ";" für eine "leere" Kanalgruppe, bleibt diese Einstellung erhalten und die Zeitleiste orientiert sich weiterhin am zuletzt eingestellten Kanalnummernlimit.

Viele Grüße

Stefan

Dass sich die Einstellung auf das "Holen des EPG" auswirkt (was immer auch du damit sagen willst), gibt der Code nicht her. Das widerspräche auch all meiner Erfahrung, da ich schon immer mit einem Kanalnummernlimit arbeite. Der VDR hat trotzdem im OSD alle Kanäle drin.

Das Kanalnummernlimit wird nur dazu herangezogen, die Programme auf den Webseiten – und mit dem Patch auch in der Auswahlliste – abzubilden bzw. zu begrenzen.

Wenn kein Kanallimit eingestellt ist, werden alle Kanäle dargestellt (bei mir geht die Liste dann derzeit bis 1106). Der Code hierzu:

int GetLastChannel() const { return m_lastChannel == 0 ? std::numeric_limits<int>::max() : m_lastChannel; }

Da fällt mir gerade noch auf, dass die Einstellung, nur Kanäle mit EPG anzuzeigen, in der Zeitleiste nicht wirkt. Das kann man bspw. leicht mit Euronews German SD oder arte (SD) überprüfen.

Viele Grüße

Stefan

Das wären meines Wissens alle. Auf die Schnelle finde ich keinen einfachen Patch für die Zeitleiste. Die bearbeitet immer (notfalls intern generierte) Kanalgruppen, bei deren Erstellung aber noch keine Informationen zum EPG vorliegen. Dort müsste aber unter Berücksichtigung der ausgewählten Startzeit bereits die Entscheidung fallen, wenn man der Systematik des Codes folgen wollte.

Puh, das wird wohl etwas komplizierter…

So, hier wäre dann der Patch on Top of Nr. 4b, der je nach Einstellung im Setup die Kanäle ohne EPG in der Zeitleiste ausgibt oder sie herausfiltert:

• 0004c Fixed display of channels without EPG in multischedule.zip

Aus Sicht der Usability funktioniert das Ganze aber erst so richtig gut, wenn Patch Nr. 4b mit im Code ist. Denn dieser Patch erlaubt, die Speicherung der Kanalnummern zu unterbinden, wenn man in der Kanalliste ein Zeichen einträgt, das keine Ziffer ist. In diesem Fall ermittelt die Zeitleiste die relevanten Kanäle jedes Mal entsprechend der Vorgaben und zeigt sie entsprechend an, selbst wenn man zwischenzeitlich das Kanalnummernlimit geändert hat. Ohne diese Erweiterung muss man die Kanalliste immer erst von Hand löschen, bevor sich die in der Zeitleiste angezeigten Kanäle auch tatsächlich ändern.

Viele Grüße

Stefan

Wenn du mehr möchtest als das, was Markus ins Git aufnimmt, musst du lokal wohl selber patchen. Auch wenn uns Markus weit entgegenkommt – vielen Dank dafür! – , muss ich doch den für mich letzten Schliff lokal ergänzen. Hier zum Beispiel finde ich es schade, dass Markus die Option, sich bei leerem Eingabefeld eine initiale Kanalliste erzeugen zu lassen, nicht übernommen hat. Ohne eine solche Vorlage kann das in viel Tipparbeit ausarten. Auch vermisse ich natürlich die Begrenzung der Kanalauswahl. Aber bei kollisionsfreiem Feinschliff ist ein git pull mit automatischem Merge eine einfache Sache – andernfalls muss man halt leider das mergetool für eine manuelle Korrektur bemühen.

Doch zurück zu deinem Fall: Derzeit wird nach fünf Kanälen ein Semikolon als Gruppentrennzeichen eingefügt. In multischedule.ecpp findet sich das in den Zeilen 91 bis 94:

if ( (++i % 5) == 0 )
    groups << ";";
else
    groups << ",";

In der ersten Zeile kannst du die automatisch erzeugte Gruppengröße ändern, und durch Löschen der ersten drei Zeilen die gesamte Gruppeneinteilung eliminieren.

Allerdings definiert in Zeile 61 die Konstante MAX_CHANNELS, wie viele Kanäle unabhängig von den definierten Kanalgruppen gleichzeitig in der Zeitleiste dargestellt werden. Den Wert kann man zwar erhöhen, aber die Zeitleiste wird mit größeren Werten meines Erachtens bezüglich der Usability zunehmend unbrauchbar.

Viele Grüße

Stefan

Quote from nobanzai

führt zu einer 10er Einteilung der Sender.

Siehe oben: Du musst MAX_CHANNELS auf einen höheren Wert setzen.

Und zumindest das Scrollen über das Mausrad (nach links/rechts drücken) funktioniert. Eine Scrollleiste habe ich auch nicht gesehen – deshalb mein Hinweis zur Usability.