[gelöst] Tägliche Meldung von Geräten, die einen schwachen Batteriezustand haben

[Dominik F.]

Hallo zusammen,

ich habe das Skript leicht verändert am laufen und es funktioniert auch alles wie es soll. Habe nun versucht eine Pushnachrichten Funktion in das Skript einzubauen. Die Funktion habe ich aus einem anderen funktionierendem Skript und ist deshalb größer als sie eigentlich sein muss, das ist jedoch nicht das Problem. Ich bekomme eine Pushnachricht die auch genauso funktioniert wie sie sein soll, jedoch produziere ich Errors im Log. Habe es auch mit einfachereren Funktionen für Push versucht und durch rumprobieren, jedoch kann ich es nicht lösen. Vielleicht kann da jemand helfen:
Hier mein Skript:

//pro Fehlertyp kann eine andere Instanz genutzt werden
const prio_Lowbat = 2;
   
//Variablen für Pushover
const sendpush   = true;      //true = verschickt per Pushover Nachrchten // false = Pushover wird nicht benutzt
const pushover_Instanz0 =  'pushover.0';     // Pushover instance für HM Alarme
const pushover_Instanz1 =  'pushover.1';     // Pushover instance für HM Warnungen
const pushover_Instanz2 =  'pushover.2';     // Pushover instance für Zigbee Alarme
const pushover_Instanz3 =  'pushover.3';     // Pushover instance für Prio = -1 oder -2
var prio = -2;              //nicht verändern die höchste Prio nach Fehlertyp wird verwendet
var titel;
var message;
var device = 'Handy_Domi';         //Welches Gerät soll die Nachricht bekommen
//var device = 'All'; 
 
//Ab hier eigentliches Script

function send_pushover (device, message, titel, prio) {
    var pushover_Instanz;
    if (prio === 0){pushover_Instanz =  pushover_Instanz0;}
    else if (prio == 1){pushover_Instanz =  pushover_Instanz1;}
    else if (prio == 2){pushover_Instanz =  pushover_Instanz2;}
    else {pushover_Instanz =  pushover_Instanz3;}
    sendTo(pushover_Instanz, { 
        device: device,
        message: message, 
        title: titel, 
        priority: 0,
    }); 
}

createState('Zigbee_Batterie_Status.mögliche_LOWBAT', 0, {type: 'number'});   // wenn benötigt: Anzahl der vorhandenen LOWBAT
createState('Zigbee_Batterie_Status.anzahl_LOWBAT', 0, {type: 'number'});     // wenn benötigt: Anzahl der tatsächlichen LOWBAT
createState('Zigbee_Batterie_Status.text_LOWBAT', " ", {type: 'string'});     // Anzeige der LOWBAT
 
const Zpercent = $('zigbee.0.*.battery');
const loggen = true;  // = false, wenn kein Logging gewünscht
 
var arrLOWBAT;

function lowbat(id) {
  let percent = getState(id).val;
  id = id.substring(0, id.lastIndexOf('.'));
  var name = getObject(id).common.name + ': ' + percent + ' %';
  arrLOWBAT.push(name);  // Zu Array hinzufügen
  if(loggen) log("Gerät: " + name);
}

function countLowbat() {  
    // Setzt die Zähler vor dem Durchlauf aller Elemente *.LOWBAT auf 0
    var moeglicheLOWBAT = 0; 
    arrLOWBAT      = [];
    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) {
                prio = prio_Lowbat; 
                titel = 'Batterie wechsel nötig:';
                message = arrLOWBAT.join(', '); 
                send_pushover(device, message, titel, prio);
            }        
    }); 
 
   // 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("Zigbee_Batterie_Status.text_LOWBAT",     arrLOWBAT.join(','));        // Schreibt die Namen der Geräte mit LOWBAT Meldung
   setState("Zigbee_Batterie_Status.anzahl_LOWBAT",   arrLOWBAT.length);           // Schreibt die Anzahl der LOWBAT-Meldungen
   setState("Zigbee_Batterie_Status.mögliche_LOWBAT", moeglicheLOWBAT);            // Schreibt die Anzahl der vorhandene Geräte 
}

// Skriptstart
countLowbat(); 
 

Hier die Errors:

host.raspberrypi4-io	2020-01-09 11:34:52.426	error	Caught by controller[0]: at process._tickCallback (internal/process/next_tick.js:63:19)
host.raspberrypi4-io	2020-01-09 11:34:52.425	error	Caught by controller[0]: at endReadableNT (_stream_readable.js:1143:12)
host.raspberrypi4-io	2020-01-09 11:34:52.425	error	Caught by controller[0]: at IncomingMessage.emit (events.js:203:15)
host.raspberrypi4-io	2020-01-09 11:34:52.424	error	Caught by controller[0]: at IncomingMessage.<anonymous> (/opt/iobroker/node_modules/pushover-notifications/lib/pushover.js:249:12)
host.raspberrypi4-io	2020-01-09 11:34:52.424	error	Caught by controller[0]: at Pushover.errors (/opt/iobroker/node_modules/pushover-notifications/lib/pushover.js:145:13)
host.raspberrypi4-io	2020-01-09 11:34:52.422	error	Caught by controller[0]: Error: message cannot be blank
pushover.2	2020-01-09 11:34:51.871	info	(31056) Terminated (NO_ERROR): Without reason
pushover.2	2020-01-09 11:34:51.870	info	(31056) terminating
pushover.2	2020-01-09 11:34:51.366	error	(31056) Error: message cannot be blank at Pushover.errors (/opt/iobroker/node_modules/pushover-notifications/lib/pushover.js:145:13) at IncomingMessage.<anonymous> (/opt/iobroker/node_modules/
pushover.2	2020-01-09 11:34:51.364	error	(31056) uncaught exception: message cannot be blank

[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 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);