FHEM Forum

FHEM => Automatisierung => Thema gestartet von: franky08 am 28 Dezember 2015, 17:19:24

Titel: WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: franky08 am 28 Dezember 2015, 17:19:24
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
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: Dietmar63 am 28 Dezember 2015, 17:28:17
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?
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: franky08 am 28 Dezember 2015, 17:31:56
Zitatalso weekdayTimer kleingeschrieben, geht fhem in´s Nirvana!

Nee, eben NICHT! schreibt man es klein, siehe oben  ;)

VG
Frank
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: Benni am 28 Dezember 2015, 17:33:10
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.

Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: Dietmar63 am 28 Dezember 2015, 19:18:17
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.
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: franky08 am 28 Dezember 2015, 19:29:45
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
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: Dietmar63 am 28 Dezember 2015, 20:26:36
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.
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag 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.
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: Puschel74 am 28 Dezember 2015, 20:36:35
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.
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag 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
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: Dietmar63 am 28 Dezember 2015, 21:08:11
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.
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: franky08 am 28 Dezember 2015, 21:13:57
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
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: Puschel74 am 28 Dezember 2015, 22:27:05
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.
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag 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
P.P.S. Dann könnte man vlt. Nachvollziehen ob es die fhem.pl oder das Modul ist, mein Interesse ist geweckt  :)
VG
Frank
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: Puschel74 am 28 Dezember 2015, 22:47:02
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.
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: Dietmar63 am 28 Dezember 2015, 23:54:54
Mit der WD wirst du nichts anfangen können. Es hat sich zuviel geändert.
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: Puschel74 am 29 Dezember 2015, 00:05:04
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.
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: franky08 am 29 Dezember 2015, 00:25:40
Tschuldigung, ich muss ins Bett, muss arbeiten  ;)

Gute Nacht
Frank
Titel: Antw:WeekdayTimer falsch definiert bringt fhem zum Absturz!
Beitrag von: Dietmar63 am 09 Januar 2016, 19:34:12

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).