UPDATE 31.10.: Amazon Alexa - ioBroker Skill läuft aus ?

[sigi234]

@paul53 sagte in [Vorlage] Alias per Skript erzeugen:

@sigi234 sagte:

warum sag er immer Instanz ist deaktiviert?

Wer sagt das ?

Hier, und sollte nicht ein DP angelegt werden? Wo?

Edit:
geht jetzt, Admin neu gestartet. JS neu gestartet. Dauerte eine Weile. :grinning:

[CruziX]

@paul53 Ich bekomme das hinzufügen der Räume und Funktionen nicht hin.
Habe mir aus deinem Skript ein eingenes geschrieben, welches auch direkt die Räume und Funktionen mit Anlegt, aber das funktioniert nicht.

4 States sollen in einen Raum und es funktioniert immer nur einer.
Wenn ich das gesamte Skript erneut ausführe wird dann der nächste dem Raum hinzugefügt und dann beim Neustart wieder der nächste..
Hab gedacht es würde daran liegen, dass der Raum zu schnell erweitert wird und habe einen Timeout eingebaut, aber gleiches Problem.
Es scheint auch nicht an der Geschwindigkeit zu liegen, wenn ich das Skript ohne Timeout schnell nacheinander neustarte wird immer wieder ein State hinzugefügt.
Weiß jemand woran das liegen kann?

[paul53]

@CruziX sagte:

4 States sollen in einen Raum und es funktioniert immer nur einer.

Das Script ist so ausgelegt, dass bei Start nur ein Alias-Datenpunkt erzeugt wird (man kann jeweils nur eine Original- und eine Alias-ID angeben). Folglich wird die Raum- und Gewerkezuordnung nur für diesen Datenpunkt erfolgen.

[CruziX]

@paul53 ich weiß, dass das Skript nur einen DP anlegt.

Ich habe ein Skript indem ich mit der Funktion createAlias, also dein Skript als Funktion, alle meine Alias anlege sofern diese nicht vorhanden sind.
Ich erzeuge also sehr viele in einem Skript.

Probleme gibt es nur beim erweitern der Räume und Funktionen.

Bei der Erzeugung von 4 Alias übergebe ich den Raum und die Funktion zum Testen, aber nur der erste, der nicht im Raum und der Funktion ist, wird erzeugt. Bei jedem Start meines Skriptes wird nur ein Wert dem Raum und der Funktion hinzugefügt.

[Garfonso]

@CruziX
Wie machst du das denn mit den Räumen und Funktionen? Da musst du ja ein Objekt modifizieren. Wenn du das jetzt für jeden Datenpunkt einzeln machst, dann überschreibst du die Änderungen ja gegenseitig und daher kommt dann nur einer an. Du müsstest entweder alle ids sammeln und am Stück in die Räume/Funktionen einfügen, so dass du die Objekte jeweils nur einmal mit setObject schreibst oder du musst in setObject einen Callback mit übergeben und dann darauf achten, dass andere setObject Aufrufe für den Raum/die Funktion erst aufgerufen werden, wenn der callback fertig ist.

(Wobei ich gerade nicht geprüft habe, ob setObject oder extendObject einen callback übergeben bekommen kann).

[CruziX]

@Garfonso ich hab mal ein paar Screenshots gemacht:

Spoiler

Das Skript von Paul befindet sich etw angepasst als Function in meinem Sktipt

Nach der Erzeugung des Alias Objekts kommt folgender Teil und soll den Raum und die Funktion zuweisen.

Die Aufrufe dieser createAlias Function sehen so aus:

Ich denke auch nicht, dass ich in meiner Variante den Raum und die Funktion immer wieder überschreibe, da ich mir ja immer das members object holen, erweiter und setze.
Es sei denn das führt doch irgendwie dazu :sweat_smile:

Edit:
Du hattest doch recht.. ich frage mich wieso das so ist, aber ich erweitere die Räume und Funktionen nun am Ende des Skripts und das scheint zu funktionieren.

[CruziX]

So, ich hatte ja schon erwähnt, dass meine Version des Skripts etwas mehr kann.

