WeekdayTimer falsch definiert bringt fhem zum Absturz!

[franky08]

Habe gerade mit WeekdayTimer gespielt und dabei festgestellt das eine falsche Definition fhem zum totalen Absturz bringt 
Schreibt man zB.

Code Auswählen Erweitern

define Kaffee_timer weekdayTimer Kaffeemaschine_Sw 1234567|06:30 on 1234567|17:00 off

also weekdayTimer kleingeschrieben, geht fhem in´s Nirvana!

VG
Frank

[Dietmar63]

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?

[franky08]

also weekdayTimer kleingeschrieben, geht fhem in´s Nirvana!

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

VG
Frank

[Benni]

Hallo Frank,

das Problem an sich dürfte bereits bekannt sein:

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.

[Dietmar63]

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

[franky08]

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

[Dietmar63]

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.

[franky08]

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.

[Puschel74]

und fhem ist weg.

Das kommt vermutlich auf die Version an - mein Test-FHEM läuft noch.

[franky08]

Ist diese:

Code Auswählen Erweitern

# $Id: 98_WeekdayTimer.pm 10126 2015-12-07 19:00:13Z dietmar63 $
##############################################################################

VG
Frank

[Dietmar63]

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:

Code Auswählen Erweitern


  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.

[franky08]

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

[Puschel74]

Ist diese:

Code Auswählen Erweitern

# $Id: 98_WeekdayTimer.pm 10126 2015-12-07 19:00:13Z dietmar63 $
##############################################################################

VG
Frank

Meine ist diese

Code Auswählen Erweitern

# $Id: 98_WeekdayTimer.pm 6186 2014-06-30 21:10:20Z dietmar63 $
Ja ich weiß - uralt.

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.

[franky08]

@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

[Puschel74]

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