[live] [partially merged] Anzeige zusätzlicher Programminformationen

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:

• 0012a Added supplemental info to short text and details.zip
  (jetzt ohne nicht-essentiellen die PO-Files)

Der Patch erlaubt per Einstellung im Setup:

… für die Übersicht der Sendungen:

… sowie der Aufzeichnungen:

… in der zweiten Zeile jeweils die Altersfreigabe – sofern in den EPG-Daten vorhanden – und in den Details zusätzlich die Programmkategorien auszugeben. Außerdem habe ich noch die Anzeige noch dahingehend optimiert, dass sich der Link für die Detailinfos bei einer leeren zweiten Zeile nicht über beide Zeilen erstreckt.

Zum Vergleich mit vorher:

Viele Grüße

Stefan

Quote from MarkusE

Das parental rating scheint auch im Kurztext zu landen. Macht das wirklich Sinn?

Leider werden die Daten von EPG und Aufzeichnungen in Live grundsätzlich unterschiedlich verarbeitet, um in der Website zu landen. Im einen Fall werden sie direkt in schedule.ecpp eingespeist, im anderen Fall steht die Information nur in recman.cpp zur Verfügung und muss deshalb "durchgetunnelt" werden; deshalb das zusätzliche Tagging im Kurztext.

Quote from MarkusE

Wozu die Änderung in function addEventRec?

Gerendert wird das Ganze für beides in ebendieser Funktion, deshalb die Erweiterungen.

Ich lege meine Patches gerne so aus, dass sie sich – falls sie nicht im Git landen – beim (dann leider häufig notwendigen) lokalen Mergen möglichst einfach einfügen; darum auch dieser Ansatz in Form eines minimalen Huckepack-Eingriffs. Ein einheitliches Pattern für beide Quellen wäre mir auch lieber. Wenn die Erweiterung ins Git soll, wäre vielleicht auch eine umfassendere Refaktiorisierung denkbar.

Quote from MarkusE

So wichtig ist es dann doch wieder nicht. (?).

Nutzer mit Kindern möchten die Altersfreigabe beim suchen nach kindgerechten Filmen vielleicht doch gleich in der Übersicht vorfinden.

Quote from MarkusE

Wenn das EPG von extern geholt wird, möchte man vielleicht nichts aus dem Sender EPG sehen.

Was die Herkunft der Daten betrifft: Es sind halt die Daten, die der VDR im Event liefert. Ob die aus dem Sender-EPG kommen oder von einem der Daemons (EPGD, Tvscraper usw.) eingespeist werden, erschließt sich mir nicht.

Quote from MarkusE

Falls möglich, möchte ich zusätzliche Einstellungen vermeiden.

Ich habe das Feature einstellbar gemacht, damit es jeder nach Gusto einsetzen kann und damit die Chance steigt, es ins Git zu schaffen. Jeder Patch, den du ins Git übernimmst, erspart mir später Stunden an Mergetool-Orgien.

Viele Grüße

Stefan

Schaut gut aus – Danke!

Hier dann Teil 2 des Patches auf Basis des aktuellen Master-Branches im Git:

• 00012b Added supplemental info to short text.zip

Vielleicht noch ein paar Gedanken zu diesem Patch:

• Manche Sender, wie etwa die ARD-Gruppe, liefern die Altersfreigabe über das EPG. Mit dem Patch erscheint die Altersfreigabe sowohl im EPG als auch in daraus resultierenden Aufzeichnungen in Zeile 2 der Listenansicht – selbst dann, wenn die zweite Zeile nur gekürzt wiedergegeben wird.
• Andere Sender, wie etwa die Pro7-Gruppe, schreiben die Altersfreigabe literal in den Kurztext. Hier kann die Altersfreigabe durch die Kürzung in der Listenansicht fehlen. Allerdings kann man das zumindest bei Aufnahmen dadurch kompensieren, dass man per Recording hook (z.B. R10.parental-rating.txt) die Altersfreigabe von dort extrahiert und in die Info-Datei der Aufzeichnung einträgt.
• Und dann gibt es natürlich noch die Sender, die diese Daten überhaupt nicht liefern.

Der Patch sorgt dafür, dass zumindest bei Sendern, die – siehe Punkt 1 – das EPG vernünftig ausstatten, die Informationen schon in der Listenansicht sichtbar sind. Dort, wo EPG-Daemons oder Recording hooks die Altersfreigabe ergänzen, wird sie ebenfalls in den Listen angezeigt.

Das spart also schon in paar Klicks, wenn man wissen will, für welche Altersgruppe eine Sendung bzw. Aufzeichnung geeignet ist.

Viele Grüße

Stefan

Kleiner, aber unangenehmer Typo in recordings.ecpp:

• recoring_item -> recording_item
  
• irecoring_item -> irecording_item

Viele Grüße

Stefan