Möchte euch dieses zur Verfügung stellen.
Dazu die Anmerkung, dass ich noch ziemlicher Anfänger in JavaScript bin :sweat_smile:

Was dieses Skript kann:

• Erzeugung von Alias Datenpunkten
• Die Werte wie Quelle des Alias, Ziel Datenpunkt des Alias, Raum, Funktion etc. können alle der Funktion selber übergeben werden
• Der Präfix alias.0. kann hierbei weggelassen werden und wird automatisch generiert
• Um Räume und Funktionen dem Alias mitzugeben sind diese beim Aufrufen der Methode mitzugeben und am Ende des Skripts muss die Funktion assignEnums() einmalig aufgerufen werden
• Die Knoten Datenpunke (Im Objektexplorer als Ordner dargestellt) über den eigentlichen Alias Werten können mit den Variablen bCreateAliasPath und bConvertExistingPath so erstell/umgewandelt werden, dass man ihnen Räume und Funktionen zuordnen kann

bCreateAliasPath
=> Beim Anlegen eines Alias, werden die Knoten Datenpunkte die noch nicht existieren, direkt so erstellt, dass man ihnen Räume und Funktionen zuweisen kann.

bConvertExistingPath (Erfordert das bCreateAliasPath auf true gesetzt ist)
=> Auch schon existierende Knoten Datenpunkte werden konvertiert um Räume und Funktionen zuweisen zu können.

UPDATE
Die Methode assignEnum muss nicht mehr am Ende des Skripts ausgeführt werden, sie wird mittels Timeout automatisch ausgelöst

https://github.com/xCruziX/ioBroker-Creating-Alias

[dslraser]

@CruziX
Kannst Du mal ein Beispiel mit mehreren original DP und Erstellung der alias DP in Deinem Script zeigen ? Ich weiß sonst nicht wie man das Script anwendet. (Ich bin sonst eher in Blockly unterwegs)

[CruziX]

@dslraser klar, mach ich später :)

[CruziX]

Hab nochmals das Skript erweitert:
https://forum.iobroker.net/topic/27295/vorlage-alias-per-skript-erzeugen/70

@dslraser hier mal ein Beispiel:
In dem Skript lege ich einen Alias unter "alias.0.Test.Licht.An" an, der auf den Shelly verweist.

Spoiler

// typeAlias = 'boolean'; // oder 'number'
// read = "val == 'Ein' ? true : false"; // Erkennung "Aus" --> false erfolgt automatisch  
// write = "val ? 'Ein' : 'Aus'";
// nameAlias = 'Licht Haustür';
// role = 'switch';
// desc = 'per Script erstellt';
// min = 0; // nur Zahlen
// max = 100; // nur Zahlen
// unit = '%'; // nur für Zahlen
// states = {0: 'Aus', 1: 'Auto', 2: 'Ein'}; // Zahlen (Multistate) oder Logikwert (z.B. Aus/Ein)

