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

[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

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

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.

[CruziX]

@Garfonso Ok, danke für die Erklärung, wieder was gelernt

[dslraser]

@CruziX
wenn ich mehrere alias auf einmal erzeuge, wie erstelle ich dann für jeden alias einen eigenen neuen Namen, wenn ich den Ursprung nicht 1:1 kopieren will.
Wenn ich nur die function ohne Raum (ich nutze keine Räume) erstellen will, läuft das Script zwar durch (dafür habe ich let rom auskommentiert und aus dem Funktionsaufruf entfernt, aber die erstellten alias befinden sich dann nicht in der Aufzählung, was mache ich falsch ?

[paul53]

@dslraser
Funktion lautet:

function createAlias(idSrc, idDst,raum, gewerk,typeAlias, read, write, nameAlias, role, desc, min, max, unit, states) {

Wenn nur der Name geändert werden soll, muss die Funktion wohl so aufgerufen werden:

createAlias(idSrc, idDst, undefined, undefined, undefined, undefined, undefined, 'neuer Name');

ist also nur etwas für "Fortgeschrittene".