Test Adapter time-switch v2.0.x GitHub/Latest

[walli545]

Dank der guten Unterstützung der Apple Nutzer konnte ich den Bug nun fixen, indem ich eine virtuelle Maschine mit MacOs aufgesetzt habe, danke dafür Bitte ausprobieren, ob es nun nach einem Update funktioniert @frankthegreat @berndhome, evtl. auch mit Fully mal ausprobieren, wobei das auch was anderes sein könnte.

@Iccube @arteck Wie schauen den bei euch die Schedule Daten aus (im State)? Vermutlich habt ihr bevor der Fehler kam das automatische Schalten angemacht?
@berndhome Das dürfte nicht das Problem sein, JSON.parse gibt ja ein aus dem String geparstes Objekt zurück und die triggers sind auch im JSON ein Array. Das wird dann auch in JS ein Array.

[Homer.J.]

@walli545 funktioniert nun auch unter Apple. Danke

[berndhome]

@walli545 neueste Version läuft nun unter Fully und auf dem IPad. Vielen Dank für die schnelle Umsetzung!
Jedoch nach wie vor der Fehler "property map....".

time-switch.0	2020-04-30 20:04:28.320	error	at process._tickCallback (internal/process/next_tick.js:68:7)
time-switch.0	2020-04-30 20:04:28.320	error	at fulfilled (/opt/iobroker/node_modules/iobroker.time-switch/build/main.js:5:58)
time-switch.0	2020-04-30 20:04:28.320	error	at Generator.next (<anonymous>)
time-switch.0	2020-04-30 20:04:28.320	error	at TimeSwitch.<anonymous> (/opt/iobroker/node_modules/iobroker.time-switch/build/main.js:127:52)
time-switch.0	2020-04-30 20:04:28.320	error	(559640) TypeError: Cannot read property 'map' of undefined
time-switch.0	2020-04-30 20:04:28.320	error	(559640) uncaught exception: Cannot read property 'map' of undefined

Deine Erklärung hinsichtlich array kann ich daher so nicht nachvollziehen:
Würde .map als Array-Funktion interpretiert, würde doch nicht nach einer Property "map" gesucht (siehe Fehlertext).
Und diese Property gibt es auch wirklich nicht im "scheduleString" (lt. Debug-Ausgabe).
Wäre in der arrayfunktion ein syntaktischer Fehler würde ein völlig anderer Fehler kommen.

so sieht bei mir übrigens der Übergabewert (lt. Debug) aus:

onScheduleChange: {"name":"Wärmepumpe","triggers":[{"type":"TimeTrigger","hour":13,"minute":35,"weekdays":[1,2,3,4,5,6,0],"id":"0","action":{"type":"OnOffStateAction","valueType":"boolean","onValue":true,"offValue":false,"booleanValue":true,"idsOfStatesToSet":["javascript.0.unipi.relays.Waermepumpe.power"]}},{"type":"TimeTrigger","hour":15,"minute":25,"weekdays":[1,2,3,4,5,6,0],"id":"8","action":{"type":"OnOffStateAction","valueType":"boolean","onValue":true,"offValue":false,"booleanValue":false,"idsOfStatesToSet":["javascript.0.unipi.relays.Waermepumpe.power"]}}]}

Die Schaltfunktion habe ich auch mal getestet: "an" und "aus" nach Zeit funktioniert offensichtlich (trotz der Fehlermeldung)....

[Wolfi]

Neue Version wieder unter https://github.com/walli545/ioBroker.time-switch/tarball/19-switch-boolean-values

??

[walli545]

@Wolfi Ja genau, sobald sich das ändert, gebe ich es hier bekannt.
@berndhome ich glaube ich hab was gefunden, bitte ausprobieren. Wenn du den Fehler weiterhin hast, bitte den ganzen Log anhängen, habe außerdem noch mehr Debug Output hinzugefügt.

FYI: Ich bin mir ziemlich sicher, dass die Stelle wo der Fehler geworfen wird so funktioniert. Ich denke, dass der der Fehler wo anders liegt, z.B. im Aufruf der Methode. Dass es funktioniert, kannst du ganz einfach in der Browser Konsole ausprobieren. Die Fehlermeldung ist ein wenig irreführend, es ist nämlich egal ob auf eine Methode oder eine Property von undefined zugegriffen wird, der Fehler ist gleich (Test: undefined.map() und undefined.a).

Spoiler

Der Test für JSON.parse:

const a = JSON.parse('{"name":"Wärmepumpe","triggers":[{"type":"TimeTrigger","hour":13,"minute":35,"weekdays":[1,2,3,4,5,6,0],"id":"0","action":{"type":"OnOffStateAction","valueType":"boolean","onValue":true,"offValue":false,"booleanValue":true,"idsOfStatesToSet":["javascript.0.unipi.relays.Waermepumpe.power"]}},{"type":"TimeTrigger","hour":15,"minute":25,"weekdays":[1,2,3,4,5,6,0],"id":"8","action":{"type":"OnOffStateAction","valueType":"boolean","onValue":true,"offValue":false,"booleanValue":false,"idsOfStatesToSet":["javascript.0.unipi.relays.Waermepumpe.power"]}}]}')

a.triggers.map(t => t.type)

Einfach zeilenweise in die Konsole eingeben.

[berndhome]

