FHEM Forum

FHEM - Hausautomations-Systeme => Homematic => Thema gestartet von: betateilchen am 14 November 2020, 20:06:00

Titel: [bug+patch] 10_CUL_HM.pm fehlende Fehlerbehandlung bei set ... deviceRename
Beitrag von: betateilchen am 14 November 2020, 20:06:00
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.)


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){
Titel: Antw:[bug+patch] 10_CUL_HM.pm fehlende Fehlerbehandlung bei set ... deviceRename
Beitrag von: frank am 27 Dezember 2020, 11:29:27
hast du mal eine pm probiert?
Titel: Antw:[bug+patch] 10_CUL_HM.pm fehlende Fehlerbehandlung bei set ... deviceRename
Beitrag von: betateilchen am 22 Februar 2021, 12:33:09
*schubs*

(damit es nicht untergeht)
Titel: Antw:[bug+patch] 10_CUL_HM.pm fehlende Fehlerbehandlung bei set ... deviceRename
Beitrag von: frank am 28 Februar 2021, 08:20:12
push
Titel: Antw:[bug+patch] 10_CUL_HM.pm fehlende Fehlerbehandlung bei set ... deviceRename
Beitrag von: martinp876 am 28 Februar 2021, 18:41:35
resolved
Titel: Antw:[bug+patch] 10_CUL_HM.pm fehlende Fehlerbehandlung bei set ... deviceRename
Beitrag von: betateilchen am 28 Februar 2021, 21:06:56
danke.