Habe gerade mit WeekdayTimer gespielt und dabei festgestellt das eine falsche Definition fhem zum totalen Absturz bringt :)
Schreibt man zB.
define Kaffee_timer weekdayTimer Kaffeemaschine_Sw 1234567|06:30 on 1234567|17:00 off
also weekdayTimer kleingeschrieben, geht fhem in´s Nirvana!
VG
Frank
Konnte man schon immer die Definition eines Gerätes Groß, Klein schreiben wie man wollte.
Bisher war ich der Ansicht, dass das nie möglich war.
Wie ist das bei anderen Geräten?
Zitatalso weekdayTimer kleingeschrieben, geht fhem in´s Nirvana!
Nee, eben NICHT! schreibt man es klein, siehe oben ;)
VG
Frank
Hallo Frank,
das Problem an sich dürfte bereits bekannt sein:
http://forum.fhem.de/index.php/topic,46252.msg380434.html#msg380434 (http://forum.fhem.de/index.php/topic,46252.msg380434.html#msg380434)
ich habe das gerade bei meinem Modul getestet. Und auch da läßt sich das define case-insensitive vornehmen, was für mein Verständnis eigentlich nicht gehen sollte. :-\
Auch bei mir wird das Device ganz normal angelegt. Aber es funktioniert hinterher korrekt, wahrscheinlich weil ich den TYPE und die Modul_Fn hartkodiert vornehme und nicht vom Define ableite, was schätzungsweise beim WeekdayTimer der fall sein dürfte (habe dort jetzt nicht nachgeschaut).
Gruß Benni.
Zitathartkodiert vornehme und nicht vom Define ableite, was schätzungsweise beim WeekdayTimer der fall sein dürfte (habe dort jetzt nicht nachgeschaut).
Ja, genau so etwas habe ich gemacht, und will das auch so lassen.
Und es kann nicht sein, dass dieser Fehler in den letzten zwei bis drei Jahren zum ersten Mal auftaucht.
Ich wollte dir nicht auf den Schlips treten, dass war nur eine Feststellung, ich verwende das Modul nicht! Teste es doch selbst einmal und schreib in define weekdayTimer klein.
P.S. Ich mir für meine Zwecke eine sub gemacht (auch hardcodiert ;) )
Vg
Frank
Fühle mich nicht auf den Schlips getreten.
Bin nur verwundert warum das heut plötzlich und aus heiterem Himmel gleich zwei Mal festgestellt wird - vermute dass sich bei der Definition der Geräte ein Fehler eingeschlichen hat, der die Definition mit Kleinbuchstaben zuläßt.
Einfach zum testen (auf meinem fhem Testsystem) ein dummy anlegen
define test_du dummy
dann
define test_wdt weekdayTimer test_du 1234567|06:30 on 1234567|17:00 off
und fhem ist weg.
Zitat von: franky08 am 28 Dezember 2015, 20:31:12
und fhem ist weg.
Das kommt vermutlich auf die Version an - mein Test-FHEM läuft noch.
Ist diese:
# $Id: 98_WeekdayTimer.pm 10126 2015-12-07 19:00:13Z dietmar63 $
##############################################################################
VG
Frank
Zitat von: franky08 am 28 Dezember 2015, 20:31:12
Einfach zum testen (auf meinem fhem Testsystem) ein dummy anlegen
define test_du dummy
dann
define test_wdt weekdayTimer test_du 1234567|06:30 on 1234567|17:00 off
und fhem ist weg.
Ich glaube dir doch!
Ich verstehe nur nicht warum es möglich ist, Geräte anzulegen, deren Typ jede beliebige Groß- Kleinschreibung annehmen kann. Dann funktionieren auch solche Sachen wie:
foreach my $wdName ( sort keys %{$modules{WeekdayTimer}{defptr}} ) {
WeekdayTimer_SetParm($wdName);
}
nicht mehr, weil nur WD gefunden werden, die genau so
WeekdayTimer geschrieben sind.
Vielleicht irre ich mich und es war schon immer möglich WD oder andere Dinge klein zu schreiben.
Vielleicht ist dies ein Fall für Rudi.
Also ich mache erst einmal nichts, bis wir Klarheit darüber haben, ob das so richtig ist bei der Definition von WeekdayTimer weekdaytimer klein zu zu schreiben.
Ich finde, dass es falsch ist.
Mehr wollte ich mit dem Hinweis auch nicht sagen, ist wohl wirklich ein Fall für Rudi eine case-insensitive Prüfung einzubauen.
VG
Frank
Zitat von: franky08 am 28 Dezember 2015, 20:42:11
Ist diese:
# $Id: 98_WeekdayTimer.pm 10126 2015-12-07 19:00:13Z dietmar63 $
##############################################################################
VG
Frank
Meine ist diese
# $Id: 98_WeekdayTimer.pm 6186 2014-06-30 21:10:20Z dietmar63 $
Ja ich weiß - uralt.
Zitat von: Dietmar63 am 28 Dezember 2015, 21:08:11
Ich finde, dass es falsch ist.
Jep, aber ich meine Rudi hat da mal was eingeworfen.
Ich denke aber auch das FHEM durchgängig case-sensitiv sein sollte ich weiß aber nicht wie das mit der direkten Eingabe in die fhem.cfg gehandhabt werden kann.
Die fhem.cfg bearbeite ich nicht direkt und wie man sieht mach ich nicht auf allen meinen Systemen ein update - never change a running system.
Aber man sieht das mein "Uralt-FHEM" immun ist gegen solche fehlerhaften Eingaben - warum auch immer.
@Puschel74
Aber interessant ;) Dürfte auch den Modulautor interessieren. Wenn du die Version noch auf deinem System hast, könntest du die mal anhängen? Würde ich gern mal testen und mit der aktuellen Version vergleichen.
P.S. während ich geschrieben habe hast du deinen Post ergänzt, ich arbeite auch nur noch in der DEF, nur wenn es unvermeidlich ist bearbeite ich die fhem.cfg direkt
P.P.S. Dann könnte man vlt. Nachvollziehen ob es die fhem.pl oder das Modul ist, mein Interesse ist geweckt :)
VG
Frank
Zitat von: franky08 am 28 Dezember 2015, 22:35:58
@Puschel74
Aber interessant ;) Dürfte auch den Modulautor interessieren. Wenn du die Version noch auf deinem System hast, könntest du die mal anhängen? Würde ich gern mal testen und mit der aktuellen Version vergleichen.
P.S. während ich geschrieben habe hast du deinen Post ergänzt, ich arbeite auch nur noch in der DEF, nur wenn es unvermeidlich ist bearbeite ich die fhem.cfg direkt
VG
Frank
Klar hab ich die Version noch auf dem System - sonst könnt ich ja die Version nicht posten ;)
Anbei
Zum P.S.: Ja da hat sich was überschnitten.
Mit der WD wirst du nichts anfangen können. Es hat sich zuviel geändert.
Auf meinem Produktivsystem hab ich noch diese
Zitat# $Id: 98_WeekdayTimer.pm 8621 2015-05-22 18:20:29Z dietmar63 $
im Einsatz.
Ob die auch "resitent" gegen Falscheingaben ist werde ich
morgen heute probieren - erstmal Gn8.
Ja auch auf meinem Produktivsystem habe ich "ältere" Versionen im Einsatz,
Memo an mich - ein Test-System mit den aktuellsten Versionen zu Testzwecken aufsetzen.
Tschuldigung, ich muss ins Bett, muss arbeiten ;)
Gute Nacht
Frank
98_WeekdayTimer, 98_Heating_Control: issue http://forum.fhem.de/index.php/topic,46312.0.html fixed.
No crash anymore when using a wrong definition name(capitalization).