[Vorlage] Generisches Fensteroffenskript + Vis

[Pittini]

@wolf-unterwegs Siehe Zeile 27. Raum 10 kontrollieren, da passt was nich, der kommt als undefined.

[Wolf-unterwegs]

@pittini
Das passiert immer wieder, wenn ich den hm-rpc.0 neu starte.
Hatte den fehlenden Raum aber danach gleich wieder richtig konfiguriert - das Problem ist aber noch immer existent.
Ist nur immer eine elende Warterei, bis ich das script wieder gestoppt wird. Da ist das Log dann gleich wieder riesig und lässt sich damit auch schlecht durchsuchen.
Ich versuche heute Abend nochmal den richtigen Teil des Logs zu extrahieren.

[Pittini]

@wolf-unterwegs Wenn Du an Aufzählungen was änderst solltest auch den JS Controller neustarten, das Problem hängt definitiv mit dem leeren Raum zusammen, dadurch entstehen dann diese Fehler:

2021-10-24 10:38:08.290  - warn: javascript.0 (19240) getState "javascript.0.FensterUeberwachung..SendVentMsg" not found (3)

Man achte auf die 2 Punkte vor SentVentMsg, da dazwischen wär normal der Raumname. Und deswegen droppen da auch so viele Fehler.

@pittini

Das passiert immer wieder, wenn ich den hm-rpc.0 neu starte.

Ja dann stell das syncen halt ab im Adapter oder nenn den Raum in der CCU gleich richtig.

[Wolf-unterwegs]

@pittini
Habe jetzt nochmals alles überprüft. Das Problem kam über den hm-rpc.2, der die HMIP devices holt. Nach jedem aktualisieren waren mal Räume und/oder Gewerke da, dann wieder weg.
Habe den dann gestoppt, die DPs gelöscht und dann wieder gestartet.
Jetzt läuft wieder alles einwandfrei - verstehe wer will.
Aber danke für deine Hinweise, die mich auf die Spur gebracht haben.

[luft-post]

Moin zusammen,
bei meinen skript habe ich gerade beobachtet das die Sammelmeldung nicht mehr aktualisiert wird (siehe Bild)

ist das bei euch auch so? Die Anzeige im VIS zeigt soweit alles richtig an.

gruß
Matthias

[luft-post]

@luft-post
ach ja, benutze die Skriptversion = "1.6.14" //vom 15.10.2021

[Pittini]

@luft-post Wenns im Vis stimmt, im Admin aber nicht, und angesichts der vielen "null" Felder, tippe ich auf Anzeigefehler im Admin. Schon mal die Seite neu geladen? Oder mal mit anderem Browser versucht?

[luft-post]

@pittini
Ja habe ich. Die sammeln Meldung werte ich über einen weiteren Adapter aus und auch der Reagiert nicht, daher denke ich das es sich hier nicht um einen Anzeige Fehler handelt.

Gruß
Matthias

[Pittini]

@luft-post Nichtsdestotrotz, die ganzen "null" Felder dürfen nicht sein, die werden ja schon bei Installation bzw. erstem Start angelegt und mit Werten gefüllt. Iwas liegt das ganz arg im Argen. Lösch mal den ganzen Zweig und starte das Skript neu und gugg ob die dann richtig, also mit Inhalt, angelegt werden. Sieht bei mir z.B. so aus:

[Mike Hellracer]

@pittini
Hi,
Habe auch hier das Skript aktualisiert und bin wieder mal beeindruckt, was alles geht.
Top Arbeit! Danke. Danke. Danke.
Ich hätte eine Idee, Anregung, Wunsch für die nächste Version.

1. Eine eigene Aufzählung für den Türalarm
2. Eine eigene Aufzählung für den Fensteralarm
   Dann kann der Benutzer alle Devices überwachen aber erhält Alarmmeldungen nur für gewünschte Geräte.

Optional, wäre es schön,wenn du die Abfrage der Aufzählung (getEnum) anstatt fest auf "functions" varaible (z. Bsp. EnumType) gestalten könntest.

[Pittini]

@mike-hellracer sagte in [Vorlage] Generisches Fensteroffenskript + Vis:

Ich hätte eine Idee, Anregung, Wunsch für die nächste Version.

1. Eine eigene Aufzählung für den Türalarm
2. Eine eigene Aufzählung für den Fensteralarm
   Dann kann der Benutzer alle Devices überwachen aber erhält Alarmmeldungen nur für gewünschte Geräte.

Versteh ich nich. Es gibt doch die Funktionen Tuer und Fenster, damit haste doch beides getrennt schon. Und ob dann Meldungen kommen oder nicht, stellste ja in den Objekten ein
bei den sendxxxMsg Dps.

