iobroker.poolcontrol 0.6.0 → 0.6.1

package/io-package.json

@@ -1,8 +1,20 @@
 {
   "common": {
     "name": "poolcontrol",
-    "version": "0.6.0",
+    "version": "0.6.1",
     "news": {
+       "0.6.1": {
+         "en": "Fixed false monthly reset when no last_reset date existed. Corrected real last_update timestamp in monthly statistics.",
+         "de": "Fehlerhaften Monatsreset ohne gültiges last_reset-Datum behoben. Korrigierter echter Zeitstempel last_update in der Monatsstatistik.",
+         "ru": "Исправлен неверный ежемесячный сброс без действительной даты last_reset. Исправлено обновление времени last_update в ежемесячной статистике.",
+         "pt": "Corrigido o reset mensal incorreto quando não havia data last_reset. Corrigido o timestamp real de last_update na estatística mensal.",
+         "nl": "Onjuiste maandelijkse reset opgelost wanneer er geen last_reset-datum was. Gecorrigeerde echte last_update-tijd in maandelijkse statistieken.",
+         "fr": "Correction du réinitialisation mensuelle erronée lorsqu'aucune date last_reset n'existait. Horodatage last_update réel corrigé dans les statistiques mensuelles.",
+         "it": "Corretto il reset mensile errato quando non esisteva una data last_reset. Corretto il timestamp reale last_update nelle statistiche mensili.",
+         "es": "Corregido el restablecimiento mensual erróneo cuando no existía una fecha last_reset. Corregida la marca de tiempo real last_update en las estadísticas mensuales.",
+         "pl": "Naprawiono błędny miesięczny reset, gdy nie istniała data last_reset. Poprawiono rzeczywisty znacznik czasu last_update w miesięcznych statystykach.",
+         "zh-cn": "修复了在不存在 last_reset 日期时错误的月度重置。修正了月度统计中的真实 last_update 时间戳。"
+       },
        "0.6.0": {
          "en": "Added complete photovoltaic control with automatic pump management (mode 'Automatik (PV)'). The adapter can now react to PV surplus based on configurable house consumption and generation values. Includes migration for existing installations. Improved system consistency and internal logging.",
          "de": "Vollständige Photovoltaik-Steuerung mit automatischer Pumpenlogik (Modus 'Automatik (PV)') hinzugefügt. Der Adapter reagiert nun auf PV-Überschuss anhand konfigurierbarer Hausverbrauchs- und Erzeugungswerte. Migration für bestehende Installationen integriert. Systemkonsistenz und interne Protokollierung verbessert.",
@@ -79,18 +91,6 @@
          "pl": "Rozszerzone statystyki tygodniowe i miesięczne z trwałymi danymi, ujednoliconym formatem JSON i ochroną przed ponowną instalacją.",
          "uk": "Розширена тижнева та місячна статистика з постійними даними, уніфікованим форматом JSON і захистом під час перевстановлення.",
          "zh-cn": "扩展的周和月统计，具有持久数据、统一的 JSON 格式和重新安装保护。"
-       },
-      "0.5.0": {
-        "en": "Added weekly and monthly temperature statistics under analytics.statistics.temperature.week and analytics.statistics.temperature.month with automatic summaries, independent helpers and persistent data points.",
-        "de": "Wöchentliche und monatliche Temperaturstatistiken unter analytics.statistics.temperature.week und analytics.statistics.temperature.month hinzugefügt – mit automatischen Zusammenfassungen, unabhängigen Helpern und persistenten Datenpunkten.",
-        "ru": "Добавлена еженедельная и ежемесячная статистика температуры в analytics.statistics.temperature.week и analytics.statistics.temperature.month с автоматическими сводками, независимыми помощниками и постоянными точками данных.",
-        "fr": "Ajout des statistiques de température hebdomadaires et mensuelles sous analytics.statistics.temperature.week et analytics.statistics.temperature.month avec résumés automatiques, helpers indépendants et points de données persistants.",
-        "it": "Aggiunte statistiche settimanali e mensili della temperatura in analytics.statistics.temperature.week e analytics.statistics.temperature.month con riepiloghi automatici, helper indipendenti e punti dati persistenti.",
-        "es": "Se añadieron estadísticas semanales y mensuales de temperatura en analytics.statistics.temperature.week y analytics.statistics.temperature.month con resúmenes automáticos, ayudantes independientes y puntos de datos persistentes.",
-        "nl": "Wekelijkse en maandelijkse temperatuurstatistieken toegevoegd onder analytics.statistics.temperature.week en analytics.statistics.temperature.month met automatische samenvattingen, onafhankelijke helpers en persistente gegevenspunten.",
-        "pl": "Dodano tygodniowe i miesięczne statystyki temperatury w analytics.statistics.temperature.week i analytics.statistics.temperature.month z automatycznymi podsumowaniami, niezależnymi pomocnikami i trwałymi punktami danych.",
-        "uk": "Додано щотижневу та щомісячну статистику температури в analytics.statistics.temperature.week та analytics.statistics.temperature.month з автоматичними зведеннями, незалежними помічниками та постійними точками даних.",
-        "zh-cn": "在 analytics.statistics.temperature.week 和 analytics.statistics.temperature.month 中添加了每周和每月温度统计，具有自动摘要、独立助手和持久数据点。"
        }
 	},
     "titleLang": {

package/lib/helpers/migrationHelper.js

@@ -9,7 +9,7 @@
  * - Wird beim Adapterstart einmalig ausgeführt.
  * - Korrigiert veraltete Definitionen (z. B. Schreibrechte, persist-Flags, etc.)
  *
- * Version: 1.0.2
+ * Version: 1.0.3
  */
 
 const migrationHelper = {
@@ -32,10 +32,7 @@ const migrationHelper = {
             await this._fixSpeechQueue();
             await this._fixSolarWarnActivePersist();
             await this._fixPumpModeStates(); // NEU: PV-Automatik hinzufügen
-
-            // Weitere Routinen folgen hier später:
-            // await this._ensurePumpReason();
-            // await this._cleanupOldStates();
+            await this._removeInvalidResetButtons(); // NEU: Entfernt Week/Month-Reset-Buttons
 
             this.adapter.log.debug('[migrationHelper] Migration-Checks abgeschlossen.');
         } catch (err) {
@@ -48,17 +45,11 @@ const migrationHelper = {
     // ------------------------------------------------------
     // Migration: Schreibrecht für speech.queue korrigieren
     // ------------------------------------------------------
-
-    /**
-     * Prüft und korrigiert den State "speech.queue", falls er noch write:false gesetzt hat.
-     * Dadurch verschwinden Warnungen beim Schreiben (Read-only state ... written without ack).
-     */
     async _fixSpeechQueue() {
         const id = 'speech.queue';
         try {
             const obj = await this.adapter.getObjectAsync(id);
             if (!obj) {
-                this.adapter.log.debug(`[migrationHelper] ${id} existiert nicht – keine Anpassung nötig.`);
                 return;
             }
 
@@ -71,8 +62,6 @@ const migrationHelper = {
                         desc: 'Nur intern durch den Adapter beschreibbar (nicht manuell ändern!)',
                     },
                 });
-            } else {
-                this.adapter.log.debug(`[migrationHelper] ${id} ist bereits korrekt konfiguriert.`);
             }
         } catch (err) {
             this.adapter.log.warn(`[migrationHelper] Fehler bei Prüfung von ${id}: ${err.message}`);
@@ -82,18 +71,11 @@ const migrationHelper = {
     // ------------------------------------------------------
     // Migration: persist-Flag für solar.warn_active ergänzen
     // ------------------------------------------------------
-
-    /**
-     * Ergänzt persist:true bei solar.warn_active,
-     * damit die Einstellung (Warnfunktion aktivieren/deaktivieren)
-     * nach einem Neustart erhalten bleibt.
-     */
     async _fixSolarWarnActivePersist() {
         const id = 'solar.warn_active';
         try {
             const obj = await this.adapter.getObjectAsync(id);
             if (!obj) {
-                this.adapter.log.debug(`[migrationHelper] ${id} existiert nicht – keine Anpassung nötig.`);
                 return;
             }
 
@@ -106,8 +88,6 @@ const migrationHelper = {
                         desc: `${obj.common?.desc || ''} (automatisch per Migration persistiert)`,
                     },
                 });
-            } else {
-                this.adapter.log.debug(`[migrationHelper] ${id} ist bereits mit persist:true versehen.`);
             }
         } catch (err) {
             this.adapter.log.warn(`[migrationHelper] Fehler bei Prüfung von ${id}: ${err.message}`);
@@ -117,18 +97,11 @@ const migrationHelper = {
     // ------------------------------------------------------
     // Migration: Ergänze neuen Pumpenmodus "Automatik (PV)"
     // ------------------------------------------------------
-
-    /**
-     * Prüft den State pump.mode und ergänzt, falls nötig, den neuen
-     * Eintrag "auto_pv" → "Automatik (PV)".
-     * Dadurch wird der PV-Modus automatisch in bestehenden Installationen sichtbar.
-     */
     async _fixPumpModeStates() {
         const id = 'pump.mode';
         try {
             const obj = await this.adapter.getObjectAsync(id);
             if (!obj) {
-                this.adapter.log.debug(`[migrationHelper] ${id} existiert nicht – keine Anpassung nötig.`);
                 return;
             }
 
@@ -136,16 +109,53 @@ const migrationHelper = {
             if (!states.auto_pv) {
                 states.auto_pv = 'Automatik (PV)';
                 this.adapter.log.info(`[migrationHelper] Ergänze neuen Modus "Automatik (PV)" in pump.mode`);
-                await this.adapter.extendObjectAsync(id, {
-                    common: { states },
-                });
-            } else {
-                this.adapter.log.debug(`[migrationHelper] pump.mode enthält bereits "Automatik (PV)".`);
+                await this.adapter.extendObjectAsync(id, { common: { states } });
             }
         } catch (err) {
             this.adapter.log.warn(`[migrationHelper] Fehler bei Prüfung von ${id}: ${err.message}`);
         }
     },
+
+    // FIX: Entferne versehentlich angelegte Reset-Buttons aus Wochen- und Monatsstatistik
+    async _removeInvalidResetButtons() {
+        try {
+            const allObjs = await this.adapter.getAdapterObjectsAsync();
+            const keys = Object.keys(allObjs);
+            let removed = 0;
+
+            for (const id of keys) {
+                if (
+                    (id.startsWith('analytics.statistics.temperature.week.') ||
+                        id.startsWith('analytics.statistics.temperature.month.')) &&
+                    id.endsWith('.reset_today')
+                ) {
+                    try {
+                        // Erst Statewert entfernen
+                        await this.adapter.delStateAsync(id);
+                    } catch {
+                        this.adapter.log.debug(`[migrationHelper] Kein Statewert für ${id} vorhanden (Überspringe).`);
+                    }
+
+                    // Danach Objekt löschen (auch wenn persist=true)
+                    try {
+                        await this.adapter.delObjectAsync(id, { recursive: false });
+                        this.adapter.log.info(`[migrationHelper] Veralteter Reset-Button entfernt: ${id}`);
+                        removed++;
+                    } catch (err) {
+                        this.adapter.log.warn(`[migrationHelper] Konnte ${id} nicht löschen: ${err.message}`);
+                    }
+                }
+            }
+
+            if (removed === 0) {
+                this.adapter.log.debug('[migrationHelper] Keine alten Reset-Buttons gefunden.');
+            } else {
+                this.adapter.log.info(`[migrationHelper] Insgesamt ${removed} alte Reset-Buttons entfernt.`);
+            }
+        } catch (err) {
+            this.adapter.log.warn(`[migrationHelper] Fehler beim Entfernen alter Reset-Buttons: ${err.message}`);
+        }
+    },
 };
 
 module.exports = migrationHelper;

package/lib/helpers/photovoltaicHelper.js

@@ -152,18 +152,29 @@ const photovoltaicHelper = {
             return this._maybeStopPump(false, 0, 'mode_not_auto_pv');
         }
 
-        // Optional: Umwälzung erreicht?
+        // FIX: PV-Helfer darf nur aktiv sein, solange Umwälzung noch nicht erfüllt ist
         if (ignoreOnCirc) {
             try {
                 const remainingState = await this.adapter.getForeignStateAsync(
                     'poolcontrol.0.circulation.daily_remaining',
                 );
                 const remaining = Number(remainingState?.val ?? NaN);
-                if (Number.isFinite(remaining) && remaining <= 0) {
-                    this.adapter.log.debug(
-                        `[photovoltaicHelper] Tagesumwälzung erreicht (daily_remaining = ${remaining}) → PV-Steuerung ignoriert.`,
-                    );
-                    return this._maybeStopPump(false, afterrunMin, 'circulation_reached');
+
+                if (Number.isFinite(remaining)) {
+                    // RULE: Wenn Umwälzung bereits erfüllt → Pumpe sofort AUS (ohne Nachlauf)
+                    if (remaining <= 0) {
+                        this.adapter.log.info(
+                            `[photovoltaicHelper] Tagesumwälzung erreicht (daily_remaining=${remaining}) → PV-Steuerung beendet, Pumpe AUS.`,
+                        );
+                        return this._maybeStopPump(true, 0, 'circulation_reached_force_off');
+                    }
+
+                    // RULE: Wenn Umwälzung noch nicht erfüllt → nur dann darf bei Überschuss eingeschaltet werden
+                    if (remaining > 0 && surplusActive) {
+                        this.adapter.log.debug(
+                            `[photovoltaicHelper] Tagesumwälzung noch nicht erfüllt (${remaining}) → PV-Steuerung aktiv.`,
+                        );
+                    }
                 }
             } catch (err) {
                 this.adapter.log.debug(
@@ -172,10 +183,31 @@ const photovoltaicHelper = {
             }
         }
 
-        // Schaltlogik
+        // FIX: PV-Schaltlogik mit Prüfung der Umwälzung
         if (surplusActive) {
-            return this._maybeStartPump('pv_surplus');
+            try {
+                const remainingState = await this.adapter.getForeignStateAsync(
+                    'poolcontrol.0.circulation.daily_remaining',
+                );
+                const remaining = Number(remainingState?.val ?? NaN);
+
+                // RULE: Einschalten nur, wenn Umwälzung noch nicht erfüllt
+                if (Number.isFinite(remaining) && remaining <= 0) {
+                    this.adapter.log.info(
+                        `[photovoltaicHelper] Tagesumwälzung bereits erfüllt (${remaining}) → Pumpe bleibt AUS (kein Start trotz Überschuss).`,
+                    );
+                    return this._maybeStopPump(true, 0, 'circulation_already_reached');
+                }
+
+                // RULE: Überschuss aktiv UND Umwälzung noch nicht erfüllt → einschalten
+                return this._maybeStartPump('pv_surplus');
+            } catch (err) {
+                this.adapter.log.warn(`[photovoltaicHelper] Fehler beim Prüfen von daily_remaining: ${err.message}`);
+                return this._maybeStartPump('pv_surplus');
+            }
         }
+
+        // RULE: Kein Überschuss → ggf. Nachlauf/Aus
         return this._maybeStopPump(false, afterrunMin, 'pv_ended_afterrun');
     },
 

package/lib/helpers/statisticsHelperMonth.js

@@ -250,6 +250,12 @@ const statisticsHelperMonth = {
         });
 
         await this._updateOverallSummary();
+
+        // FIX: last_update korrigieren (echtes Aktualisierungsdatum)
+        await adapter.setStateAsync(`${basePath}.last_update`, {
+            val: new Date().toISOString(),
+            ack: true,
+        });
     },
 
     /**
@@ -385,6 +391,20 @@ const statisticsHelperMonth = {
     async _resetMonthlyTemperatureStats() {
         const adapter = this.adapter;
 
+        // 🟢 Neuer Logikblock: Monatsreset erfolgt nur, wenn wirklich neuer Monat begonnen hat
+        try {
+            const now = new Date();
+            const currentDay = now.getDate();
+
+            // Nur am 1. Tag des Monats ausführen (00:05 Uhr geplant)
+            if (currentDay !== 1) {
+                adapter.log.debug('[statisticsHelperMonth] Kein Monatsanfang – Reset übersprungen.');
+                return;
+            }
+        } catch (err) {
+            adapter.log.warn(`[statisticsHelperMonth] Fehler bei Monatsbeginn-Prüfung: ${err.message}`);
+        }
+
         // 🟢 NEU: Schutz vor Endlosschleifen und Mehrfachausführung
         if (this.isResetting) {
             adapter.log.debug('statisticsHelperMonth: Reset bereits aktiv – übersprungen.');

package/lib/stateDefinitions/statisticsStates.js

@@ -100,7 +100,6 @@ async function _createTemperatureStatsGroup(adapter, periodId, displayName) {
             { id: 'temp_avg', name: 'Durchschnittstemperatur', type: 'number', role: 'value.temperature', unit: '°C' },
             { id: 'data_points_count', name: 'Anzahl Messwerte', type: 'number', role: 'value' },
             { id: 'last_update', name: 'Letzte Aktualisierung', type: 'string', role: 'value.time' },
-            { id: 'reset_today', name: 'Tagesstatistik zurücksetzen', type: 'boolean', role: 'button' },
             {
                 id: 'summary_json',
                 name: `${displayName} (JSON)`,
@@ -115,6 +114,16 @@ async function _createTemperatureStatsGroup(adapter, periodId, displayName) {
             },
         ];
 
+        // FIX: Reset-Button nur bei Tagesstatistik anlegen
+        if (periodId === 'today') {
+            stateDefs.push({
+                id: 'reset_today',
+                name: 'Tagesstatistik zurücksetzen',
+                type: 'boolean',
+                role: 'button',
+            });
+        }
+
         for (const def of stateDefs) {
             await adapter.setObjectNotExistsAsync(`${basePath}.${def.id}`, {
                 type: 'state',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "iobroker.poolcontrol",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "Steuerung & Automatisierung für den Pool (Pumpe, Heizung, Ventile, Sensoren).",
   "author": "DasBo1975 <dasbo1975@outlook.de>",
   "homepage": "https://github.com/DasBo1975/ioBroker.poolcontrol",