[Aufruf] hm-rega Erweiterung

[apollon77]

Die Warnungen kommen von zwei (optionalen) Node-Modulen, mag sein das die irgendwo in hm-rega eine indirekte Abhängigkeit sind (also von einer benutzten Library benutzt werden). Also von daher…ignore

[AlGu]

Danke Apollon, dann bin ich ja beruhigt das es nicht an meinen Änderungen liegt und kann mich um die anderen Baustellen kümmern.

[eric2905]

Moin,

@apollon77:

@Eric: Welche npm Version? 2 oder 3? Bei 3 ist es so … üblicherweise hat node 4 noch npm2 und node 6 dann die 3er ... aber kann man auch einzeln upgraden ... `

@eric2905:

npm : 2.15.11

Node : v4.8.4 `

Ich überlege, ob ich am WE mal auf Node 6 gehe (und ggf. auch auf ein 64Bit-Debian).

Mal sehen, wie das Wetter wird, usw.

@AlGu:

Ich habe den Adapter gerade so umgebaut, dass die ganzen String-Methoden aus dem "dutycycle.fn" in den Adapter gewandert sind und somit keine Abhängigkeit zu den neu implementierten Methoden (Replace, LTrim, RTrim und Mod) der neuen ReGaHss mehr besteht. Dadurch sollte der Adapter nun auch mit älteren CCU Versionen funktionieren. `

Dann probiere ich es doch glatt nochmal aus und meldemich dann.

Gruß,

Eric

[eric2905]

Hi ho,

@eric2905:

Dann probiere ich es doch glatt nochmal aus und meldemich dann. `
habe mir jetzt die 1.6.3 gezogen und es sieht gut aus - keine Meldungen im Log und die beiden Ordner (Raspi-CCU und LAN-GW) wurden angelegt und auch befüllt.

Ich lasse das jetzt mal eine Zeit so laufen.

Gruß,

Eric

Von unterwegs getippert

[bahnuhr]

Rückinfo von mir:

alles geklappt.

Keine Fehlermeldungen im log

DC wird wie gewünscht ausgelesen.

Nun kann ich DC Cuxd wieder löschen.

Danke.

mfg

[lobomau]

Ich habe diesen Rega Adapter 1.6.3 installiert. Ich habe eine CCU2 mit 2.29.22.

ioBroker läuft auf ubuntu-server mit

npm version 3.10.10

node version v6.11.3

Bei mir ist auch das Problem, dass ich DutyCycle zwar anwählen kann es aber aber nicht sichern kann oder etwas in das Feld eintragen kann.

Ausprobiert habe ich schon js-controller Neustart und Host Neustart.

Ansonsten scheint der Rega Adapter so zu laufen wie vorher auch.

[AlGu]

Hallo lobomau,

danke erst mal fürs Installieren und Testen. Ich habe das Problem mit der Einstellen im Adapter jetzt identifiziert und vorerst folgenden Workaround dafür:

• Nach dem Installieren des Adapters über die URL und dem öffnen der Konfiguration im "Instanzen"-Reiter muss auch eine schon vorher vorhandene Einstellung geändert werden, damit die neuen Felder bearbeitet werden können. z.B. eine andere Checkbox "Polling" oder dessen "Intervall" ändern. Danach sollten dann auch die "DutyCycle" Einstellungen möglich sein und mit Abgespeichert werden. Das ganze muss nur einmal so gemacht werden, bis die neuen Einstellungen "DutyCycle" und dessen "Intervall" in ioBroker gespeichert sind. Danach werden beim Laden der Konfig auch die passenden Events abonniert und alles funktioniert wie gewohnt.

Viele Grüße

AlGu

[AlGu]

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

[AlCalzone]

@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=7914

tl;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.

[apollon77]

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

[AlCalzone]

@apollon77:

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)

[apollon77]

hehe … ok, hast Recht

[AlGu]

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

[eric2905]

Moin,

@AlGu:

ich habe jetzt die Version 1.6.4 Veröffentlicht…. `
gilt die URL aus dem ersten Posting noch?

Wenn ich die Nutze, bleibt es bei der Version 1.6.3.

Gruß,

Eric

[AlGu]

Moin,

ja die URL ist immer noch dir richtige. Sorry hatte die Version in einer Datei nicht korrekt hochgezählt.

Habe ich nachgeholt.

Danke und Gruß

AlGu

[eric2905]

Jetzt klappt es. Und alles läuft, wie es soll.

Gruß,

Eric

Von unterwegs getippert

[apollon77]

Habs gerade erstmals installiert, also von ner 1.5xer … klappt alles, konnte es einstellen. Super!

[apollon77]

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

[eric2905]

Ist mir auch gerade aufgefallen … das Log fliegt nur so vorbei [emoji16]

Von unterwegs getippert

[AlGu]

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