Optional, wäre es schön,wenn du die Abfrage der Aufzählung (getEnum) anstatt fest auf "functions" varaible (z. Bsp. EnumType) gestalten könntest.

Warum? Wozu?

[Mike Hellracer]

@pittini
@pittini
Vlt ist der Sinn nicht ganz klar geworden.
Ich möchte erreichen das alle Fenster und Türen in der Übersicht im VIS zu sehen sind, ich Warnungen bei zu langem Öffnen für die Aussentüren und Fenster bekomme und nur für die Aussentüren bzw. TerrassenTüren oder ausgewählte Fenster zusätzlich eine Meldung (Alexa/Telegram) bekomme.

Punkt optional Ist eher ein nice to have.
Das wären 2 Zeilen Code im Script und würde ein wenig mehr Übersicht (eine eigene Gruppe für die Aufzählungen ) in den Aufzählungen ermöglichen. So könnten die "normalen" Aufzählungen von denen, die direkt auf den DP liegen getrennt werden.

[Pittini]

@mike-hellracer Hmm, ich verstehs immer noch nicht, bzw. bin der Meinung dass das alles schon vorhanden ist. Wenns nur 2 Zeilen Code sind, mach doch nen Fork und zeigs mir, evtl. steh ich ja auch nur aufm Schlauch.

[Mike Hellracer]

@pittini
Mag sein, daß ich es falsch umgesetzt habe, aber ich habe alle Fenster und Türen mit der Funktion versehen -> alle werden in der Liste angezeigt.
Wenn ich jetzt die Telegram Option einschalte bekomme ich für ALLE Türen und Fenster ein Telegram. Also auch für indoor Türen.
Ebenso verhält es sich für die Warnung wenn zu lange geöffnet. Bei 4 Personen die sich im Haushalt bewegen. Mutiert das Handy zum Vibrator.
Und das dann auch noch per Alexa und der WAF ist im Keller.

Ich möchte erreichen das alle Fenster und Türen in der Übersicht im VIS zu sehen sind.
Und Warnungen bei zu langem Öffnen nur für die Aussentüren und Fenster bekomme.
Am flexibelsten wäre es wenn man für alle eigene Aufzählungen ermöglichen würde.
Z.Bsp

• WindowWarnMessage
• DoorWarnMessage
  Ggf. Sogar getrennt in Alexa und Telegram

Für das optionale Thema mache ich die 2 Zeilen wenn ich wieder vor einer Tastatur sitze.

[Pittini]

@mike-hellracer Naja, das Skript arbeitet Raumbezogen, da kannste alles einstellen. Wenn ich Dich richtig versteh willst Du jetzt bei nem Raum mit ner Innen und ner Außentüre das getrennt haben, das geht tatsächlich nicht und das werd ich auch nicht einbauen, da müßt ich das ganze Konzept ändern und Anfänger/Einsteiger blicken iwann dann auch nimmer durch. Und genau für die ist das Skript ja ursprünglich konzipiert. Poweruser können ja gern das Skript forken und sich austoben.

[Mike Hellracer]

@pittini
Wahrscheinlich hast du recht mit der Komplexität. Und wan ich bei 2x 3Jährige dazu komme das zu implementieren weiß ich nicht.
Aber die 2 Zeilen für die variable aufzahlubgsgruppierung schicke ich noch.
Danke für deinen Aufwand .

[Mike Hellracer]

@pittini
@pittini
Hi, hoffe ich habe mich nicht vertippt, aber habe es jetzt doch mal auf der Handy Tastatur versucht...
hier die 2 Zeilen Code

Die Definition der Variable:

Spoiler

const WhichEnumCategoryToUse = "functions"; // Legt fest in welcher Kategorie sich die Aufzählungen befinden!

Und die Umstellung in Zeile 129 (getEnums) von 'Function' auf Variable:

Spoiler

let Funktionen = getEnums(WhichEnumCategoryToUse); //Array mit Aufzählung der Funktionen

zum Vergleich die Orignialzeile 129

Spoiler

let Funktionen = getEnums('functions'); //Array mit Aufzählung der Funktionen

[Mike Hellracer]

@pittini
sorry falsches Thema

[JoergH]

@Pittini

Was mir gestern aufgefallen ist, wenn der Raumname ein Punkt oder Freizeichen enthält, dann wird ein weiterer Unterordner angelegt. Ist kein Beinbruch, führt halt dazu, dass "gr. Bad" zu einem Ordner "gr" und einem weiteren Unterordner "Bad" führt, in dem dann die DP zu finden sind.

[Pittini]

@joergh Punkte und Leerzeichen sind ja auch keine gute Idee bei Funktions und/oder Raumnamen. Im neuen Admin kannste sowas auch gar nimmer anlegen.