Neuer Blogbeitrag: Monatsrückblick - Dezember 2025 🎄

[Dominik F.]

Ich glaube ich weiß mittlerweile wo der Fehler liegt, ich bin nur etwas überfragt wie ich das lösen kann.

Der Datenpunkt :javascript.0.Zigbee_Batterie_Status.text_LOWBAT ist wenn keine Fehlermeldung vorliegt leer. Wenn man also das skript so verändert, dass dort wenn nicht vorliegt sowas wie : "Alle Batterien Okay" steht, dann sollte message nicht mehr blank sein.
Kann mir da jemand helfen?

[Dominik F.]

@paul53
du hast hier im Thread schon so viel geholfen und habe auch passagen von dem code den du geschrieben hast übernommen, könntest du dir das vielleicht ansehen? Wäre dir sehr dankbar

[paul53]

@Dominik-F
Zeile 56:

            if(sendpush && arrLOWBAT.length) {

[Dominik F.]

@paul53

Zeile 41 ist die Funktion, Zeile 54 sollte der Aufruf sein. Das Skript habe ich aus diesem Thread kopiert

[paul53]

@Dominik-F sagte:

Zeile 54 sollte der Aufruf sein.

Ja, habe den Aufruf inzwischen gefunden und den Post schon geändert.

[Dominik F.]

@paul53

habe die passage geändert, bekomme den gleichen Error.

[paul53]

@Dominik-F
Was ist der Inhalt der beiden Datenpunkte "Zigbee_Batterie_Status.text_LOWBAT" und "Zigbee_Batterie_Status.anzahl_LOWBAT" ?

[Dominik F.]

@paul53
In Zigbee_Batterie_Status.text_LOWBAT steht welche Batterien leer sind, in Zigbee_Batterie_Status.anzahl_LOWBAT die Anzahl der Geräte mit leerer Batterie.
Ich glaube man müsste den ersten Datenpunkt befüllen, wenn keine Battierie leer ist z.B. mit "Alle Batterien Okay". Wenn ich das Skript nämlich ausführe wenn schon was drin steht, dann kommt der Fehler nicht.

[paul53]

@Dominik-F sagte:

In Zigbee_Batterie_Status.text_LOWBAT steht welche Batterien leer sind, in Zigbee_Batterie_Status.anzahl_LOWBAT die Anzahl der Geräte mit leerer Batterie.

Das ist mir klar. Ich wollte den konkreten Inhalt wissen, wenn die Fehlermeldung aufftritt.

@Dominik-F sagte:

man müsste den ersten Datenpunkt befüllen, wenn keine Battierie leer

pushover greift nicht auf den Datenpunkt zu. Wenn keine Batterie leer ist, muss arrLOWBAT.length den Wert 0 haben und somit in Zeile 56 der Aufruf von pushover verhindert werden.

[Dominik F.]

@paul53

oh sorry, das hab ich falsch verstanden :D Ich habe den Aufruf von Pushoven "nach besten Wissen und Gewissen" nach Zeile 56 gepackt, ob das jedoch ob das überhaupt dort richtig ist weiß ich ehrlich gesagt nicht.

[paul53]

@Dominik-F
Es gibt eine leere Batterie. Somit wird auch pushover aufgerufen und als Message "EG_Bad_Fenster: 86 %" übergeben.

[Dominik F.]

@paul53

Genau, das habe ich so eingestellt, um das Skript zu testen. Wenn es eine leere Batterie gibt, dann tritt keine Error Meldung auf. habe in Zeile 54 die 20 durch 90 ersetzt. Stelle ich das wieder auf 20 tritt die Error Meldung auf.

[paul53]

@paul53
Ich seh gerade, dass pushover in der Schleife aufgerufen wird. Das ist natürlich falsch. Ändere es mal:

    Zpercent.each(function (id, i) {            // Schleife für jedes gefundenen Element *.percent
        if(getState(id).val < 20) lowbat(id);
        ++moeglicheLOWBAT;                     // Zählt die Anzahl der vorhandenen Geräte unabhängig vom Status
    }); 

    if(sendpush && arrLOWBAT.length) {
        prio = prio_Lowbat; 
        titel = 'Batterie wechsel nötig:';
        message = arrLOWBAT.join(', '); 
        send_pushover(device, message, titel, prio);
    }        

[Dominik F.]

@paul53

perfekt, das war die Lösung! Wäre es möglich, wenn keine leere Batterie gefunden wird, das im Datenpunkt Zigbee_Batterie_Status.text_LOWBAT steht: "Alle Batterien Okay"?

[paul53]

@Dominik-F sagte:

wenn keine leere Batterie gefunden wird, das im Datenpunkt Zigbee_Batterie_Status.text_LOWBAT steht: "Alle Batterien Okay"?

   // die ermittelten Werte werden als javascript.0\. Variable in ioBroker gespeichert (z.B. für die Verarbeitung in VIS)
   var txt = "Alle Batterien Okay";
   if(arrLOWBAT.length) txt = arrLOWBAT.join(','); 
   setState("Zigbee_Batterie_Status.text_LOWBAT",  txt);        // Schreibt die Namen der Geräte mit LOWBAT Meldung

[Dominik F.]

@paul53

perfekt, jetzt ist es genau so, wie ich es mir vorstelle :) Vielen vielen Dank.
Darf ich dich bei einem anderen Skript wo du auch geholfen hast nochmal um Hilfe fragen? Dort geht es "nur" um einen Schönheitsfehler bei der Anzeige.

