[TVB-515] Program.getID() eindeutig machen

[hampelratte]

Hallo zusammen,

wie aufwändig wäre es, wenn man devplugin.Program.getID() wirklich eindeutig machen würde und nicht nur eindeutig für einen Tag? Für mein Plugin wäre das wirklich eine Vereinfachung, weil ich ständig über die Falle des Tageswechsels stolper.

Gruß
Henrik

[ds10]

Ziemlich aufwändig. Das Laden von alten Einstellungen würde sich damit deutlich verkomplizieren. Es wäre halt am Besten gewesen, wenn diese ID von Anfang an eindeutig gewesen wäre.

[hampelratte]

Es wäre halt am Besten gewesen, wenn diese ID von Anfang an eindeutig gewesen wäre.

Was ja eigentlich auch eine ID ausmacht. Schade, dann muss ich mir irgendwas schlaues überlegen, wie ich trotzdem klarkomme.

[ds10]

Naja, ich könnte ja jetzt behaupten, dass es dann getUniqueID() geheißen hätte, aber das würde mir sowieso niemand abkaufen.

[ds10]

Allerdings könnte man das jetzt zusätzlich einbauen, also getUniqueID() und dann das Laden der Sendung mit dieser im PluginManager. Wäre aber etwas was frühstens mit 3.0 käme, also nicht so bald.

[hampelratte]

Dann mach ich jetzt mal einen offiziellen Feature Request dafür

[hampelratte]

Wieso erst mit 3.0? Ist das eine zeitliche Frage, weil Ihr sonst noch genügend zu tun habt, oder hat das eher damit zu tun, die API nicht verändern zu wollen? Sonst würde ich sagen, ich kümmere mich selbst darum und schicke Euch einen Patch.

[ds10]

Ja, es hat was mit unseren Festlegungen für die 2-7-x-Linie zu tun, dort sollen nach Möglichkeit ausschließlich Bugs gefixt werden. Neue Features sind der 3.0 vorbehalten, obwohl ich das an sich nicht so eng sehe, wäre die Frage was die Anderen dazu sagen. Aber abgesehen davon je nachdem was die Anderen sagen, kannst du das gerne übernehmen, da wir kaum noch Zeit haben irgendwas zu machen.

[hampelratte]

Dann schau ich mir das mal an.

[ds10]

Nur so als Tipp am Rande getUniqueID() sollte das ISO-Datumsformat verwenden, also 2009-02-20.

[hampelratte]

Achso ja, darüber sollte man vll noch sprechen. Wie soll die ID denn aussehen? So wie jetzt plus Datum, oder was hast Du Dir vorgestellt? Ansonsten könnte man ja auch einfach eine UUID oder Ähnliches generieren. Aber wahrscheinlich würde die jetzige Form + Datum die Navigation in den Datenstrukturen vereinfachen, was? Da könnte man wahrscheinlich die meisten Methoden direkt übernehmen.

[ds10]

Die aktuelle ID lautet:

Code: Alles auswählen

DATENSERVICE-ID_SENDERGRUPPEN-ID_LANDESKENNUNG_SENDERNAME_STUNDE:MINUTE:ZEITABWEICHUNG

die Zusätzliche sollte:

Code: Alles auswählen

DATENSERVICE-ID_SENDERGRUPPEN-ID_LANDESKENNUNG_SENDERNAME_JAHR-MONAT-TAG_STUNDE:MINUTE:ZEITABWEICHUNG

lauten, jeweils mit nötigen führenden Nullen beim Datum würde ich sagen. Das sollte sich auch recht leicht umsetzen lassen, da eine Sendung ja weiß, an welchem Datum sie läuft (tvdataservice.MutableProgram).

[hampelratte]

Ist geritzt.

[hampelratte]

Und schon fertig. Patch ist angehängt.

Sorry, hatte eine Datei für den Patch vergessen. Hab den Patch nochmal neu hochgeladen.

[ds10]

Ok, danke, mal sehen wie die Meinung der Anderen ist.