@walli545 Danke für die Infos, ich werde mal testen.
Noch ein Verbesserungsvorschlag: "die "ID vom geschalteten State" besser nicht in jeden "TimeTrigger" Datenpunkt mit aufnehmen, sondern dafür einen weiteren Datenpunkt spendieren.
Ich hatte erst angenommen, dass das zeit-gesteuerte Schalten nicht funktioniert, bis ich festgestellt habe,
dass (kommt wohl normal nicht vor, jedoch bei meinen Tests...) Änderungen bei "ID vom geschalteten State" sich auf neue Zeitvorgaben, jedoch nicht auf bis dato gesetzte Zeitvorgaben auswirkt. Daher der Vorschlag mit dem Datenpunkt
( keine redundante Daten) oder aber sicher stellen, dass bei einer ID-Änderung diese in alle "TimeTrigger" erfolgt.

Ansonsten geht bei mir um 10:00Uhr die Beleuchtung an und um 13:00Uhr die Wärmepumpe.
Dies ist aber wohl kein geplantes Feature

[Iccube]

@walli545 mit der neuen version kommt der Fehler nicht mehr

[Wolfi]

Was ist jetzt das Probehm?

1. Habe das Adapter komplett deinstalliert.
2. iobroker auch neu gestartet usw.
   3, Adapter Neu installiert. (Mit Meldung erfolgreich)

aber die Instanz fehlt?

[FredF]

@Wolfi sagte in Test Adapter time-switch v1.0.x GitHub/Latest:

Was ist jetzt das Probehm?

Bei dir Du musst bei Adaptern von Github die Instanz selber erzeugen mit nochmaligen betätigen des plus buttons.
Erst dann siehst du bei installiert eine 1

[Wolfi]

@FredF

a jetzetle sagt der schwabe

Danke

[walli545]

@berndhome Wenn du die geschaltene Id änderst, sollten auch für alle bestehenden Actions die Ids geändert werden. Zum Thema Redundanz: Ja dazu habe ich mir auch schon Gedanken gemacht. Zumindest in der Adapterlogik möche ich die Struktur so aufrecht halten, da ich da geplant habe per Decorator Pattern dann z.B. eine ConditionalAction hinzuzufügen, die eine normale Action wrappt und nur bei einer Bedingung auslöst. Evtl. ändere ich die Serialisierung so ab, dass es pro Schedule eine On- und eine Off Action gibt und in den Triggern dann nur eine Referenz dazu exisitiert

[SVallant]

Hallo,
ich habe mal ein Widget für unsere Visu gebaut:
Vielleicht habt ihr Zeit und guckt Euch das mal an:
http://dev.builder.minukodu.de/minukodu/?url=http%3A%2F%2F94.130.57.38%3A9090&file=devTimeSwitch_ReadOnly.json&forceUpdate

Die Konfiguration kann hier verändert werden:
http://dev.builder.minukodu.de/

connect to: http://94.130.57.38:9090
ConfigFile: devTimeSwitch_ReadOnly

auf http://94.130.57.38:8081 läuft übrigens unsere ioBroker-demo-Instanz.
RO-Zugang mit demo/demo

Danke
Sepp

[walli545]

@SVallant Cool! Falls sich in dieser Version noch was an der Widgetlogik ändert, gebe ich dir bescheid. Wo gibt es den Quellcode dazu?

[SVallant]

@walli545 Danke, nightly Build hier zum Download

TimeSwitch-Widget ist minuvis-webapp.iobroker\src\components\widgets\TimeSwitch.js

Die Visu ist eine React-App ...

Werde das noch ein bißchen testen und dann in die nächste Version bei Github einfliessen lassen.
Sage dir dann auch Bescheid !

[frankthegreat]

@walli545 Gerade gesehen, das es eine neue Version gibt
Wird jetzt auch auf iPad und iPhone richtig dargestellt

Super Job, den du hier machst. Ich freue mich schon auf weitere Features von dir.

[Wolfi]

@walli545
Komisches verhalten unter Android
Sobald ich mit Android Tablet oder Handy versuche im Zeit zu ändern oder den Namen zu ändern.
Kommt kurz die Tastatur hoch die View mit Time-switch schließt sich und ich lande in meiner Hauptview dem Dasbord??
Via PC funktioniert es.

[liv-in-sky]

@walli545 sorry - ich schon wieder - habe heute auf js-controller 3 upgedatet

jetzt kommt das im browser:

die widget scheinen aber zu funktionieren und auch zu schalten

im log vom iobroker kommt das:

[walli545]

@Wolfi kann ich bei mir so nicht feststellen, hat sonst jmd das Problem?
@liv-in-sky Ersters ist normal, zweites sollte nach einem Update auf die neueste Version behoben sein

[liv-in-sky]

@walli545 die neue version , die kommt oder die schon da ist? 1.1.0 ?

[walli545]

@all Ich habe nun noch die vorgeschlagenen Änderung von @berndhome und @SVallant eingebaut. Dadurch hat sich erneut
die Struktur der Schedule Daten geändert, also bitte nochmal bereits vorhandene vor dem Update löschen. Ich versuche von jetzt an solche Änderungen zu vermeiden bzw. wenn dann Upgrade Logik einzubauen. Aktuell denke ich aber, das das noch nicht sinnvoll ist, da der Adapter ja noch in der Testphase ist.
Zudem habe ich noch einige Unit Tests geschrieben und die Adapterlogik dahingehend geändert, dass nun nicht mehr bei einer Änderung an einem Schaltplan alles komplett jedes mal neu eingelesen/registriert wird, sondern, nur wenn Änderungen von außen kommen. Das sollte besonders auf schwächeren Hosts und vielen Triggern weniger Performance kosten.

Bitte die Version von https://github.com/walli545/ioBroker.time-switch/tarball/dev installieren und nochmal ausgiebig testen. Dann kann man über einen neuen Release und über neue Features nachdenken

Außerdem habe ich das README angepasst und es gibt nun auch eine deutsche Anleitung @SMS.