[paul53]

@Dominik-F sagte:

nochmal um Hilfe fragen?

Machen.
Ich hoffe, dass ich das Thema nicht auf "ignorieren" gestellt habe, denn dann bekomme ich Deine Frage nicht mit. Falls ich nicht antworte, dann poste hier ein Link auf die Frage.

[Dominik F.]

kann gelöscht werden

[K_o_bold]

@paul53 ,

ich bin auf der Suche nach einem Skript zur Überwachung der Batteriezustände auf dein Skript gestoßen.
Da ich wenig Javaskipt Kenntnisse habe, bin ich auf Hilfe angewiesen. Ich hoffe du kannst mich unterstützen. :-) Wäre jedenfalls sehr nett.

Dein Skript funktioniert nicht mit dem alias.0 Objektbaum. Gibt es dafür eine Lösung?

Mein Skipt sieht aktuell wie folgt aus. So funktioniert es nicht.
Stelle ich in Zeile 6 zum Beispiel auf $('deconz....) um funktioniert es.

createState('lowbat.zählenLowbat.möglicheLOWBAT', 0, {type: 'number'});   // wenn benötigt: Anzahl der vorhandenen LOWBAT
createState('lowbat.zählenLowbat.anzahlLOWBAT', 0, {type: 'number'});     // wenn benötigt: Anzahl der tatsächlichen LOWBAT
createState('lowbat.zählenLowbat.textLOWBAT', " ", {type: 'string'});     // Anzeige der LOWBAT
 
//const hmLOWBAT  = $('hm-rpc.*:0.LOWBAT');
const aliaspercent = $('alias.*.battery');
const loggen = false;  // = false, wenn kein Logging gewünscht
 
var arrLOWBAT;
 
function lowbat(id) {
   var obj = getObject(id);
   arrLOWBAT.push(obj.common.name);  // Zu Array hinzufügen
   if(loggen) log("Gerät: " + obj.common.name);
}
 
function countLowbat() {
   // Setzt die Zähler vor dem Durchlauf aller Elemente *.LOWBAT auf 0
   var moeglicheLOWBAT = 0;
   arrLOWBAT      = [];
   aliaspercent.each(function (id, i) {   // Schleife für jedes gefundenen Element *.percent
      if(getState(id).val < 99) lowbat(id);
      ++moeglicheLOWBAT;                         // Zählt die Anzahl der vorhandenen Geräte unabhängig vom Status
   }); 
 
   // Schleifen sind durchlaufen. Im Log wird der aktuelle Status (Anzahl, davon LOWBAT zutreffend) ausgegeben
   if(loggen) log("Text: " + arrLOWBAT.join(', '));
   if(loggen) log("Anzahl Geräte: " + moeglicheLOWBAT + " # davon LOWBAT erkannt: " +  arrLOWBAT.length);
 
   // die ermittelten Werte werden als javascript.0\. Variable in ioBroker gespeichert (z.B. für die Verarbeitung in VIS)
   setState("lowbat.zählenLowbat.textLOWBAT",     arrLOWBAT.join(', ')); // Schreibt die Namen der Geräte mit LOWBAT Meldung
   setState("lowbat.zählenLowbat.anzahlLOWBAT",   arrLOWBAT.length);        // Schreibt die Anzahl der LOWBAT-Meldungen
   setState("lowbat.zählenLowbat.möglicheLOWBAT", moeglicheLOWBAT);          // Schreibt die Anzahl der vorhandene Geräte 
}
 
countLowbat(); // Skriptstart
 
schedule("* * * * *", countLowbat);

[paul53]

@K_o_bold sagte:

Skript funktioniert nicht mit dem alias.0 Objektbaum.

Wie hast Du die Alias-Datenpunkte erstellt ? Falls manuell im Tab "Objekte", danach mal die Javascript-Instanz neu gestartet ?