iobroker.alias-workbench 0.9.0 → 0.9.7

package/admin/js/mqtt.js

@@ -38,10 +38,28 @@ export function mqttEinstellung(id) {
   if (!o || !o.common) { return null; }
   var cu = o.common.custom || {};
   var k = null;
-  Object.keys(cu).forEach(function (x) { if (!k && /^mqtt/.test(x)) { k = x; } });
+  /* Drei Bedingungen, nicht eine.
+
+     `cu[x]` kann `null` sein — ioBroker raeumt den Eintrag nicht weg,
+     wenn eine Instanz verschwindet. Der Zugriff auf `.topic` riss dann
+     die ganze Ergebnisansicht mit (gemessen 09.09.2026).
+
+     Und `enabled` kam hier gar nicht vor: `{ enabled: false,
+     publish: true }` galt als sendefaehig, obwohl die Instanz diesen
+     Punkt nicht bedient. Fehlt das Feld, gilt der Eintrag als an — so
+     legt der Admin es an, wenn man die Verknuepfung setzt. */
+  Object.keys(cu).forEach(function (x) {
+    if (!k && /^mqtt/.test(x) && cu[x] && cu[x].enabled !== false) { k = x; }
+  });
   if (!k) {
-    /* Manche Objekte tragen das Thema nur in native. */
-    if (o.native && o.native.topic) { return { instanz: null, topic: o.native.topic, publish: false }; }
+    /* Manche Objekte tragen das Thema nur in native — ioBroker.mqtt und
+       sonoff tun das. Fuer sie ist `publish` schlicht UNBEKANNT, nicht
+       nein: Frueher stand hier `false`, die Pruefung meldete „kann nicht
+       senden", der Chip stand rot, und der Knopf „senden erlauben"
+       schrieb das Objekt unveraendert zurueck — eine Meldung, die sich
+       nicht abstellen liess (gemessen 09.09.2026). `null` heisst: wir
+       wissen es nicht, also nicht behaupten. */
+    if (o.native && o.native.topic) { return { instanz: null, topic: o.native.topic, publish: null }; }
     return null;
   }
   return { instanz: k, topic: cu[k].topic || (o.native || {}).topic || '', publish: !!cu[k].publish,
@@ -110,6 +128,12 @@ export function tasmotaBefehle(kanal) {
   var liste = [];
   Object.keys(aus).forEach(function (f) {
     if (weg.indexOf(f) > -1) { return; }
+    /* Aus dem Schluessel wird eine Kennung UND ein MQTT-Thema. Ein `#`
+       oder `+` darin ergaebe ein Wildcard-Abo, ein Punkt eine
+       zusaetzliche Ebene im Objektbaum. Der Inhalt kommt vom Broker,
+       also von aussen — erreichbar ueber jeden fremden Publisher auf
+       demselben Broker. Tasmota-Befehle heissen ohnehin alle so. */
+    if (!/^[A-Za-z][A-Za-z0-9_]*$/.test(f)) { return; }
     if (aus[f] !== null && typeof aus[f] === 'object' && !Array.isArray(aus[f])) { return; }
     liste.push({ name: f, wert: aus[f] });
   });
@@ -143,13 +167,21 @@ export function befehlsWissen(name) {
    naechste Befehl die Antwort. Gemessen: nachfragen → „eingeschaltet",
    einmal schalten → wieder „unbekannt", obwohl sich am Geraet nichts
    geaendert hatte. Der Befund war praktisch nie stabil zu sehen. */
-export function sofortRueckmeldung(kanal) {
+export function sofortRueckmeldung(kanal, merken) {
   var id = hatPunkt(kanal, 'stat.RESULT');
   if (id) {
     var j = jsonVon(id);
     if (j && j.SetOption59 !== undefined) {
       var w = String(j.SetOption59).toUpperCase() === 'ON';
-      merkeSo59(kanal, w);
+      /* Gemerkt wird nur nach einer Abfrage, nicht beim Zeichnen.
+
+         Vorher schrieb diese Funktion aus dem Zeichenpfad heraus ein
+         Objekt — ein Waechter verhinderte nur das Mehrfachschreiben,
+         der Fehler wurde verschluckt, und der Wert stand lokal schon.
+         Das widerspricht der Zusage, dass die Pruefungen nichts
+         anfassen: Wer die rechte Seite ansieht, aendert nichts.
+         `so59()` ruft mit `merken = true`, alle anderen lesen nur. */
+      if (merken) { merkeSo59(kanal, w); }
       return w;
     }
   }
@@ -203,7 +235,10 @@ export function mqttLage(kanal) {
       var id = hatPunkt(kanal, 'cmnd.' + x.name);
       if (!id) { fehlen.push(x); return; }
       var m = mqttEinstellung(id);
-      if (m && !m.publish) { stumm.push({ name: x.name, id: id }); } else { da.push(x.name); }
+      /* `publish === null` heisst unbekannt (Thema nur in `native`) —
+         so ein Punkt kommt zu den vorhandenen, nicht zu den stummen.
+         „Senden erlauben" haette an ihm ohnehin nichts zu schreiben. */
+      if (m && m.publish === false) { stumm.push({ name: x.name, id: id }); } else { da.push(x.name); }
     });
   }
   /* Auch cmnd-Punkte, die es schon gibt, ohne dass sie im Zustand
@@ -215,7 +250,7 @@ export function mqttLage(kanal) {
     if (stumm.some(function (x) { return x.name === name; })) { return; }
     if (da.indexOf(name) > -1) { return; }
     var m = mqttEinstellung(id);
-    if (m && !m.publish) { stumm.push({ name: name, id: id }); }
+    if (m && m.publish === false) { stumm.push({ name: name, id: id }); }
   });
   return { geraet: g, befehle: b, fehlen: fehlen, stumm: stumm, da: da,
            sofort: sofortRueckmeldung(kanal) };
@@ -509,7 +544,7 @@ function so59(kanal, knopf, wert) {
             setTimeout(function () {
               holeZweig(kanal + '.', function () {
                 delete so59Laeuft[kanal];
-                so59Stumm[kanal] = (sofortRueckmeldung(kanal) === null);
+                so59Stumm[kanal] = (sofortRueckmeldung(kanal, true) === null);
                 zeichneErgebnis();
               });
             }, 4000);
@@ -556,6 +591,18 @@ function mqttFragen(kanal, _knopf) {
           /* Status 5 gleich mit: dort steht die IP, und die fehlt an
              jedem Geraet, das seit dem Einbinden nicht neu gestartet
              ist — tele/INFO2 kommt nur beim Start. */
+          /* Wie alt sind die Quellen JETZT?
+
+             Vorher wurde nur die Merkstelle geleert. Ein `stat.RESULT`,
+             das retained von vorgestern dasteht, reichte danach fuer
+             die gruene Rueckmeldung — auch wenn das Geraet aus ist und
+             gar nicht antwortet (gemessen 09.09.2026). Gewertet wird
+             jetzt nur, was nach dem Senden hereinkam. */
+          var vorTs = {};
+          ['stat.STATUS11', 'tele.STATE', 'stat.RESULT'].forEach(function (q) {
+            var qid = hatPunkt(kanal, q);
+            if (qid && S.werte[qid]) { vorTs[q] = S.werte[qid].ts || 0; }
+          });
           socket.emit('setState', id, { val: '5', ack: false }, function () {});
           socket.emit('setState', id, { val: '11', ack: false }, function () {
             setTimeout(function () {
@@ -569,7 +616,12 @@ function mqttFragen(kanal, _knopf) {
               holeZweig(kanal + '.', function () {
                 delete abfrageLaeuft[kanal];
                 var bb = tasmotaBefehle(kanal);
-                mqttAbfrage[kanal] = (bb && bb.befehle.length) ? bb.befehle.length : 0;
+                /* Nur wenn die Quelle, aus der die Liste stammt, seit
+                   dem Senden frisch geworden ist. */
+                var qid = bb && hatPunkt(kanal, bb.woher);
+                var jetzt = (qid && S.werte[qid]) ? (S.werte[qid].ts || 0) : 0;
+                var frisch = bb && jetzt > (vorTs[bb.woher] || 0);
+                mqttAbfrage[kanal] = (frisch && bb.befehle.length) ? bb.befehle.length : 0;
                 zeichneErgebnis();
               });
             }, 5000);
@@ -606,7 +658,18 @@ function mqttCustom(g, thema) {
 export function mqttEinzelnStill(kanal, name, fertig) {
   var bau = mqttPunktBauen(kanal, name, 'neu');
   if (!bau) { return fertig && fertig(); }
-  socket.emit('setObject', bau.id, bau.obj, function () {
+  socket.emit('setObject', bau.id, bau.obj, function (err) {
+    /* Ein abgelehntes Schreiben ist kein Erfolg.
+
+       Der Rueckruf nahm `err` frueher nicht einmal entgegen und trug das
+       Objekt trotzdem in den eigenen Bestand ein. Danach hielt
+       `pruefeSchreiben` den Sendepunkt fuer vorhanden, die Sperre griff
+       nicht mehr, und „Jetzt anlegen" schrieb `alias.id.write` auf eine
+       Kennung, die es in der Datenbank nie gab — genau der Fehler, den
+       der js-controller sich dauerhaft merkt und nur durch Loeschen und
+       Neuanlegen vergisst. Der Zwilling `mqttEinzeln` prueft ihn seit
+       jeher richtig (gemessen 09.09.2026). */
+    if (err) { return fertig && fertig(err); }
     uebernimmObjekt(bau.id, bau.obj);
     if (fertig) { fertig(); }
   });
@@ -622,7 +685,7 @@ function mqttPunktBauen(kanal, name, was) {
     if (!alt0) { return null; }
     obj = JSON.parse(JSON.stringify(alt0));
     var cu0 = obj.common.custom || {};
-    Object.keys(cu0).forEach(function (i) { if (/^mqtt/.test(i)) { cu0[i].publish = true; } });
+    Object.keys(cu0).forEach(function (i) { if (/^mqtt/.test(i) && cu0[i]) { cu0[i].publish = true; } });
     obj.common.custom = cu0;
   } else {
     var w0 = befehlsWissen(name) || {};
@@ -656,7 +719,7 @@ export function mqttEinzeln(kanal, name, was, knopf) {
     if (!alt) { return; }
     obj = JSON.parse(JSON.stringify(alt));
     var cu = obj.common.custom || {};
-    Object.keys(cu).forEach(function (i) { if (/^mqtt/.test(i)) { cu[i].publish = true; } });
+    Object.keys(cu).forEach(function (i) { if (/^mqtt/.test(i) && cu[i]) { cu[i].publish = true; } });
     obj.common.custom = cu;
   } else {
     var w = befehlsWissen(name) || {};
@@ -846,7 +909,7 @@ function mqttZuSchreiben() {
     if (!alt) { return; }
     var kopie = JSON.parse(JSON.stringify(alt));
     var cu = kopie.common.custom || {};
-    Object.keys(cu).forEach(function (i) { if (/^mqtt/.test(i)) { cu[i].publish = true; } });
+    Object.keys(cu).forEach(function (i) { if (/^mqtt/.test(i) && cu[i]) { cu[i].publish = true; } });
     kopie.common.custom = cu;
     raus.push({ id: x.id, obj: kopie, neu: false });
   });

package/admin/js/musterwahl.js

@@ -112,6 +112,16 @@ export function baueMusterwahl(e, haupt, pflichtFehlt) {
   var markeCache = {};
   function bewerte(m) {
     if (markeCache[m] !== undefined) { return markeCache[m]; }
+    /* Die angebotene Liste ist gefiltert — der gewaehlte Typ nicht:
+       `mBeschriften` ruft `markeFuer(e.want)` direkt, und `e.want` kommt
+       ungeprueft aus `v.geraetetyp`. Eine eingelesene Vorlage mit dem
+       Musternamen eines neueren type-detector liess `musterVon(m).states`
+       werfen, und der Entwurf wurde gar nicht erst gezeichnet — die
+       rechte Seite blieb leer (gemessen 09.09.2026). */
+    if (!musterVon(m)) {
+      markeCache[m] = { mark: tr('pattern.doesNotFit'), passt: false, belegt: 0 };
+      return markeCache[m];
+    }
     /* Die Vorschau muss vorhersagen, was das Auswaehlen wirklich tut.
        Beim Vorschlag werden Lampe und Steckdose samt Rollen neu gebaut -
        also auch hier gegen den umgebauten Entwurf pruefen. Sonst stuende

package/admin/js/objekte.js

@@ -81,6 +81,19 @@ export function nachziehenErledigt() {
   nachziehBaum = false;
 }
 
+/* Nach einem Abbruch sind beim Server alle Abos weg.
+
+   `subscribeObjects` und `subscribe` gelten nur fuer die Sitzung, die
+   sie gestellt hat — der WebSocket-Shim des Admin erneuert von sich aus
+   nichts (das Wort `subscribe` kommt in seinem Quelltext kein einziges
+   Mal vor). Der `objectChange`-Horcher wird hier ausdruecklich NICHT
+   noch einmal angemeldet: der haengt am Socket-Objekt und hat den
+   Abbruch ueberlebt. Nur die Anmeldung beim Server fehlt. */
+export function abosErneuern() {
+  if (horcht) { socket.emit('subscribeObjects', '*'); }
+  (S.abo || []).forEach(function (m) { socket.emit('subscribe', m); });
+}
+
 export function horcheAufObjekte() {
   if (horcht) { return; }
   horcht = true;
@@ -95,11 +108,20 @@ export function horcheAufObjekte() {
     var kern = function (o) {
       return o ? JSON.stringify([o.type, o.common, o.native]) : null;
     };
-    var vorher = kern(S.objects[id]);
     var nachher = kern(obj);
-    if (vorher === nachher) { return; }
 
-    /* Aufzaehlungen gehen einen eigenen Weg.
+    /* Aufzaehlungen zuerst — und zwar VOR dem Vergleich mit `S.objects`.
+
+       Der Vergleich unten fragt `S.objects[id]`, und dort liegen
+       Aufzaehlungen nie. Beim Loeschen einer Aufzaehlung sind alter und
+       neuer Wert deshalb beide `null`, der Frueheinstieg griff, und
+       `delete enums[id]` wurde nie erreicht. Die geloeschte Aufzaehlung
+       blieb im Vorrat stehen, und der naechste Alias mit diesem Raum
+       kopierte sie samt aller alten Mitglieder zurueck an ihren Platz.
+       Gemessen 09.09.2026 am Testsystem: `enum.rooms.WBP7` war am System
+       geloescht und waere mit zwei fremden Mitgliedern wieder entstanden.
+
+       Aufzaehlungen gehen einen eigenen Weg.
 
        Sie gehoeren nicht in `objects` - dort stuenden sie im Geraetebaum,
        wo sie nichts zu suchen haben. Nachziehen muss man sie aber, und
@@ -128,6 +150,10 @@ export function horcheAufObjekte() {
       return;
     }
 
+    /* Alles Uebrige: nur was zaehlt, nicht jeder Schreibvorgang. */
+    var vorher = kern(S.objects[id]);
+    if (vorher === nachher) { return; }
+
     var gabEs = S.objects[id] !== undefined;
     if (obj) { S.objects[id] = obj; } else { delete S.objects[id]; }
     if (gabEs !== (S.objects[id] !== undefined)) { nachziehBaum = true; }

package/admin/js/pruefungen.js

@@ -68,7 +68,7 @@ export function pruefungDopplung(e) {
 export function pruefungSenden(e) {
   var ziele = e.states.filter(function (x) { return x.on && x.srcW; });
   if (!ziele.length) { return { s: 'mut', t: tr('check.publish'), d: tr('check.noWritePath') }; }
-  var mqtt = [], stumm = [], fehlt = [];
+  var mqtt = [], stumm = [], fehlt = [], unklar = [];
   ziele.forEach(function (x) {
     /* Drei Faelle, nicht zwei: den Punkt gibt es gar nicht, es gibt ihn
        und er ist stumm, oder er sendet. Der erste kam dazu, seit die
@@ -77,11 +77,19 @@ export function pruefungSenden(e) {
     var m = mqttEinstellung(x.srcW);
     if (!m) { return; }
     mqtt.push(x);
-    if (!m.publish) { stumm.push(x.n); }
+    /* Nur ein ausdrueckliches Nein ist ein Nein. `null` heisst
+       unbekannt: Objekte von ioBroker.mqtt und sonoff tragen das Thema
+       nur in `native`, ohne `custom`-Block — ueber ihre Sendefaehigkeit
+       sagt das nichts. Vorher stand dort fest `false`, und die Pruefung
+       meldete rot „kann nicht senden", ohne dass sich etwas daran
+       aendern liess (gemessen 09.09.2026). */
+    if (m.publish === false) { stumm.push(x.n); }
+    else if (m.publish === null) { unklar.push(x.n); }
   });
   if (fehlt.length) { return { s: 'bad', t: tr('check.publish'), d: tr('check.sendPointMissing', fehlt.join(', ')) }; }
   if (!mqtt.length) { return { s: 'mut', t: tr('check.publish'), d: tr('check.notMqtt') }; }
   if (stumm.length) { return { s: 'bad', t: tr('check.publish'), d: tr('check.cannotSend', stumm.join(', ')) }; }
+  if (unklar.length) { return { s: 'mut', t: tr('check.publish'), d: tr('check.sendUnknown', unklar.join(', ')) }; }
   return { s: 'ok', t: tr('check.publish'), d: tr('check.canSend', mqtt.length) };
 }
 

package/admin/js/quellentausch.js

@@ -162,7 +162,15 @@ export function tauschPlan(kanal, neuKanal) {
     }
     if (!zeile.neuW && zeile.altW) {
       var gw = geraten(zeile.altW, neuKanal);
-      if (gw) { zeile.neuW = gw; }
+      /* Was hier geraten wird, muss ein anderer Punkt sein als die
+         Lesequelle. Bei getrennten Quellen heissen Melder und Befehl oft
+         gleich - `stat.POWER` und `cmnd.POWER` -, und das Raten greift
+         nach dem Namen: aus der Schreibquelle `cmnd.POWER` wurde am
+         neuen Geraet `stat.POWER`, also genau der Melder. Der Alias
+         haette danach seinen Befehl ins Status-Thema geschickt; das
+         Geraet schaltet nicht und die Rueckmeldung ist verfaelscht
+         (gemessen 09.09.2026). Lieber kein Treffer als dieser. */
+      if (gw && gw !== zeile.neuR) { zeile.neuW = gw; }
     }
     /* Bei einfacher Quelle gilt die Lesequelle fuer beides. */
     if (zeile.einfach && zeile.neuR) { zeile.neuW = zeile.neuR; }
@@ -191,7 +199,19 @@ export function tauschObjekte(kanal, neuKanal, plan, weg) {
     delete neu.ts; delete neu.from; delete neu.user; delete neu.acl; delete neu._id;
     var a = neu.common.alias || {};
     if (z.neuW && z.neuW !== z.neuR) { a.id = { read: z.neuR, write: z.neuW }; }
-    else { a.id = z.neuR; }
+    else if (z.einfach) { a.id = z.neuR; }
+    else {
+      /* Getrennte Quellen ohne Schreibtreffer: die Objektform bleibt.
+
+         Eine blosse Zeichenkette heisst fuer den js-controller „lesen
+         und schreiben auf dieselbe Kennung". Aus einem Alias, der den
+         Zustand liest und den Befehl woanders hinschickt, wurde damit
+         einer, der auf den Melder schreibt - und aus einem bewusst nur
+         lesenden Punkt bei `common.write: true` sogar ein schreibender.
+         Ohne Treffer bleibt `write` einfach weg: dann schreibt der Alias
+         nirgendwohin, und das ist die ehrliche Auskunft. */
+      a.id = { read: z.neuR };
+    }
     neu.common.alias = a;
     raus.push({ id: z.id, neu: false, obj: neu });
   });
@@ -461,6 +481,15 @@ function zeichneTausch() {
       rechts.appendChild(paarZeile(tr('write.diffNew'),
         z.neuR + (z.neuW && z.neuW !== z.neuR ? '  →  ' + z.neuW : ''),
         'sn', gleich ? 'var(--ink-3)' : 'var(--ok)'));
+      /* Lesequelle gefunden, Schreibquelle nicht — das gehoert gesagt.
+         Sonst steht nur die neue Lesequelle in Gruen da, und dass der
+         Punkt danach nirgendwohin schreibt, merkt man erst am Geraet. */
+      if (z.altW && z.altW !== z.altR && !z.neuW) {
+        var kw = el('div');
+        kw.style.color = 'var(--bad)';
+        kw.textContent = tr('swap.noWriteMatch');
+        rechts.appendChild(kw);
+      }
     } else {
       var fehlt = el('div');
       fehlt.style.color = 'var(--bad)';