From 75c0c1126e6a3db69766f09d20ad60e8702f10e8 Mon Sep 17 00:00:00 2001 From: duffyduck Date: Mon, 8 Jun 2026 13:35:55 +0200 Subject: [PATCH] Log what changed; add notification on/off settings Protokoll: - SyncResult.Changes erfasst jede tatsaechliche Aenderung (erstellt/aktualisiert/ geloescht/verknuepft/zusammengefuehrt je Kontakt) per Action()-Helfer. - Beide Sync-Pfade (Auto/Tray via RunSync UND manueller Sync via SyncProgressForm) schreiben Start, Ergebnis, Aenderungen, Konflikte und Fehler ins persistente Protokoll. Benachrichtigungen: - UserSettings: NotificationsEnabled (allgemein) + NotifyWarningsErrors (Konflikte/Fehler), beide in der Einstellungen-Maske als Haken. - MainForm.Balloon() zeigt Tray-Meldungen nur, wenn der passende Haken aktiv ist; Zusammenfassung gilt als Warnung, wenn Fehler/Konflikte auftraten. - Protokoll wird unabhaengig von den Benachrichtigungs-Einstellungen geschrieben. Co-Authored-By: Claude Opus 4.8 (1M context) --- CHANGELOG.md | 9 ++++ src/StarfaceOutlookSync/Models/SyncProfile.cs | 4 ++ .../Models/UserSettings.cs | 5 +++ .../Services/SyncEngine.cs | 41 ++++++++++-------- src/StarfaceOutlookSync/UI/MainForm.cs | 42 ++++++++++++++----- src/StarfaceOutlookSync/UI/SettingsForm.cs | 32 ++++++++++---- .../UI/SyncProgressForm.cs | 15 +++++++ 7 files changed, 113 insertions(+), 35 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 06ee6a7e..adbf33ee 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -38,6 +38,15 @@ Versionsschema ist `x.x.x.x` (siehe `release.sh`). ### Hinzugefuegt +- **Protokoll zeigt jetzt, WAS geaendert wurde.** Pro Sync werden die einzelnen + Aktionen (erstellt / aktualisiert / geloescht / verknuepft / zusammengefuehrt je + Kontakt) ins Protokoll geschrieben - sowohl beim manuellen Sync (Fenster) als + auch bei Auto-/Tray-Sync. +- **Benachrichtigungen schaltbar (Einstellungen).** Zwei unabhaengige Haken: + "Tray-Benachrichtigungen (allgemein)" fuer Info-Meldungen (Sync laeuft/fertig) + und "Benachrichtigungen bei Warnungen/Fehlern" fuer Konflikte und Fehler. So + kann man z.B. nur noch bei Konflikten/Fehlern benachrichtigt werden - oder gar + nicht. Das Protokoll wird unabhaengig davon immer geschrieben. - **Protokoll (Logdatei) + "Protokoll"-Button.** Syncs, Ergebnisse, Konflikte (lokal und von anderen Arbeitsplaetzen) und Fehler werden dauerhaft in `%AppData%\StarfaceOutlookSync\sync.log` festgehalten - so ist auch nach dem diff --git a/src/StarfaceOutlookSync/Models/SyncProfile.cs b/src/StarfaceOutlookSync/Models/SyncProfile.cs index 0d0a39aa..416f9f5e 100644 --- a/src/StarfaceOutlookSync/Models/SyncProfile.cs +++ b/src/StarfaceOutlookSync/Models/SyncProfile.cs @@ -73,6 +73,10 @@ namespace StarfaceOutlookSync.Models public int Errors { get; set; } public System.Collections.Generic.List ErrorMessages { get; set; } = new System.Collections.Generic.List(); + // Was konkret veraendert wurde (erstellt/aktualisiert/geloescht je Kontakt), + // fuer das Protokoll. + public System.Collections.Generic.List Changes { get; set; } = new System.Collections.Generic.List(); + // Echte Feld-Konflikte (dasselbe Feld auf beiden Seiten geaendert), die // ueber die Vorrang-Regel aufgeloest wurden. Fuer Benutzer-Hinweise. public System.Collections.Generic.List Conflicts { get; set; } = new System.Collections.Generic.List(); diff --git a/src/StarfaceOutlookSync/Models/UserSettings.cs b/src/StarfaceOutlookSync/Models/UserSettings.cs index be28bd7c..5877679a 100644 --- a/src/StarfaceOutlookSync/Models/UserSettings.cs +++ b/src/StarfaceOutlookSync/Models/UserSettings.cs @@ -11,6 +11,11 @@ namespace StarfaceOutlookSync.Models public bool SyncOnStart { get; set; } = false; public bool AutoAcceptOutlookPrompt { get; set; } = false; + // Tray-Benachrichtigungen. Allgemein = Info-Meldungen (Sync laeuft/fertig). + // WarnungenFehler = nur Konflikte und Fehler. Beide unabhaengig schaltbar. + public bool NotificationsEnabled { get; set; } = true; + public bool NotifyWarningsErrors { get; set; } = true; + // Gemeinsames Verzeichnis (Netzlaufwerk/UNC) fuer die clientuebergreifende // Sync-Sperre. Leer = keine Sperre (nur lokaler Schutz). Verhindert, dass // mehrere Arbeitsplaetze gleichzeitig dasselbe Adressbuch synchronisieren. diff --git a/src/StarfaceOutlookSync/Services/SyncEngine.cs b/src/StarfaceOutlookSync/Services/SyncEngine.cs index 932a5b9c..f67e91b4 100644 --- a/src/StarfaceOutlookSync/Services/SyncEngine.cs +++ b/src/StarfaceOutlookSync/Services/SyncEngine.cs @@ -15,6 +15,13 @@ namespace StarfaceOutlookSync.Services private void Log(string message) => OnProgress?.Invoke(message); + /// Protokolliert eine tatsaechliche Aenderung (fuer Live-Log UND SyncResult.Changes). + private void Action(SyncResult result, string message) + { + Log(" " + message); + result?.Changes.Add(message); + } + /// /// Setzt die Baseline eines Mappings auf den uebergebenen Stand beider /// Seiten (Snapshot + Hash). Der Snapshot wird fuer das Feld-Merge bei @@ -184,7 +191,7 @@ namespace StarfaceOutlookSync.Services if (await starface.DeleteContactAsync(mapping.StarfaceId)) { result.Updated++; - Log($" Geloescht (OL->SF): {sc.DisplayName}"); + Action(result, $"Geloescht (OL->SF): {sc.DisplayName}"); } continue; } @@ -202,7 +209,7 @@ namespace StarfaceOutlookSync.Services if (await starface.DeleteContactAsync(mapping.StarfaceId)) { result.Updated++; - Log($" Geloescht (OL->SF): {sc.DisplayName}"); + Action(result, $"Geloescht (OL->SF): {sc.DisplayName}"); } continue; } @@ -245,7 +252,7 @@ namespace StarfaceOutlookSync.Services if (_outlookService.DeleteContact(oc.OutlookEntryId)) { result.Updated++; - Log($" Geloescht (SF->OL): {oc.DisplayName}"); + Action(result, $"Geloescht (SF->OL): {oc.DisplayName}"); } continue; } @@ -263,7 +270,7 @@ namespace StarfaceOutlookSync.Services if (_outlookService.DeleteContact(oc.OutlookEntryId)) { result.Updated++; - Log($" Geloescht (SF->OL): {oc.DisplayName}"); + Action(result, $"Geloescht (SF->OL): {oc.DisplayName}"); } continue; } @@ -322,7 +329,7 @@ namespace StarfaceOutlookSync.Services { SetBaseline(mapping, oc, updated); result.Updated++; - Log($" Aktualisiert (OL->SF): {oc.DisplayName}"); + Action(result, $"Aktualisiert (OL->SF): {oc.DisplayName}"); } } else if (sfChanged && !olChanged && (profile.SyncDirection == SyncDirection.Both || profile.SyncDirection == SyncDirection.StarfaceToOutlook)) @@ -333,7 +340,7 @@ namespace StarfaceOutlookSync.Services { SetBaseline(mapping, updated, sc); result.Updated++; - Log($" Aktualisiert (SF->OL): {sc.DisplayName}"); + Action(result, $"Aktualisiert (SF->OL): {sc.DisplayName}"); } } else if (olChanged && sfChanged) @@ -366,9 +373,9 @@ namespace StarfaceOutlookSync.Services mapping.LastSyncHash = ""; result.Updated++; foreach (var cf in conflicts) result.Conflicts.Add(cf); - Log(conflicts.Count > 0 - ? $" Beidseitig geaendert, zusammengefuehrt ({conflicts.Count} Feld-Konflikt(e), Outlook gewinnt): {oc.DisplayName}" - : $" Beidseitig geaendert, zusammengefuehrt: {oc.DisplayName}"); + Action(result, conflicts.Count > 0 + ? $"Beidseitig geaendert, zusammengefuehrt ({conflicts.Count} Feld-Konflikt(e), Outlook gewinnt): {oc.DisplayName}" + : $"Beidseitig geaendert, zusammengefuehrt: {oc.DisplayName}"); } } else if (profile.SyncDirection != SyncDirection.StarfaceToOutlook) @@ -380,7 +387,7 @@ namespace StarfaceOutlookSync.Services { SetBaseline(mapping, oc, updated); result.Updated++; - Log($" Konflikt (OL gewinnt): {oc.DisplayName}"); + Action(result, $"Konflikt (OL gewinnt): {oc.DisplayName}"); } } else @@ -390,7 +397,7 @@ namespace StarfaceOutlookSync.Services { SetBaseline(mapping, updated, sc); result.Updated++; - Log($" Konflikt (SF gewinnt): {sc.DisplayName}"); + Action(result, $"Konflikt (SF gewinnt): {sc.DisplayName}"); } } } @@ -441,7 +448,7 @@ namespace StarfaceOutlookSync.Services processedStarfaceIds.Add(match.StarfaceId); unmappedStarface.Remove(match); result.Updated++; - Log($" Verknuepft (OL->SF): {oc.DisplayName}"); + Action(result, $"Verknuepft (OL->SF): {oc.DisplayName}"); } } else @@ -462,7 +469,7 @@ namespace StarfaceOutlookSync.Services LastStarfaceHash = created.GetHash() }); result.Created++; - Log($" Erstellt (OL->SF): {oc.DisplayName}"); + Action(result, $"Erstellt (OL->SF): {oc.DisplayName}"); } else { @@ -521,7 +528,7 @@ namespace StarfaceOutlookSync.Services processedOutlookIds.Add(match.OutlookEntryId); unmappedOutlook.Remove(match); result.Updated++; - Log($" Verknuepft (SF->OL): {sc.DisplayName}"); + Action(result, $"Verknuepft (SF->OL): {sc.DisplayName}"); } } else @@ -541,7 +548,7 @@ namespace StarfaceOutlookSync.Services LastOutlookHash = created.GetHash() }); result.Created++; - Log($" Erstellt (SF->OL): {sc.DisplayName}"); + Action(result, $"Erstellt (SF->OL): {sc.DisplayName}"); } } } @@ -581,7 +588,7 @@ namespace StarfaceOutlookSync.Services if (await starface.DeleteContactAsync(sc.StarfaceId)) { result.Updated++; - Log($" Geloescht (nur in Starface): {sc.DisplayName}"); + Action(result, $"Geloescht (nur in Starface): {sc.DisplayName}"); } } catch (Exception ex) @@ -612,7 +619,7 @@ namespace StarfaceOutlookSync.Services if (_outlookService.DeleteContact(oc.OutlookEntryId)) { result.Updated++; - Log($" Geloescht (nur in Outlook): {oc.DisplayName}"); + Action(result, $"Geloescht (nur in Outlook): {oc.DisplayName}"); } } catch (Exception ex) diff --git a/src/StarfaceOutlookSync/UI/MainForm.cs b/src/StarfaceOutlookSync/UI/MainForm.cs index 063df882..5851803e 100644 --- a/src/StarfaceOutlookSync/UI/MainForm.cs +++ b/src/StarfaceOutlookSync/UI/MainForm.cs @@ -28,6 +28,10 @@ namespace StarfaceOutlookSync.UI // ein manueller Sync und der Auto-Sync-Timer nicht gleichzeitig starten. private int _syncRunning = 0; + // Aktive Benachrichtigungs-Einstellung (pro Sync-Lauf gesetzt). + private bool _notifyGeneral = true; + private bool _notifyWarn = true; + public MainForm() { InitializeComponent(); @@ -353,7 +357,10 @@ namespace StarfaceOutlookSync.UI } SyncLock crossLock = null; - var sharedDir = UserSettings.Load().SharedDirectory; + var settings = UserSettings.Load(); + var sharedDir = settings.SharedDirectory; + _notifyGeneral = settings.NotificationsEnabled; + _notifyWarn = settings.NotifyWarningsErrors; try { // Clientuebergreifende Sperre (falls gemeinsames Verzeichnis konfiguriert). @@ -373,8 +380,7 @@ namespace StarfaceOutlookSync.UI ShowRemoteConflictNotices(sharedDir); SetStatus($"Synchronisiere '{profile.Name}'..."); - _trayIcon.ShowBalloonTip(2000, "Starface Sync", - $"Synchronisiere '{profile.Name}'...", ToolTipIcon.Info); + Balloon(2000, "Starface Sync", $"Synchronisiere '{profile.Name}'...", ToolTipIcon.Info, warn: false); var result = await _syncEngine.SyncProfileAsync(profile); @@ -382,10 +388,15 @@ namespace StarfaceOutlookSync.UI if (result.Errors > 0) msg += $", {result.Errors} Fehler"; if (result.Conflicts.Count > 0) msg += $", {result.Conflicts.Count} Konflikt(e)"; - _trayIcon.ShowBalloonTip(3000, "Starface Sync", msg, - result.Errors > 0 ? ToolTipIcon.Warning : ToolTipIcon.Info); + // Zusammenfassung als Warnung werten, wenn Fehler/Konflikte auftraten. + bool noteworthy = result.Errors > 0 || result.Conflicts.Count > 0; + Balloon(3000, "Starface Sync", msg, + result.Errors > 0 ? ToolTipIcon.Warning : ToolTipIcon.Info, warn: noteworthy); + // Protokoll (immer, unabhaengig von Benachrichtigungs-Einstellung). Logger.Log($"Sync fertig: {msg}"); + foreach (var ch in result.Changes) + Logger.Log($" {ch}"); foreach (var em in result.ErrorMessages) Logger.Log($" Fehler: {em}"); @@ -396,8 +407,7 @@ namespace StarfaceOutlookSync.UI var detail = string.Join("\n", result.Conflicts.Take(5).Select(c => c.ToString())); if (result.Conflicts.Count > 5) detail += $"\n... und {result.Conflicts.Count - 5} weitere"; - _trayIcon.ShowBalloonTip(10000, - $"Konflikt bei {result.Conflicts.Count} Kontakt(en)", detail, ToolTipIcon.Warning); + Balloon(10000, $"Konflikt bei {result.Conflicts.Count} Kontakt(en)", detail, ToolTipIcon.Warning, warn: true); foreach (var c in result.Conflicts) Logger.Log($" KONFLIKT: {c}"); @@ -410,8 +420,7 @@ namespace StarfaceOutlookSync.UI } catch (Exception ex) { - _trayIcon.ShowBalloonTip(3000, "Starface Sync Fehler", - ex.Message, ToolTipIcon.Error); + Balloon(3000, "Starface Sync Fehler", ex.Message, ToolTipIcon.Error, warn: true); SetStatus($"Fehler: {ex.Message}"); Logger.Log($"Sync FEHLER '{profile.Name}': {ex.Message}"); } @@ -481,8 +490,8 @@ namespace StarfaceOutlookSync.UI var detail = string.Join("\n", pending.Take(5).Select(p => p.ToString())); if (pending.Count > 5) detail += $"\n... und {pending.Count - 5} weitere"; - _trayIcon.ShowBalloonTip(10000, - $"Konflikt an anderem Arbeitsplatz ({pending.Count})", detail, ToolTipIcon.Warning); + Balloon(10000, + $"Konflikt an anderem Arbeitsplatz ({pending.Count})", detail, ToolTipIcon.Warning, warn: true); } catch { } } @@ -506,6 +515,17 @@ namespace StarfaceOutlookSync.UI return map; } + /// + /// Zeigt eine Tray-Meldung - aber nur, wenn die passende Benachrichtigungs- + /// Einstellung aktiv ist. warn=true -> Warnungen/Fehler (Konflikte, Fehler), + /// warn=false -> allgemeine Info-Meldungen. + /// + private void Balloon(int ms, string title, string text, ToolTipIcon icon, bool warn) + { + bool allow = warn ? _notifyWarn : _notifyGeneral; + if (allow) _trayIcon.ShowBalloonTip(ms, title, text, icon); + } + private void SetStatus(string text) { if (InvokeRequired) diff --git a/src/StarfaceOutlookSync/UI/SettingsForm.cs b/src/StarfaceOutlookSync/UI/SettingsForm.cs index c11be357..6c25d408 100644 --- a/src/StarfaceOutlookSync/UI/SettingsForm.cs +++ b/src/StarfaceOutlookSync/UI/SettingsForm.cs @@ -7,6 +7,7 @@ namespace StarfaceOutlookSync.UI public class SettingsForm : Form { private CheckBox _chkStartMinimized, _chkSyncOnStart, _chkAutoAcceptOutlook; + private CheckBox _chkNotifGeneral, _chkNotifWarn; private TextBox _txtSharedDir; private Button _btnBrowseShared; private Button _btnSave, _btnCancel; @@ -61,21 +62,35 @@ namespace StarfaceOutlookSync.UI Font = new Font("Segoe UI", 8) }; + _chkNotifGeneral = new CheckBox + { + Text = "Tray-Benachrichtigungen (allgemein)", + Left = 20, Top = 144, AutoSize = true, + Checked = _settings.NotificationsEnabled + }; + + _chkNotifWarn = new CheckBox + { + Text = "Benachrichtigungen bei Warnungen/Fehlern (Konflikte, Fehler)", + Left = 20, Top = 170, AutoSize = true, + Checked = _settings.NotifyWarningsErrors + }; + var lblShared = new Label { Text = "Gemeinsames Verzeichnis fuer Sync-Sperre (Mehrplatz, optional):", - Left = 20, Top = 150, AutoSize = true + Left = 20, Top = 206, AutoSize = true }; _txtSharedDir = new TextBox { - Left = 20, Top = 172, Width = 250, + Left = 20, Top = 228, Width = 250, Text = _settings.SharedDirectory }; _btnBrowseShared = new Button { - Text = "...", Left = 274, Top = 171, Width = 36, Height = 24 + Text = "...", Left = 274, Top = 227, Width = 36, Height = 24 }; _btnBrowseShared.Click += (s, e) => BrowseSharedDir(); @@ -83,25 +98,26 @@ namespace StarfaceOutlookSync.UI { Text = "Netzlaufwerk/UNC, das alle Arbeitsplaetze erreichen. Leer = keine\n" + "clientuebergreifende Sperre (nur Schutz auf diesem PC).", - Left = 20, Top = 198, Width = 330, Height = 32, + Left = 20, Top = 254, Width = 330, Height = 32, ForeColor = Color.Gray, Font = new Font("Segoe UI", 8) }; _btnSave = new Button { - Text = "Speichern", Left = 95, Top = 240, Width = 85, Height = 28, + Text = "Speichern", Left = 95, Top = 300, Width = 85, Height = 28, DialogResult = DialogResult.None }; _btnSave.Click += (s, e) => Save(); _btnCancel = new Button { - Text = "Abbrechen", Left = 189, Top = 240, Width = 85, Height = 28, + Text = "Abbrechen", Left = 189, Top = 300, Width = 85, Height = 28, DialogResult = DialogResult.Cancel }; - Size = new Size(380, 330); + Size = new Size(380, 390); Controls.AddRange(new Control[] { _chkStartMinimized, _chkSyncOnStart, _chkAutoAcceptOutlook, lblHint, + _chkNotifGeneral, _chkNotifWarn, lblShared, _txtSharedDir, _btnBrowseShared, lblSharedHint, _btnSave, _btnCancel }); AcceptButton = _btnSave; CancelButton = _btnCancel; @@ -126,6 +142,8 @@ namespace StarfaceOutlookSync.UI _settings.StartMinimized = _chkStartMinimized.Checked; _settings.SyncOnStart = _chkSyncOnStart.Checked; _settings.AutoAcceptOutlookPrompt = _chkAutoAcceptOutlook.Checked; + _settings.NotificationsEnabled = _chkNotifGeneral.Checked; + _settings.NotifyWarningsErrors = _chkNotifWarn.Checked; _settings.SharedDirectory = _txtSharedDir.Text.Trim(); _settings.Save(); DialogResult = DialogResult.OK; diff --git a/src/StarfaceOutlookSync/UI/SyncProgressForm.cs b/src/StarfaceOutlookSync/UI/SyncProgressForm.cs index 381dffd8..da0f14e2 100644 --- a/src/StarfaceOutlookSync/UI/SyncProgressForm.cs +++ b/src/StarfaceOutlookSync/UI/SyncProgressForm.cs @@ -94,6 +94,8 @@ namespace StarfaceOutlookSync.UI _engine.OnProgress += AppendLog; + Logger.Log($"Sync gestartet (manuell): '{_profile.Name}' (Richtung: {_profile.SyncDirection})"); + try { var result = await Task.Run(() => _engine.SyncProfileAsync(_profile)); @@ -105,11 +107,23 @@ namespace StarfaceOutlookSync.UI _lblResult.Text = resultText; _lblResult.ForeColor = result.Errors > 0 ? Color.OrangeRed : Color.Green; + // Persistentes Protokoll (was wurde geaendert). + Logger.Log($"Sync fertig: {_profile.Name}: {result.Created} erstellt, {result.Updated} aktualisiert" + + (result.Errors > 0 ? $", {result.Errors} Fehler" : "") + + (result.Conflicts.Count > 0 ? $", {result.Conflicts.Count} Konflikt(e)" : "")); + foreach (var ch in result.Changes) + Logger.Log($" {ch}"); + foreach (var c in result.Conflicts) + Logger.Log($" KONFLIKT: {c}"); + if (result.ErrorMessages.Count > 0) { AppendLog("--- Fehler ---"); foreach (var err in result.ErrorMessages) + { AppendLog(err); + Logger.Log($" Fehler: {err}"); + } } } catch (Exception ex) @@ -117,6 +131,7 @@ namespace StarfaceOutlookSync.UI _lblResult.Text = $"Fehler: {ex.Message}"; _lblResult.ForeColor = Color.Red; AppendLog($"FEHLER: {ex.Message}"); + Logger.Log($"Sync FEHLER '{_profile.Name}': {ex.Message}"); } _engine.OnProgress -= AppendLog;