SOLVED Alle Mitglieder einer Aufzählung schalten (gelöst)

bahnuhr

habe dein Script bei mir probiert.
Bei mir mit dem Gewerk "Licht" funktioniert es einwandfrei:

var cacheSelectorState = $('channel[state.id=*.STATE](functions="Licht")')
cacheSelectorState.each(function (id,i) { 
    setState(id, true);
});  
  console.log('Gesamtes Licht an');

NyanCat

const enum_sonoff = $('state[id=*.POWER](functions="light")');
const enum_tradfri = $('state[id=*.state](functions="light")');

on({id: "javascript.0.scenes.general.lightAllOn", val: true}, function (obj) {
  enum_sonoff.each(function (id, i) {
        setState(id, true);
      });
  enum_tradfri.each(function (id, i) {
        setState(id, true);
      });
  console.log('Gesamtes Licht an');
});

funktioniert (aus geht natürlich auch).

Dann erklärt sich auch warum ich den sonoff-Adapter abgeschossen habe mit unpassenden ID's. Die SetState's müssen also definitiv getrennt für unterschiedliche Endungen der Mitglieder gesetzt werden.

Vielen Dank an alle, das Problem ist somit gelöst.

tempestas

Setz doch Mal setState("true") anstelle von setState(true).

In der Fehlermeldung steht doch, dass der sonoff einen String erwartet

paul53

@NyanCat sagte in Alle Mitglieder einer Aufzählung schalten:

da sind sie alle drin

Die IDs enden nicht mit .STATE, sondern mit .POWER und .state.

const Sonoff = $('state[id=*.POWER](functions="light")');
const Tadfri = $('state[id=*.state](functions="light")');

NyanCat

@tempestas said in Alle Mitglieder einer Aufzählung schalten:

Setz doch Mal setState("true") anstelle von setState(true).

In der Fehlermeldung steht doch, dass der sonoff einen String erwartet

Da passiert leider weiterhin nichts. Ich hab ja so ein bisschen das Gefühl, dass es etwas damit zu tun hat, dass ein paar Mitglieder mit .POWER enden und andere mit .state ... weil ich hatte mal eine Version, da gingen immerhin alle .POWER an, und zwar ohne Fehler. Aber die Tradfris halt nicht. (ich kann mich nur leider nicht mehr erinnern, was ich dafür genau geändert hatte)

// Button ALLE Mitglieder der Aufzählung function.light an
on({id: "javascript.0.scenes.general.lightAllOn", val: true}, function (obj) {
  $('state(functions=light)').setState(true);
  console.log('Gesamtes Licht an');
});
// Button ALLE Mitglieder der Aufzählung function.light aus
on({id: "javascript.0.scenes.general.lightAllOff", val: true}, function (obj) {
  $('state(functions=light)').setState(false);
  console.log('Gesamtes Licht aus');
});

Das ist die Version, bei der ich mir den sonoff-Adapter mit den bekannten Fehlermeldungen abschieße. Es funktioniert zwar (also es wird einmal alles an- oder ausgeschaltet), aber danach ist der sonoff down.

NyanCat

@paul53 said in Alle Mitglieder einer Aufzählung schalten:

@NyanCat sagte in Alle Mitglieder einer Aufzählung schalten:

da sind sie alle drin

Die IDs enden nicht mit .STATE, sondern mit .POWER und .state.
const Sonoff = $('state[id=*.POWER](functions="light")');
const Tadfri = $('state[id=*.state](functions="light")');

Ja in die Richtung hatte ich auch schon was probiert, probiere es aber nochmal. Kann man eigentlich auch irgendwie einer Variabel die ID's von beiden zuweisen, so dass ich keine 2 .each machen muss?

schon mal vielen Dank

paul53

@NyanCat sagte in Alle Mitglieder einer Aufzählung schalten:

Kann man eigentlich auch irgendwie einer Variabel die ID's von beiden zuweisen, so dass ich keine 2 .each machen muss?

Nein. Teste erst einmal, ob alle passenden IDs gefunden werden.

const Sonoff = $('state[id=*.POWER](functions="light")');
const Tadfri = $('state[id=*.state](functions="light")');

Sonoff.each(function(id, i) {
   log(id + ': ' + getState(id).val);
});
Tadfri.each(function(id, i) {
   log(id + ': ' + getState(id).val);
});

BBTown

@NyanCat wie @paul53 bereits geschrieben hat:
". STATE" ist nicht das gleiche wie ".state"

NyanCat

const enum_sonoff = $('state[id=*.POWER](functions="light")');
const enum_tradfri = $('state[id=*.state](functions="light")');

