$we funktioniert nicht mehr

[Beta-User]

Kannst Du da bitte einmal für die entsprechenden Definitionen ein modify machen und schauen ob es dann weg ist?

Vermutlich ist nicht mal das notwendig, die timer müßten seit heute morgen wieder stimmen.Ein weekdaytimer aktualisiert seine Zeiten nur beim Define und dann einmal am Tag, und soweit ich das mitbekommen habe, ist das Modul nur eine Kopie von heating_control.

[CoolTux]

Vermutlich ist nicht mal das notwendig, die timer müßten seit heute morgen wieder stimmen.Ein weekdaytimer aktualisiert seine Zeiten nur beim Define und dann einmal am Tag, und soweit ich das mitbekommen habe, ist das Modul nur eine Kopie von heating_control.

Guten Morgen,

Den Gedanken hatte ich auch schon und gehofft das der User bevor er das macht kurz nach schaut. Na bestimmt erzählt er uns nachher noch ob alles nun stimmt.


Grüße

[ToKa]

Ich muss Euch enttäuschen, weder das modify gestern noch die nächtliche Neuberechnung der Schaltzeiten haben eine Besserung gebracht.

[CoolTux]

Dann gib mal bitte ein List von allen Holiday Devices welche Du im global Device als we2holiday definiert hast. Ausserdem bitte ein list der weekdaytimer Devices.

[Beta-User]

Hmmm,

also noch eine weitere modulinterne Implementierung von holiday2we und $we (Zeilen 310ff in weekdaytimer?)...

Allerdings ist mir nicht klar, wieso das jetzt so eine Auswirkung haben sollte, der get-Befehl von holiday-Devices wurde doch gar nicht angefaßt? Oder doch?

[CoolTux]

Hmmm,

also noch eine weitere modulinterne Implementierung von holiday2we und $we (Zeilen 310ff in weekdaytimer?)...

Allerdings ist mir nicht klar, wieso das jetzt so eine Auswirkung haben sollte, der get-Befehl von holiday-Devices wurde doch gar nicht angefaßt? Oder doch?

Ist mir nicht bekannt. Bin aber auch noch nicht dazu gekommen mir die Änderungen im ganzen an zu schauen.

[ToKa]

Danke schon Mal für Eure Hilfe, da ich jetzt aber unterwegs bin, kann ich Euch die Daten erst nächste Woche hier einstellen.

Beste Grüße und ein schönes Wochenende
Torsten

[CoolTux]

Ich habe mal geschaut. In der letzten Änderung von holiday gab es nur Anpassungen der Commandref, davor wurde das Modul zuletzt am 01.01.2019 angefasst.
Also am Holiday Modul kann es nicht liegen.

[Beta-User]

Kann es sein, dass du (neuerdings) zwei oder mehr holiday2we-Devices nutzt?

Dann hat das m.E. seit dem schon  nicht mehr funktioniert, weil dann das "get" ein mehrzeiliges Array mit jeweils "none" zurückgibt... Das ist aber 'ne "none"'
Eigentlich ist das mit dem get-Aufruf ja schon eine coole Sache, war mir auch neu...

@CoolTux:
Wäre es zielführend, hier wieder auf die Idee einer zentralen IsHoliday()-Funktion zurückzukommen und igami vorzuschlagen, das  entsprechend umzubauen? Aber dann wohl besser unter dem Namen IsHoliday2we() und einem nummerischen Wert als (optionalem) Argument, mit dem man ausgehend vom heutigen Tag vor- und zurück abfragen kann, welche holidays bekannt sind, Rückgabe dann auch als Zeilen-Array?

Also: IsHoliday2we(-3) gäbe 0 zurück, wenn vor drei Tagen kein holiday in den h2we-Devices drin steht, IsHoliday2we(2) gäbe "Fantasieferien \n Pfingstsonntag" (ohne Leerzeichen) zurück, wenn das übermorgen beides zuträfe.

Ansonsten wäre der fix wohl am einfachsten dadurch zu machen, dass man eine foreach-loop mit einem split nutzt, oder?

[CoolTux]

Das hatte mich auch schon gewundert das dort das globale we2holiday eingelesen wird und das Ergebnis pur als get Befehl verwendet wird.
Ist die Frage ob das schon immer so war oder erst vor lurzem dazu kam. Ich muss da mal im Archiv schauen.

[CoolTux]

Der Code ist neu und wurde am 14.02 eingecheckt. Also entweder geht es seit dem nicht mehr oder seit mindestens ein zweites we2holiday Device in global als Attribut eingetragen wurde.

[Beta-User]

Wie Rudi in dem anderen Thread irgendwo geschrieben hat: es ist "relativ neu", dass es überhaupt mehrere h2we-Geräte geben kann. Ich nehme an, dass das Dietmar63 noch gar nicht berücksichtigen konnte und igami bisher nicht über das Thema gestolpert war.

Wird wohl Zeit für einen neuen Thread in "unterstützende Dienste", oder wäre es besser, igami direkt anzuschreiben? (Verschieben macht m.E. nicht den großen Sinn, weil es mind. zwei unterschiedliche Themen sind, wovon eines allerdings m.E. gelöst/geklärt ist.)

@ToKa: Wenn du einen neuen Thread aufmachst, bitte hierher verlinken.

@CoolTux:
Jedenfalls auf die Schnelle sieht es so aus, also wäre das eine größere Umbauaktion gewesen, aber dieser Teil scheint auch vorher schon genau so funktioniert zu haben (Changeset ca. Zeile 733).

[CoolTux]

Ich habe da mal versucht was zu schreiben. Da ich aber selbst nicht weiß wie das Modul funktioniert müsste das bitte einmal jemand testen.

[Beta-User]

Eine Kleinigkeit wäre zu ändern, jedenfalls soweit erkennbar (mit h2we noch nicht auf Funktionalität getestet):
Zeile 312 enthält ein Argument zu viel in CommandGet (das ",1" hinten). Mit der folgenden läd das Modul:

Code Auswählen Erweitern

my $ergebnis = CommandGet(undef,$h2we . ' ' . sprintf("%02d-%02d",$mon+1,$mday));


[CoolTux]

Habe das mal geändert. Die ,1 gehören mit zum Get Befehl. Hatte ich falsch aufgefasst.