NEWS
[Aufruf] hm-rega Erweiterung
-
Nun zum eigentlichen Problem: (ggf. an die Profis gerichtet:)
Wenn ich eine neue Einstellung in dem Adapter einbaue (also in "admin/index.html") dann wir ja beim öffnen der Konfig die Methode "function load(settings, onChange)" ausgeführt. Leider ist dann ja zu diesem Zeitpunkt noch nicht meine neu hinzugekommen Einstellung in der "settings" Property und wird daher nicht mit "setValue" initialisiert und die Events "change" nicht gesetzt.
Ich denke dieses Problem werden schon mehrere Adapter-Entwickler gehabt haben und vielleicht gibt es da ein einheitliches Vorgehen das zu lösen?!
Alternativ kann ich natürlich im Code prüfen ob eine Einstellung nicht in den "settings" beim "load" vorhanden ist und diese dann einzeln initialisieren. Das ist dann aber nicht mehr so schön dynamisch wie bisher und man muss bei jedem neuen Feld daran denken dies dort wieder mit einzubauen. z.B.:
function load(settings, onChange) { for (var key in settings) { if (settings.hasOwnProperty(key)) setValue(key, settings[key], onChange); } if(settings["syncDutyCycle"] == null) { setValue("syncDutyCycle", false, onChange); } if(settings["pollingIntervalDC"] == null) { setValue("pollingIntervalDC", 60, onChange); } ... }Viele Grüße
AlGu
-
Ich denke dieses Problem werden schon mehrere Adapter-Entwickler gehabt haben und vielleicht gibt es da ein einheitliches Vorgehen das zu lösen?! `
Relevant: http://forum.iobroker.net/viewtopic.php?f=24&t=7914tl;dr: Problem ist bekannt. Eine elegante Lösung lässt noch auf sich warten:
> Geofency fixed that by handling it alone. read io-package and parse instanceObjects and create the objects by it's own, but I think js-controller should handle that
Wobei, das war bezogen auf instanceObjects. Bin mir gerade nicht 100% sicher, wie es mit .native (Konfig) aussieht. -
Undefiniert ist "undefined" und nicht "null" :-) Versuch das mal
Idee wäre auch eher die Fehlenden Settings vor der Schleife in das Array mit dem Defaultwert einzufügen.
also eher ein "if (settings.syncDutyCycle===undefined) settings.syncDutyCycle = false;" und dann sid sie für die schleife drin :-)
-
Undefiniert ist "undefined" und nicht "null" :-) `
== null prüft sowohl auf undefined als auch null.=== null funktioniert nur wenn der Wert exakt null ist.
Alternativen wären
typeof settings.syncDutyCycle === "undefined"und
!("syncDutyCycle" in settings) -
Hallo zusammen,
ich habe jetzt die Version 1.6.4 Veröffentlicht mit einem Fix für die Adapter-Konfiguration nach einem Update:
- Wenn der Adapter von einer älteren Version (vor 1.6.0) auf diese Version aktualisiert wird, dann sollten jetzt die neuen Einstellungen auch direkt änderbar sein und nicht mehr erst mit anderen Settings zusammen.
Ich hoffe es gibt noch jemanden, der den Adapter jetzt Testet der noch von der Version kleiner als 1.6.0 kommt. Dann kann ich sicher sein, dass der Adapter nun auch in der Konfig-Seite richtig funktioniert.
@apollon77 und AlCalzone:
Wenn es irgendwann ein einheitliches Vorgehen für Adapter-Default-Werte gibt, dann baue ich das gerne wieder um. Ich denke eine wie auch immer gebaute "adapterDefault"-Liste könnte in Zukunft auch für einen Button "Reset Defaults" oder so nützlich sein.
Viele Grüße
AlGu
-
PS:Bitte nur noch am Logging schrauben. Es werden gerade wohl alle Neuen Werte als info geloggt, das ist ein bissl viel … debug reicht dafür. SDag kurz wenn Du mit allem zufrieden bist dann checke ich den PullRequest nochmal genau und gebe Ihn frei!
Super Arbeit!
Ingo F
-
Ich habe Version 1.6.5 veröffentlicht mit der einzigen Anpassung, dass die Log-Einträge mit "debug" gemacht werden.
Vielen Dank noch mal an alle Tester und wenn es in den nächsten Tagen keine weiteren Fehler oder so gibt, dann mache ich den PullRequest :-)
Viele Grüße
AlGu
-
Hi,
ich habe gerade auf Rega 1.6.5 upgedated und den DutyCycle aktiviert mit polling 30s.
Ich habe node version 4.8.1 mit NPM 2.15.11 und eine CCU2 mit version 2.29.22 mit aktivierter community Rega.
Dazu dann noch zwei Gateways. (die Runden)
Ich bekomme einen Fehler. Es sieht so aus, dass die Gateways nicht eingelesen werden ?
Also, den Thread habe ich mir durchgelesen. Ich meine, dass ich alle Voraussetzungen erfülle.
@AlGu Hast du eine eine Ahnung woran es liegen könnte ?
hm-rega.0 2017-09-19 15:13:42.903 error hm-rega.0 2017-09-19 15:13:42.903 error Cannot parse answer for dutycycle: {ADDRESS KEQ0102260 CONNECTED 1 DEFAULT 1 DESCRIPTION CCU2-Coprocessor DUTY_CYCLE 6 FIRMWARE_VERSION 2.8.5 TYPE CCU2} {ADDRESS KEQ0851563 CONNECTED 1 DEFAULT 0 DESCEdit: noch ein Hinweis. Im Moment nutze ich das TCL Script auf der CCU2. Ich bekomme dort Werte für die Zentrale und für die beiden gateways.
Bei der CCU2 steht in der Fehlermeldung, dass sie einen DC 6 hat. Was korrekt war. Die States werden aber nicht angelegt.
-
Hallo loxxer01,
kannst du das Skript hier bitte einmal auf deiner CCU2 (z.B. bei Skript Testen) ausführen und mir das Ergebnis hier (oder per PN) schicken. Dann kann ich damit vielleicht feststellen, warum bei dir das Parsen der Bidcos Antwort nicht funktioniert.
string stderr; string stdout; system.Exec("/bin/sh -c '" # 'echo "load tclrpc.so; puts [xmlrpc http://127.0.0.1:2001/ listBidcosInterfaces ]" |tclsh' # "'", &stdout, &stderr); WriteLine(stdout);Danke und Gruß AlGu
-
Hallo loxxon1,
ich habe in meinem Fork: https://github.com/AlGu1/ioBroker.hm-rega einen fix für den Fehler gebaut und veröffentlicht. Wenn du möchtest, kannst du diese Version bei dir gerne mal testen. Das Problem lag an dem Einlesen der Daten, dabei habe ich nicht an allen Stellen mit einem String wie z.B. "Lan Interface" (mit Leerzeichen) gerechnet. Ich habe das Einlesen jetzt etwas umgebaut, so dass es stabiler und toleranter sein sollte.
Danke und Gruß AlGu
-
Hallo zusammen,
es wäre total super, wenn noch ein paar andere meine letzte Anpassung testen könnten, bevor ich einen Pull Request mache.
Einfach über meinen Fork https://github.com/AlGu1/ioBroker.hm-rega installieren und gerne Feedback hier geben.
Danke und Gruß AlGu
621
Online32.5k
Benutzer81.6k
Themen1.3m
Beiträge