let bCreateAliasPath = false;
let bConvertExistingPath = false;
let arEnum = [];
let arId = [];
let timeoutAssignEnum;
function createAlias(idSrc, idDst,raum, gewerk,typeAlias, read, write, nameAlias, role, desc, min, max, unit, states) {
   if(!idDst.includes('alias.0.'))
       idDst = 'alias.0.' + idDst;
   if(!existsObject(idSrc))
   {
       log('Source-Id ' + idSrc +' does not exists.','warn');
       return;
   }
   
   // Create the object Path for alias id, 
   // so you can assign rooms and function to the parents
   var createAliasPath = (id) => {
       let mergedId = 'alias.0';
       id = id.replace(mergedId + '.', '');
       let split = id.split('.');
       let bCreated = false;
       for(let i=0;i<split.length-1;i++){
           mergedId += '.' + split[i];
           if(!existsObject(mergedId) || bConvertExistingPath){ // not exists
               bCreated = true;
               if(bConvertExistingPath)
                   log('Converting existing Path: ' + mergedId);
                   
               let obj;
               if(existsObject(mergedId))
                   obj = getObject(mergedId);
               else
                   obj = {};

               if(obj.type == undefined || obj.type != 'meta')
                   obj.type = 'meta';
               if(obj.common == undefined || obj.common != {})
                   obj.common = {};
               if(obj.common.type == undefined || obj.common.type != 'meta.folder')
                   obj.common.type = 'meta.folder';
               if(obj.common.desc == undefined || obj.common.desc != 'createAliasPath')
                   obj.common.desc = 'createAliasPath';
               if(obj.common.def == undefined || obj.common.def != false)
                   obj.common.def = false;
               if(obj.native == undefined || obj.native != {})
                   obj.native = {};
               setObject(mergedId, obj);
           }
       }
       if(bCreated)
           log('Create Alias-Path ' + id);
   }
   
   if(bCreateAliasPath)
       createAliasPath(idDst);
   
   // Create alias object
   if(!existsObject(idDst)){
       let obj = {};
       obj.type = 'state';
       obj.common = getObject(idSrc).common;
       obj.common.alias = {};
       obj.common.alias.id = idSrc;
       if(typeAlias !== undefined) 
           obj.common.type = typeAlias;
       if(obj.common.read !== undefined) 
           obj.common.alias.read = read;
       if(obj.common.write !== undefined) 
           obj.common.alias.write = write;
       if(nameAlias !== undefined) 
           obj.common.name = nameAlias;
       if(role !== undefined) 
           obj.common.role = role;
       if(desc !== undefined) 
           obj.common.desc = desc;
       if(min !== undefined) 
           obj.common.min = min;
       if(max !== undefined) 
           obj.common.max = max;
       if(unit !== undefined) 
           obj.common.unit = unit;
       if(states !== undefined) 
           obj.common.states = states;

       obj.native = {};
       obj.common.custom = []; // Damit die Zuordnung zu iQontrol, Sql etc. nicht übernommen wird
       log('Created Alias-State ' + idDst);
       setObject(idDst, obj);
   }
   
    
   var attach = (id, enu,value) => {
       value = value.toLowerCase();
       enu = enu.toLowerCase();
       if(id.length == 0 || !existsObject(id)){
           log(String(id.length == 0));
           log(String(!existsObject(id)));
           return;
       }
       let sEnuId = 'enum.' + enu + '.' + value;
       if(enu.length > 0 && existsObject(sEnuId)) 
       {
           let obj = getObject(sEnuId)
           let members = obj.common.members;
           if(!members.includes(id)){
               arEnum.push(sEnuId);
               arId.push(id);
           }
       }
   }

   let bRoom = raum !== undefined && raum.length > 0;
   let bGewerk = gewerk !== undefined && gewerk.length > 0;

   if(bRoom)
       attach(idDst,'rooms',raum);
   if(bGewerk)
       attach(idDst,'functions',gewerk);
   if(bRoom || bGewerk){
       if(timeoutAssignEnum){
           clearTimeout(timeoutAssignEnum);
           timeoutAssignEnum = null;
       }
       timeoutAssignEnum = setTimeout(assignEnums,1000);
   }
}

function assignEnums(){
  if(arEnum.length != arId.length){
       log('Arrays have different size','warn');
       return;
  }
  let mapEnumId = new Map();
  
  for(let i=0;i < arEnum.length; i++){
      let enu = arEnum[i];
      let id = arId[i];
   
      let obj = getObject(enu)
      let members;
      if(!mapEnumId.has(enu)){
          members = obj.common.members;
          mapEnumId.set(enu,members);
      }
      else
          members = mapEnumId.get(enu);
      
      if(!members.includes(id)){
           log("Adding " + id + " to " + enu);
           members.push(id);
       }      
  }
   
   function setMembers(members,enu,map){
       let obj = getObject(enu);
       obj.common.members = members;
       setObject(enu,obj);
   }
   mapEnumId.forEach(setMembers);
}

let idSource = 'shelly.0.SHSW-1#68BAC3#1.Relay0.Switch';
let idTargetAlias = 'Test.Licht.An';
let room = 'yannick';
let funct = 'licht';

