[bug+patch] 10_CUL_HM.pm fehlende Fehlerbehandlung bei set ... deviceRename

[betateilchen]

Hallo Martin,

wenn man versucht, ein Homematic device (und seine channels) mit "set ... deviceRename <newName>" umzubenennen und dabei einen (neuen) Namen verwendet, der nicht den FHEM Richtlinien entspricht, passiert einfach gar nix. Der verwendete Aufruf von CommandRename() gibt zwar eine Fehlermeldung zurück, diese wird aber in 10_CUL_HM.pm einfach nirgends entgegengenommen, um sie auszugeben.

Mit dem untenstehenden kleinen patch wird als allererster Schritt geprüft, ob der angegebene neue Name den FHEM-Konventionen entspricht, falls nicht, wird die weitere Verarbeitung mit Ausgabe einer Fehlermeldung beendet.

Wäre schön, wenn Du das bei Gelegenheit einbauen könntest.

(Hintergrund: ich hatte - statt _ im neuen Namen verwendet und musste dann ziemlich lange suchen, warum mein Vorhaben nicht funktioniert.)

Code Auswählen Erweitern


Index: 10_CUL_HM.pm
===================================================================
--- 10_CUL_HM.pm        (revision 23156)
+++ 10_CUL_HM.pm        (working copy)
@@ -4761,6 +4761,8 @@
   }
   elsif($cmd eq "deviceRename") { #############################################
     my $newName = $a[2];
+    return "Invalid characters in name (not A-Za-z0-9._): $newName"
+            if(!goodDeviceName($newName));
     my @chLst = ("device");# entry 00 is unsed
     if ($roleV){
       foreach(1..50){


[frank]

hast du mal eine pm probiert?

[betateilchen]

*schubs*

(damit es nicht untergeht)

[frank]

push

[martinp876]

resolved

[betateilchen]

danke.