on({id: "javascript.0.scenes.general.lightAllOn", val: true}, function (obj) {
  enum_sonoff.each(function (id, i) {
        setState(id, true);
      });
  enum_tradfri.each(function (id, i) {
        setState(id, true);
      });
  console.log('Gesamtes Licht an');
});

funktioniert (aus geht natürlich auch).

Dann erklärt sich auch warum ich den sonoff-Adapter abgeschossen habe mit unpassenden ID's. Die SetState's müssen also definitiv getrennt für unterschiedliche Endungen der Mitglieder gesetzt werden.

Vielen Dank an alle, das Problem ist somit gelöst.

smartboart

Hallo,
beim folgenden Selektor werden auch Geräte aufgeführt welche nicht dem Gewerk Licht zugeordnet sind.
var cacheSelectorState = $('channel[state.id=hm-rpc.0*.STATE](functions= Licht)');

zum Beispiel dieser hier... also Steckdose wird mit verarbeitet..

Fehler im Javascriptadapter ?

paul53

@smartboart sagte:

$('channel[state.id=hm-rpc.0*.STATE](functions= Licht)');

Die Gewerke sind nicht Kanälen, sondern Datenpunkten zugeordnet. Versuche mal

$('state[id=hm-rpc.0.*.STATE](functions=Licht)');

In der ID enum.functions.Licht ist "Licht" groß geschrieben ?

smartboart

Hatte ich vorher.. Gleiches Verhalten!!!

smartboart

@paul53 ja ist gross geschrieben..hab schon alle Varianten durch und getestet...
Screenshot_20200213_210145_org.mozilla.firefox.jpg

paul53

@smartboart sagte:

hab schon alle Varianten durch und getestet...

Welche Versionen haben ?

js-controller
Javascript-Adapter

smartboart

Hallo mein repo ist auf latest...

Js-controller 2.2.9

Javascript 4.4.2

Ist aber schon länger so.. Habe jetzt erst nach unzähligen lösungsversuchen das Thema hier angesprochen..

smartboart

@paul53
Hi Paul53,
konntest du das nachvollziehen?
vlt. kannst du das mit einem Homematic 2 Kanalaktor nachvollziehen.
Das 1. Relais einer anderen Funktion zuordnen als das 2.
Damit müsste es gehen.

Hier die Ausgabe zu meinem Problem mit folgendem script.....

const pruef = $('state[state.id=hm-rpc.0*.STATE](functions= Licht)'); //Gewerk Licht
pruef.each(function(id, i) {
log(id);
});

dabei ist zu sagen, dass die HM Geräte mit dem 2. Relais nicht dem Gerwerk Licht zugeordnet sind, sondern Steckose...Dennoch tauchen sie hier auf...

Wenn ich wüsste im welchen Bereich der fehler liegt, würde ich auch ein issue auf github aufmachen..Weiß nur nicht ob es am javascript Adapter oder dem js-controller liegt...

paul53

@smartboart sagte:

konntest du das nachvollziehen?

Nein, bei mir funktionieren die Selektoren wie erwartet. Versuche mal

const pruef = $('*.STATE(functions=Licht)'); //Gewerk Licht

ohne Leerzeichen vor Licht.

smartboart

Ja das Leerzeichen macht keinen Unterschied...wenn es bei dir funktioniert, hast du Objekt und state als redis oder file? Vlt. Wird das nicht sauber überschrieben oder gelöscht wenn die Zuordnung der functions vorher schon mal anders war?

paul53

@smartboart sagte:

hast du Objekt und state als redis oder file?

Beides als file. Wobei das keine Rolle spielen sollte, da js-Controller und Javascript-Adapter alle Objekte und Zustände puffern. Deshalb ist nach Änderungen an den enums per Admin ein Restart der Javascript-Instanz hilfreich.

smartboart

Werde mal die enums der betroffenen states löschen, neustarten, neu zuordnen und wieder neustarten.... Habe das aber denke ich schon gemacht... Aufgefallen ist mir das Verhalten aber erst nachdem ich auf redis umgestellt habe.. Deshalb die Frage...

smartboart

@paul53 also habe weiter getestet...da ist definitiv ein bug...
habe jetzt 2 objekte mit jeweils den selben Informationsinhalt erstellt. Der Unterschied ist nur die Instanz...ein objekt instanz 0 das andere instanz 1.
0 ist einer enum zugeordnet , 1 nicht....Auch hier werden beide mit der selector funktion eingesammelt.

NEWS

SOLVED Alle Mitglieder einer Aufzählung schalten (gelöst)

Support us

884
Online

31.8k
Users

79.9k
Topics

1.3m
Posts

NEWS

SOLVED Alle Mitglieder einer Aufzählung schalten (gelöst)

Support us

884Online

31.8kUsers

79.9kTopics

1.3mPosts

884
Online

31.8k
Users

79.9k
Topics

1.3m
Posts