Bekannter Bug -> https://github.com/agent4788/SHC_Framework/issues/20

agent47 wrote:

Bekannter Bug -> https://github.com/agent4788/SHC_Framework/issues/20

Bis zur nächsten Version dauert es ja noch etwas - gibt es für uns User eine Möglichkeit, den Fehler manuell in einer Datei zu fixen?

Gruß, Sven

Das hatte ich agent auch schon mal gefragt. Er antwortete mit Hilfe des github. Ich habe aber keine Ahnung wie und wo man den fix dort herausfischt.

Ich wäre auch sehr daran interessiert, da es für mich eine wichtige Funktion ist und 2 Monate noch lang sind.

Für diesen Fehler habe ich noch keinen Fix, bis jetzt bin ich der Ursache noch nicht auf die Spur gekommen. Es könnte sein das dies ein Bug in der PHP Version auf dem Rasperry ist, auf meinem Entwicklunssystem mit PHP 5.6 funktioniert das Problemlos.

Super, das ist eine klare Aussage. Ich hätte mal vermutet, dass es vielleicht am Stundenformat liegt, da er ja genau alle 12 Stunden, anstelle von 24 Stunden geschaltet wird.

Wo kommt denn die Routine bzw. die Tabelle zur Berechnung der Zeiten her? Die tatsächlichen Zeiten werden jedoch noch richtig unter Administrator Einstellungen Datum und Zeit angezeigt. Angelegte Schaltpunkte funktionieren ja einwandfrei zu den hinterlegten Zeiten.

Jetzt weiß ich natürlich nicht, wie Du diese Zeiten in Schaltpunkte umsetzt. Da scheint mir die Stelle zu sein, wo der Fehler ins Spiel kommt, da es eben genau 12 Stunden sind.

Das war meine erste Vermutung das es einen Fehler mit dem 12/24 Stunden Format gibt, das ist aber i.O. auch eine Zusätzliche Einschränkung auf vor bzw. nach 12 Uhr mittags hat nichts gebracht.Ich kann dann nur noch mit dem Debugger angreifen, dazu brauche ich aber mal viel Zeit die ich aktuell nicht habe.

wie ich festgestellt habe, besteht das Problem auch mit Testversion 2.2

ich halte die Sonnenuntergang-Funktion ist eine der besten und nützlichsten Funktionen, für Steckdosen.
Wie wahrscheinlich ist es, dass die Funktion "behoben" wird?

Woher (File) kommen denn die Zeiten für SoAu/SoUn? Ich würde gern ein Offset einbauen, da mir zB SoUn ne Stunde zu spät schaltet.

Das offset kannst du einfach unter einstellungen einstellen.

agent47 wrote:

Das offset kannst du einfach unter einstellungen einstellen.

Aaaahh, das hatte ich überhaupt nicht gesehen. Super, danke

Ist das Problem mittlerweile behoben(12std Versatz zweites auslösen)?

nein, das wird erst in der Testphase passieren, das zu finden und beheben wird sehr viel Zeit brauchen

Der Fehler ist lokalisiert und so einfach das er kaum zu finden ist. Gefunden hat ihn artek aus dem Raspberry Pi Forum.
Der Fix wird ab der Beta der Version 2.2 in den kommenden Tagen enthalten sein.

Evtl ein Hinweis für ne manuelle Korrektur in der aktuellen Version?

Schreibe ich morgen wenn der Fix in die dev. Version eingeflossen ist.

In der Datei https://github.com/agent4788/SHC_Framework/blob/stable/shc/lib/event/events/sunrise.class.php muss die Zeile 43

[code=php]if($now->format('Ymdhi') == $sunriseDate->format('Ymdhi') && $this->state['lastExecute']->format('Ymdhi') != $now->format('Ymdhi')) {[/php]

durch


[code=php]if($now->format('YmdHi') == $sunriseDate->format('YmdHi') && $this->state['lastExecute']->format('YmdHi') != $now->format('YmdHi')) {[/php]

ersetzt werden.

Gleiches in der Datei https://github.com/agent4788/SHC_Framework/blob/stable/shc/lib/event/events/sunset.class.php Zeile 43:


[code=php]if($now->format('Ymdhi') == $sunsetDate->format('Ymdhi') && $this->state['lastExecute']->format('Ymdhi') != $now->format('Ymdhi')) {[/php]

durch


[code=php]if($now->format('YmdHi') == $sunsetDate->format('YmdHi') && $this->state['lastExecute']->format('YmdHi') != $now->format('YmdHi')) {[/php]

Wem es auf gefallen ist 'Ymdhi' wurde nur durch 'YmdHi' ersetzt (kleines h durch großes H) was den kleinen feinen Unterschied mach zwischen 12h und 24h Formatierung. Durch das kleine h war somit die 12h Formatierung verwendet und somit auch 2x am Tag 6 Uhr statt 6 und 18 Uhr. Der Fix ist in der heutigen Development Version enthalten, wird dann in die Beta mit einfliesen und schlislich in die 2.2 stable.

Hervorragend! Vielen Dank!

Habe die Zeilen gestern entsprechend geändert, dennoch hat der "Sunset" heute morgen einen Schaltvorgang vorgenommen.

Werde es weiter beobachten.