iobroker.device-watcher 2.8.4 → 2.8.5

package/README.md

@@ -189,6 +189,11 @@ It's possible to select:
     Placeholder for the next version (at the beginning of the line):
     ### **WORK IN PROGRESS**
 -->
+### 2.8.5 (2023-04-20)
+
+-   (ciddi89) Fixed: Error that instances can not be added to the blacklist because of the popup message [#172](https://github.com/ciddi89/ioBroker.device-watcher/issues/172)
+-   (ciddi89) Fixed: That messages have been sent when the instance has briefly jumped from Enabled to Disabled and back to Enabled [#173](https://github.com/ciddi89/ioBroker.device-watcher/issues/173)
+
 ### 2.8.4 (2023-04-16)
 
 -   (ciddi89) Fixed: Functions for instance status completely renewed ([#170](https://github.com/ciddi89/ioBroker.device-watcher/issues/170))

package/admin/jsonConfig.json

@@ -1389,15 +1389,7 @@
 							"sort": false,
 							"jsonData": "{\"instanceID\":\"${globalData.instanceID}\",\"adapter\":\"${globalData.adapter}\"}",
 							"command": "instancesList",
-							"width": "60%",
-							"confirm": {
-								"condition": "_alive",
-								"title": "Attention!",
-								"text": "This selection is only possible with a running instance!",
-								"ok": "Got it!",
-								"cancel": "cancel",
-								"type": "warning"
-							}
+							"width": "60%"
 						},
 						{
 							"type": "checkbox",

package/io-package.json

@@ -1,8 +1,21 @@
 {
 	"common": {
 		"name": "device-watcher",
-		"version": "2.8.4",
+		"version": "2.8.5",
 		"news": {
+			"2.8.5": {
+				"en": "Fixed: Error that instances can not be added to the blacklist because of the popup message [#172](https://github.com/ciddi89/ioBroker.device-watcher/issues/172)\nFixed: That messages have been sent when the instance has briefly jumped from Enabled to Disabled and back to Enabled [#173](https://github.com/ciddi89/ioBroker.device-watcher/issues/173)",
+				"de": "Behoben: Fehler, dass Instanzen aufgrund der Popup-Nachricht nicht zur Blacklist hinzugefügt werden können [#172](https://github.com/ciddi89/ioBroker.device-watcher/issues/172)\nBehoben: Das Meldungen gesendet wurden, wenn die Instanz kurz von Aktiviert auf Deaktiviert und zurück zu Aktiviert gesprungen ist [#173](https://github.com/ciddi89/ioBroker.device-watcher/issues/173)",
+				"ru": "Исправлено: Ошибка, которую нельзя добавить в черный список из-за всплывающего сообщения [#172](https://github.com/ciddi89/ioBroker.device-watcher/issues/172)\nИсправлено: Эти сообщения были отправлены, когда экземпляр кратко перепрыгнул из Enabled в Disabled и обратно в Enabled [#173](https://github.com/ciddi89/ioBroker.device-watcher/issues/173)",
+				"pt": "Corrigido: Erro que as instâncias não podem ser adicionadas à lista negra por causa da mensagem pop-up [#172](https://github.com/ciddi89/ioBroker.device-watcher/issues/172)\nCorrigido: Essas mensagens foram enviadas quando a instância saltou brevemente de Enabled para Disabled e de volta para Enabled [#173](https://github.com/ciddi89/ioBroker.device-watcher/issues/173)",
+				"nl": "Dat kan niet worden toegevoegd aan de blacklist vanwege de popup-bericht\nVertaling: Die berichten zijn verstuurd toen het instituut kort van Enabled naar Disabled en terug naar Enabled 173: jittps, /ciddi89-Brokerdewadet-17",
+				"fr": "Correction : Erreur que les instances ne peuvent pas être ajoutées à la liste noire en raison du message popup [#172](https://github.com/ciddi89/ioBroker.device-watcher/issues/172)\nCorrection : Ces messages ont été envoyés lorsque l ' instance a brièvement sauté de Enabled à Disabled et de retour à Enabled [#173](https://github.com/ciddi89/ioBroker.device-watcher/issues/173)",
+				"it": "Risolto: Errore che le istanze non possono essere aggiunte alla lista nera a causa del messaggio popup [#172](https://github.com/ciddi89/ioBroker.device-watcher/issues/172)\nRisolto: Tali messaggi sono stati inviati quando l'istanza è saltata brevemente da Enabled a disabilitare e di nuovo a Enabled [#173](https://github.com/ciddi89/ioBroker.device-watcher/issues/173)",
+				"es": "Corregido: Error de que los casos no se pueden agregar a la lista negra debido al mensaje popup [#172](https://github.com/ciddi89/ioBroker.device-watcher/issues/172)\nArreglado: Esos mensajes se han enviado cuando el caso ha saltado brevemente de Enabled a Disabled y de regreso a Enabled [#173](https://github.com/ciddi89/ioBroker.device-watcher/issues/173)",
+				"pl": "Error, że instancje nie mogą być dodane do czarnej listy z powodu popupu #172(https:/github.com/ciddi89/ioBroker.device-watcher/issues/172)\nFixed: Wiadomości te zostały wysłane, gdy instancja przeskakuje z Enabled do Disabled #173 (https:/github.com/ciddi89/ioBroker.device-watcher/issues/173)",
+				"uk": "Виправлено: помилка, що екземпляри не можуть бути додані в чорний список через повідомлення електронної пошти [#172] (https://github.com/ciddi89/ioBroker.device-watcher/issues/172)\nВиправлено: Що повідомлення було відправлено, коли екземпляр коротко стрибнув з Увімкненого і назад до Увімкненого [#173] (https://github.com/ciddi89/ioBroker.device-watcher/issues/173)",
+				"zh-cn": "固定:由于人口信息[第172号](http://github.com/ciddi89/ioBroker.device-watcher/issues/172)而无法增加黑名单。\n固定: 这一信息是在以下情况下发出的:从便携起降到残疾人,回到了可居住的[第173号](http://github.com/ciddi89/ioBroker.device-watcher/issues/173)。"
+			},
 			"2.8.4": {
 				"en": "Fixed: Functions for instance status completely renewed ([#170](https://github.com/ciddi89/ioBroker.device-watcher/issues/170))\nAdded: Popup message when the instance is not running and the user tries to select a device/instance in the settings tables ([#170](https://github.com/ciddi89/ioBroker.device-watcher/issues/170))",
 				"de": "Behoben: Funktionen für Instanzen Status vollständig erneuert [#170](https://github.com/ciddi89/ioBroker.device-watcher/issues/170)\nHinzugefügt: Popup-Nachricht, wenn die Instanz nicht läuft und der Benutzer versucht ein Gerät/Instance in den Einstellungstabellen auszuwählen [#170](https://github.com/ciddi89/ioBroker.device-watcher/issues/170)",
@@ -80,19 +93,6 @@
 				"pl": "Wsparcie dla Ecovac-Deebota\nZastrzeżenie: Zależność\nPoprawa: drobne rozwiązanie",
 				"uk": "Додано: Підтримка Ecovacs-Deebot\nОновлено: Залежності\nУдосконалення: Маленькі кріплення",
 				"zh-cn": "增加:支持Ecovacs-Deebot\n更新:属地\n改进: 小型配件"
-			},
-			"2.7.0": {
-				"en": "Added: Fully-MQTT\nAdded: Notification Service Matrix\nAdded: List and number of active instances\nAdded: HTML lists for instances/adapters\nAdded: HM-RPC device update pending datapoint\nAdded: User can define reporting time for error instances\nFixed: Send notification about adapter updates only if there are new ones\nImprovements: Hide notification services only if no instance is choosen\nImprovements: Some small code changes to avoid error messages",
-				"de": "Hinzugefügt: Fully-MQTT\nHinzugefügt: Notification Service Matrix\nHinzugefügt: Liste und Anzahl der aktiven Instanzen\nHinzugefügt: HTML-Listen für Instanzen/Adapter\nHinzugefügt: HM-RPC Geräteupdate Datenpunkt\nHinzugefügt: Benutzer kann Berichtszeit für Fehlerinstanzen definieren\nBehoben: Sende Benachrichtigung über Adapter-Updates nur, wenn es neue gibt\nVerbesserungen: Ausblenden von Benachrichtigungsdiensten nur, wenn keine Instanz davon gewählt wurde\nVerbesserungen: Einige kleine Codeänderungen, um Fehlermeldungen zu vermeiden",
-				"ru": "Добавлена: Fully-MQTT\nДобавлена: Notification Service Matrix\nДобавлена: Список и количество активных инстанций\nДобавлены: HTML списки для инстанций/адаптеров\nДобавлена: HM-RPC обновление устройства в ожидании данных\nДобавлена: Пользователь может определить отчетное время для ошибок\nИсправлено: Отправьте уведомление об обновлениях адаптера только если есть новые\nУлучшения: Скрыть услуги уведомлений только если не выбрано\nУлучшения: Некоторые небольшие изменения кода, чтобы избежать сообщений об ошибках",
-				"pt": "Adicionado: Fully-MQTT\nAdicionado: Notification Service Matrix\nAdicionado: Lista e número de instâncias ativas\nAdicionado: listas HTML para instâncias/adapters\nAdicionado: HM-RPC dispositivo atualização pendente datapoint\nAdicionado: O usuário pode definir tempo de relatório para instâncias de erro\nCorrigido: Enviar notificação sobre atualizações do adaptador somente se houver novos\nMelhorias: Ocultar serviços de notificação apenas se nenhuma instância for escolhida\nMelhorias: Algumas pequenas alterações de código para evitar mensagens de erro",
-				"nl": "Vertaling:\nVertaling:\nVertaling:\nVertaling:\nVertaling:\nUser kan de tijd definiëren voor foute instanties\nVerwittiging over updates als er nieuwe zijn\nImprovementen: Verberg de inlichtingendienst alleen als er geen voorkeur wordt gekozen\nImprovementen: Een kleine code verandert om foute berichten te vermijden",
-				"fr": "Ajouté : Fully-MQTT\nAjouté: Service de notification Matrix\nAjouté : Liste et nombre de cas actifs\nAjouté: Listes HTML pour les instances/adaptateurs\nAjouté : Mise à jour du dispositif HM-RPC en attendant le point de données\nAjouté: L'utilisateur peut définir le temps de déclaration des instances d'erreur\nCorrection : Envoyez une notification sur les mises à jour de l'adaptateur seulement s'il y en a de nouveaux\nAméliorations : Cacher les services de notification seulement si aucune instance n'est choisie\nAméliorations : Quelques petits changements de code pour éviter les messages d'erreur",
-				"it": "Aggiunto: Fully-MQTT\nAggiunto: Servizio di notifica Matrix\nAggiunto: Lista e numero di istanze attive\nAggiunto: liste HTML per istanze / adattatori\nAggiunto: HM-RPC dispositivo aggiornamento in sospeso datapoint\nAggiunto: L'utente può definire il tempo di reporting per le istanze di errore\nFisso: Invia la notifica sugli aggiornamenti dell'adattatore solo se ci sono nuovi\nMiglioramenti: Nascondi i servizi di notifica solo se non si sceglie un'istanza\nMiglioramenti: Alcuni piccoli cambiamenti di codice per evitare messaggi di errore",
-				"es": "Añadido: Fully-MQTT\nAñadido: Matriz de servicio de notificación\nAñadido: Lista y número de casos activos\nAñadido: listas HTML para casos/adapteros\nAñadido: Actualización del dispositivo HM-RPC hasta el punto de datos\nAñadido: El usuario puede definir el tiempo de presentación de informes para casos de error\nFijo: Enviar notificación sobre actualizaciones de adaptador solamente si hay nuevas\nMejoras: Ocultar los servicios de notificación sólo si no se elige ningún caso\nMejoras: Algunos cambios de código pequeño para evitar mensajes de error",
-				"pl": "Oficjalna strona Fully-MQTT\nStrona oficjalna\nW: Lista i liczba aktywnych instancji\nAdded: lista HTML dla przykładów/adaptatorów\nAdded: HM-RPC (ang.)\nAdded: Użytkownicy mogą zdefiniować czas błędu\nFixed: Send notification about adapter updates (ang.)\nPoprawa: Usługi Hide nie tylko wtedy, gdy żadna instancja nie jest wybierana\nPoprawa: Niektóre małe zmiany kodu pozwalają uniknąć błędu",
-				"uk": "Доданий: Повністю-MQTT\nДодано: Notification Service Matrix\nДодано: Список та кількість активних екземплярів\nДодано: Списки HTML для екземплярів/адептиків\nДодано: HM-RPC оновлення завантажувальної точки даних\nДодано: Користувач може визначити час звіту для екземплярів помилок\nВиправлено: Надіслати повідомлення про оновлення адаптера тільки якщо є нові\nУдосконалення: Прихильники, які не вибирають\nУдосконалення: Деякі невеликі зміни коду, щоб уникнути повідомлення про помилки",
-				"zh-cn": "增 编:\n增 编:通知处\n增 编:积极事例清单和数目\n增 编:HTML案/招待人员名单\n添加:HM-RPC装置在数据点前更新\n增 编:用户可以确定报告错误时间\n九. 固定:关于适应性更新的通知,只有有新的资料才有更新。\n改进: 通知服务只不过是没有选择的。\n改进: 避免错误信息的一些小守则变化"
 			}
 		},
 		"titleLang": {

package/main.js

@@ -416,19 +416,18 @@ class DeviceWatcher extends utils.Adapter {
 								instanceData.isHealthy = instanceStatusRaw[1];
 								instanceData.status = instanceStatusRaw[2];
 
-								if (oldIsHealthyValue !== instanceData.isHealthy) {
-									if (this.config.checkSendInstanceDeactivatedMsg && !instanceData.isAlive) {
-										if (!this.blacklistInstancesNotify.includes(instanceID)) {
-											await this.sendStateNotifications('deactivatedInstance', instanceID);
-										}
-									}
-
-									if (this.config.checkSendInstanceFailedMsg && instanceData.isAlive && !instanceData.isHealthy) {
-										if (!this.blacklistInstancesNotify.includes(instanceID)) {
-											await this.sendStateNotifications('errorInstance', instanceID);
-										}
-									}
+								if (oldIsHealthyValue === instanceData.isHealthy) continue;
+								// send message when instance was deactivated
+								if (this.config.checkSendInstanceDeactivatedMsg && !instanceData.isAlive) {
+									if (this.blacklistInstancesNotify.includes(instanceID)) continue;
+									await this.sendStateNotifications('deactivatedInstance', instanceID);
+								}
+								// send message when instance has an error
+								if (this.config.checkSendInstanceFailedMsg && instanceData.isAlive && !instanceData.isHealthy) {
+									if (this.blacklistInstancesNotify.includes(instanceID)) continue;
+									await this.sendStateNotifications('errorInstance', instanceID);
 								}
+
 								instanceData.checkIsRunning = false;
 							}
 							break;
@@ -2127,6 +2126,49 @@ class DeviceWatcher extends utils.Adapter {
 		return [isAlive, isHealthy, instanceStatusString];
 	}
 
+	/**
+	 * Check if instance is alive and ok
+	 * @param {string} instanceID
+	 * @param {number} instanceDeactivationTime
+	 */
+	async checkDaemonIsAlive(instanceID, instanceDeactivationTime) {
+		const aliveState = await this.getInitValue(`system.adapter.${instanceID}.alive`);
+		let daemonIsAlive;
+
+		let isAlive = false;
+		let isHealthy = false;
+		let instanceStatusString = 'Instanz deaktiviert';
+
+		if (aliveState) {
+			daemonIsAlive = await this.checkDaemonIsHealthy(instanceID);
+			isAlive = Boolean(daemonIsAlive[0]);
+			isHealthy = Boolean(daemonIsAlive[1]);
+			instanceStatusString = String(daemonIsAlive[2]);
+		} else if (!aliveState) {
+			await this.delay(instanceDeactivationTime);
+			daemonIsAlive = await this.checkDaemonIsHealthy(instanceID);
+			if (!daemonIsAlive[0]) {
+				await this.delay(instanceDeactivationTime);
+				daemonIsAlive = await this.checkDaemonIsHealthy(instanceID);
+				if (!daemonIsAlive[0]) {
+					isAlive = Boolean(daemonIsAlive[0]);
+					isHealthy = Boolean(daemonIsAlive[1]);
+					instanceStatusString = String(daemonIsAlive[2]);
+				} else {
+					isAlive = Boolean(daemonIsAlive[0]);
+					isHealthy = Boolean(daemonIsAlive[1]);
+					instanceStatusString = String(daemonIsAlive[2]);
+				}
+			} else {
+				isAlive = Boolean(daemonIsAlive[0]);
+				isHealthy = Boolean(daemonIsAlive[1]);
+				instanceStatusString = String(daemonIsAlive[2]);
+			}
+		}
+
+		return [isAlive, isHealthy, instanceStatusString];
+	}
+
 	async checkScheduleisHealty(instanceID, scheduleTime) {
 		let lastUpdate;
 		let previousCronRun = null;
@@ -2190,24 +2232,8 @@ class DeviceWatcher extends utils.Adapter {
 					instanceErrorTime = this.userTimeInstancesList.get(instanceID).errorTime;
 					instanceErrorTime = (instanceErrorTime * 1000) / 2; // calculate sec to ms and divide into two
 				}
-				daemonIsAlive = await this.checkDaemonIsHealthy(instanceID);
 
-				daemonIsNotAlive = async () => {
-					// wait first time
-					await this.delay(instanceDeactivationTime);
-					daemonIsAlive = await this.checkDaemonIsHealthy(instanceID);
-					if (!daemonIsAlive[0]) {
-						// wait second time
-						await this.delay(instanceDeactivationTime);
-						daemonIsAlive = await this.checkDaemonIsHealthy(instanceID);
-						if (!daemonIsAlive[0]) {
-							// finally
-							isAlive = Boolean(daemonIsAlive[0]);
-							isHealthy = Boolean(daemonIsAlive[1]);
-							instanceStatusString = String(daemonIsAlive[2]);
-						}
-					}
-				};
+				daemonIsAlive = await this.checkDaemonIsHealthy(instanceID);
 
 				if (daemonIsAlive[0]) {
 					if (daemonIsAlive[1]) {
@@ -2228,16 +2254,28 @@ class DeviceWatcher extends utils.Adapter {
 								isHealthy = Boolean(daemonIsAlive[1]);
 								instanceStatusString = String(daemonIsAlive[2]);
 							} else if (!daemonIsAlive[0]) {
-								await daemonIsNotAlive();
+								daemonIsNotAlive = await this.checkDaemonIsAlive(instanceID, instanceDeactivationTime);
+								isAlive = Boolean(daemonIsNotAlive[0]);
+								isHealthy = Boolean(daemonIsNotAlive[1]);
+								instanceStatusString = String(daemonIsNotAlive[2]);
 							}
 						} else if (!daemonIsAlive[0]) {
-							await daemonIsNotAlive();
+							daemonIsNotAlive = await this.checkDaemonIsAlive(instanceID, instanceDeactivationTime);
+							isAlive = Boolean(daemonIsNotAlive[0]);
+							isHealthy = Boolean(daemonIsNotAlive[1]);
+							instanceStatusString = String(daemonIsNotAlive[2]);
 						}
 					} else if (!daemonIsAlive[0]) {
-						await daemonIsNotAlive();
+						daemonIsNotAlive = await this.checkDaemonIsAlive(instanceID, instanceDeactivationTime);
+						isAlive = Boolean(daemonIsNotAlive[0]);
+						isHealthy = Boolean(daemonIsNotAlive[1]);
+						instanceStatusString = String(daemonIsNotAlive[2]);
 					}
 				} else if (!daemonIsAlive[0]) {
-					await daemonIsNotAlive();
+					daemonIsNotAlive = await this.checkDaemonIsAlive(instanceID, instanceDeactivationTime);
+					isAlive = Boolean(daemonIsNotAlive[0]);
+					isHealthy = Boolean(daemonIsNotAlive[1]);
+					instanceStatusString = String(daemonIsNotAlive[2]);
 				}
 				break;
 		}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "iobroker.device-watcher",
-  "version": "2.8.4",
+  "version": "2.8.5",
   "description": "Watchdog for devices",
   "author": {
     "name": "Christian Behrends",