opendreambox / enigma2-plugins

Python plug-ins for enigma2 (DreamOS)
https://dreambox.de/
Other
37 stars 49 forks source link

2 kleine Fixes #14

Closed ghost closed 6 years ago

ghost commented 6 years ago

1) Wenn im Erweiterungsmenü die Zeit für tägliche Transfers geändert wurde, wurde diese bis zum enigma2 Neustart nicht übernommen. 2) Um den Timer für den täglichen Transfer zu berechnen, wurde "now" benutzt, das allerdings bereits gesetzt wurde, während das Plugin geladen wird. Dadurch wurde der Timer falsch berechnet und der Transfer ab dem 2. Tag viele male täglich gestartet.

drbest2 commented 6 years ago

Hm, ich finde die Lösung für "Reset Timer on config change" nicht sehr gelungen.

Es ist nicht notwendig, dafür eine Instance-Klasse zu bauen und diese dann als global ansprechbares Objekt zu "missbrauchen". Also die Lösung würde zwar zum Ziel führen, ist aber, wie gesagt, unnötig und nicht sehr elegant. :)

In Enigma2 gibt es einen Notifier für configs, so dass man auf Veränderungen automatisch einen Callback erhält. Es reicht auch durchaus aus, diesen Notifier nur für config.plugins.epgCopy.copytime zu setzen, so dass nach einer Bearbeitung der Zeit der Long-Timer neu berechnet wird.

Ich werde das später ändern (oder maluhi will das selber anpassen? :) )