createAlias(idSource,idTargetAlias,room,funct);

[dslraser]

@CruziX
Danke Dir, aber mich interessiert es wie mehrere gleichzeitig erstellt werden können, also nicht nur ein alias. (ein Beispiel) Jedenfalls hatte ich Dich so verstanden das auch mehrere auf einmal gehen ?

[CruziX]

@dslraser Vielleicht verstehe ich dich auch falsch..
Du kannst doch die function createAlias in einem Skript mehrfach ausführen?

In meinem Skript "Erzeuge_Alias", rufe ich sehr oft die Methode createAlias auf um alle meine Alias Werte zu erzeugen.

Spoiler

// Erzeugung des ersten Alias
let idSource = 'shelly.0.SHSW-1#68BAC3#1.Relay0.Switch';
let idTargetAlias = 'Test.Licht.Tischlicht.An';
let room = 'yannick';
let funct = 'licht';

createAlias(idSource,idTargetAlias,room,funct);

// Anpasssen der IDs und Erzeugung des zweiten
idSource = 'Sonoff.0.Deckenfluter.Power';
idTargetAlias = 'Test.Licht.Deckenfluter.An';

createAlias(idSource,idTargetAlias,room,funct);

[dslraser]

@CruziX
Jetzt habe ich es verstanden und werde es ausprobieren, danke.
Ich nutze z.B. das Script von @paul53 bzw. die Erweiterung von @Mic für die Erstellung von Datenpunkten in 0_userdata.0 oder javascript.x
Da sieht es etwas anders aus.

Das hier

[CruziX]

@dslraser Ja, meine Version lässt sich mit den entsprechenden Parametern einfach ausführen :)

Gut, das Skript von Mic hat ja auch einen anderen Sinn, hier werden ja die Datenpunkte mit den eigenen Eigenschaften erstellt, das Skript für Alias Werte kopiert diese ja weitesgehend ^^

Also ich fand es so wie ich es habe etw. praktischer und da ich am Wochenende die Zeit gefunden habe, dass Problem mit den Räumen und Funktionen zu lösen, was mich schon länger gequält hat, wollte ich das euch mal zur Verfügung stellen :)

[dslraser]

@CruziX

Ja, das ist auch prima und auch das Du es teilst. Ich möchte mir halt, so wie in dem Script von Mic meine eigene "Sammlung" erstellen, wo ich in einem einzigen Script alle meine eigenen Datenpunkte und nun auch mit Deinem Script alle meine alias erstellen könnte.

[CruziX]

@dslraser Jep, genau so ein Skript habe ich auch für die Alias Werte, finde ich auch sehr praktisch :)

[CruziX]

@dslraser was mir gerade noch einfällt, was bei mir noch anders als im Skript von paul ist.
Ich habe die Logmeldungen verändert, bei mir werden nur welche ausgegeben, wenn ein Fehler erkannt wird und wenn tatsächlich Alias Werte angelegt werden.
Falls die Alias Werte schon vorhanden sind, bekommt man keine Meldung, so kann das Skript halt ausgeführt werden, ohne, dass das Log vollgemüllt wird.

[CruziX]

@dslraser Wenn das Skript bei dir funktioniert könnte es ja vielleicht auch @paul53 im Ursprungsbeitrag mit ergänzen, so sehen es andere besser :)

[dslraser]

@CruziX
ich komme heute erst sehr spät dazu es zu probieren. Ich gebe aber Rückmeldung.

[Garfonso]

@CruziX
Doch, du überschreibst sie. Das liegt an der Asynchronität von setObject. Dein Code wird quasi "direkt" aus geführt, inklusive des Lesens der Members (was daran liegt, dass der JavaScript Adapter den Wert für dich zwischenspeichert). Nur das "setObject" passiert später. Du liest also x-mal denselben Members-Wert, fügst immer nur einen dazu und überschreibst deine anderen Änderungen mit jedem setObject. :-)

Aber das hast du ja dann auch gemerkt und gelöst. :-) Da muss man bei Javascript/NodeJS immer etwas extra aufpassen.