[chore:] logging for ResultWindow.axaml.cs

DataStore/Settings.cs

@@ -45,9 +45,8 @@ public class Settings
         }
         catch (Exception ex)
         {
-            Console.WriteLine(ex.Message);
+            Logger.Log($"Error while reading settings. Generating new... {ex.Message}", Logger.LogType.Warning);
-            Console.WriteLine(ex.StackTrace);
+
-            Console.WriteLine("Error while reading settings. Generating new...");
             _instance = new Settings();
         }
     }
@@ -123,9 +122,7 @@ public class Global
         }
         catch (Exception ex)
         {
-            Console.WriteLine(ex.Message);
+            Logger.Log($"Error while reading global settings. Generating new... {ex.Message}", Logger.LogType.Warning);
-            Console.WriteLine(ex.StackTrace);
-            Console.WriteLine("Error while reading global settings. Generating new...");
             _instance = new Global();
             Save();
         }

Logger.cs

@@ -0,0 +1,40 @@
+using System;
+using System.IO;
+using Newtonsoft.Json;
+
+namespace Logof_Client;
+
+public static class Logger
+{
+    public static void Log(string text, LogType logType = LogType.Info)
+    {
+        try
+        {
+            string config_path = Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData),
+                "logofclient");
+            if (!Directory.Exists(config_path))
+                Directory.CreateDirectory(config_path);
+
+            string log_path = Path.Combine(config_path, $"log-{DateTime.Now:dd-MM-yy}.log");
+            if(!File.Exists(log_path))
+                File.Create(log_path).Close();
+
+            string line = $"[{DateTime.Now:dd.MM.yyyy - hh:mm:ss}]: ({logType.ToString()}) {text}";
+            Console.WriteLine(line);
+            
+            File.AppendAllLines(log_path, [line]);
+        }
+        catch (Exception ex)
+        {
+            Console.WriteLine(ex);
+        }
+        
+    }
+
+    public enum LogType
+    {
+        Error,
+        Warning,
+        Info
+    }
+}

MainWindow.axaml.cs

@@ -24,6 +24,9 @@ public partial class MainWindow : Window
 
     public MainWindow()
     {
+        Logger.Log("Welcome to logofclient");
+        Logger.Log($"Session on {DateTime.Now:G}");
+        Logger.Log("Initializing...");
         InitializeComponent();
         //Hide();
         
@@ -32,6 +35,7 @@ public partial class MainWindow : Window
 
         _instance = this;
         WindowState = WindowState.Maximized;
+        Logger.Log("Loading settings...");
         Global.Load();
         Settings.Load();
         LoadPdfExportOptions();
@@ -40,6 +44,7 @@ public partial class MainWindow : Window
         RefreshCountryView();
 
         // Initialize wiki integration
+        Logger.Log("Building wiki...");
         _wikiService = new WikiService();
         try
         {
@@ -54,8 +59,9 @@ public partial class MainWindow : Window
                 if (!string.IsNullOrWhiteSpace(Global._instance.wiki_storage_path))
                     TbWikiPath.Text = Global._instance.wiki_storage_path;
             }
-            catch
+            catch (Exception ex)
             {
+                Logger.Log($"Error while setting paths: {ex.Message}", Logger.LogType.Error);
             }
 
             try
@@ -64,12 +70,14 @@ public partial class MainWindow : Window
                 BtnFontPath.Click += BtnFontPath_Click;
                 BtnConfigPath.Click += BtnConfigPath_Click;
             }
-            catch
+            catch (Exception ex)
             {
+                Logger.Log($"Error while adding click functions?: {ex.Message}", Logger.LogType.Error);
             }
         }
-        catch
+        catch (Exception ex)
         {
+            Logger.Log($"Error while doing wiki stuff: {ex.Message}", Logger.LogType.Error);
         }
 
         //Thread.Sleep(3000);
@@ -121,23 +129,6 @@ public partial class MainWindow : Window
         //await MessageBox.Show(_instance, $"{result.Count} Einträge fehlerhaft.", "Fertig");
     }
     
-    // private async void StartAddressRepair(Uri path)
-    // {
-    //     var addresses = DataImport.ImportKasAddressList(path); // Ihr Code hier
-    //     var progressWindow = new ProgressWindow();
-    //
-    //     progressWindow.Show(_instance);
-    //
-    //     var processor = new AddressRepair(progressWindow);
-    //     //var result = await processor.Perform(addresses.Item2, errors);
-    //
-    //
-    //     progressWindow.Close();
-    //
-    //
-    //     //new ResultWindow(result, addresses.Item2).Show();
-    //     //await MessageBox.Show(_instance, $"{result.Count} Einträge fehlerhaft.", "Fertig");
-    // }
 
     private void MnuExit_OnClick(object? sender, RoutedEventArgs e)
     {
@@ -156,7 +147,7 @@ public partial class MainWindow : Window
         }
         catch (Exception ex)
         {
-            Console.WriteLine($"Fehler beim Öffnen des Links: {ex.Message}");
+            Logger.Log($"Error while starting process: {ex.Message}", Logger.LogType.Error);
         }
     }
 
@@ -172,7 +163,7 @@ public partial class MainWindow : Window
         }
         catch (Exception ex)
         {
-            Console.WriteLine($"Fehler beim Öffnen des Links: {ex.Message}");
+            Logger.Log($"Error while starting process: {ex.Message}", Logger.LogType.Error);
         }
     }
 
@@ -188,7 +179,7 @@ public partial class MainWindow : Window
         }
         catch (Exception ex)
         {
-            Console.WriteLine($"Fehler beim Öffnen des Links: {ex.Message}");
+            Logger.Log($"Error while starting process: {ex.Message}", Logger.LogType.Error);
         }
     }
 
@@ -232,9 +223,9 @@ public partial class MainWindow : Window
                 var rendered = MarkdownRenderer.Render(text ?? string.Empty);
                 PreviewPanel.Children.Add(rendered);
             }
-            catch
+            catch (Exception ex)
             {
-                // fallback: plain text
+                Logger.Log($"Error while rendering markdown: {ex.Message}", Logger.LogType.Error);
                 PreviewPanel.Children.Clear();
                 PreviewPanel.Children.Add(new TextBlock { Text = text ?? string.Empty });
             }
@@ -250,18 +241,24 @@ public partial class MainWindow : Window
 
     public void PopulateNavTree(string? expandToPath = null, string? selectPath = null)
     {
-        var roots = _wikiService.GetRootItems();
+        try
-        var nodes = new List<TreeViewItem>();
+        {
-        foreach (var r in roots) nodes.Add(BuildNode(r));
+            var roots = _wikiService.GetRootItems();
-        NavTree.ItemsSource = nodes;
+            var nodes = new List<TreeViewItem>();
+            foreach (var r in roots) nodes.Add(BuildNode(r));
+            NavTree.ItemsSource = nodes;
 
-        if (!string.IsNullOrWhiteSpace(expandToPath))
+            if (!string.IsNullOrWhiteSpace(expandToPath))
-            ExpandAndFindNode(nodes, expandToPath, out _);
+                ExpandAndFindNode(nodes, expandToPath, out _);
 
-        if (!string.IsNullOrWhiteSpace(selectPath) &&
+            if (!string.IsNullOrWhiteSpace(selectPath) &&
-            ExpandAndFindNode(nodes, selectPath, out var selectedNode) &&
+                ExpandAndFindNode(nodes, selectPath, out var selectedNode) &&
-            selectedNode != null)
+                selectedNode != null)
-            NavTree.SelectedItem = selectedNode;
+                NavTree.SelectedItem = selectedNode;
+        }catch (Exception ex)
+        {
+            Logger.Log($"Error while populating nav tree: {ex.Message}", Logger.LogType.Error);
+        }
     }
 
     private TreeViewItem BuildNode(WikiItem item)
@@ -276,16 +273,23 @@ public partial class MainWindow : Window
 
     private string GetSelectedWikiTargetDirectory()
     {
-        var wikiRoot = Global._instance.wiki_storage_path;
+        try
-        if (NavTree.SelectedItem is TreeViewItem treeItem && treeItem.Tag is WikiItem selectedItem)
         {
-            if (selectedItem.IsFolder) return selectedItem.Path;
+            var wikiRoot = Global._instance.wiki_storage_path;
+            if (NavTree.SelectedItem is TreeViewItem treeItem && treeItem.Tag is WikiItem selectedItem)
+            {
+                if (selectedItem.IsFolder) return selectedItem.Path;
 
-            var parentDir = Path.GetDirectoryName(selectedItem.Path);
+                var parentDir = Path.GetDirectoryName(selectedItem.Path);
-            if (!string.IsNullOrWhiteSpace(parentDir)) return parentDir;
+                if (!string.IsNullOrWhiteSpace(parentDir)) return parentDir;
+            } 
+            return wikiRoot;
+        }catch (Exception ex)
+        {
+            Logger.Log($"Error while getting selected wiki target directory: {ex.Message}", Logger.LogType.Error);
         }
 
-        return wikiRoot;
+        return null;
     }
 
     private static bool PathsEqual(string left, string right)
@@ -344,8 +348,9 @@ public partial class MainWindow : Window
                 UseShellExecute = true
             });
         }
-        catch
+        catch (Exception ex)
         {
+            Logger.Log($"Error while starting process: {ex.Message}", Logger.LogType.Error);
         }
     }
 
@@ -443,8 +448,9 @@ public partial class MainWindow : Window
         {
             LstSettingsCustomers.SelectedIndex = LstSettingsCustomers.Items.Count - 1;
         }
-        catch
+        catch (Exception ex)
         {
+            Logger.Log($"Error while setting selected item: {ex.Message}", Logger.LogType.Error);
         }
     }
 
@@ -515,17 +521,25 @@ public partial class MainWindow : Window
         {
             LstSettingsCustomers.SelectedIndex = index;
         }
-        catch
+        catch (Exception ex)
         {
+            Logger.Log($"Error while setting selected index: {ex.Message}", Logger.LogType.Error);
         }
     }
 
     private void TbSettingsCustomerDescription_OnTextChanged(object? sender, TextChangedEventArgs e)
     {
-        if (LstSettingsCustomers.SelectedIndex == null || LstSettingsCustomers.SelectedIndex == -1) return;
+        try
-        foreach (var customer in Settings._instance.customers.customers)
+        {
-            if (customer.ID == Settings._instance.customers.current.ID)
+            if (LstSettingsCustomers.SelectedIndex == null || LstSettingsCustomers.SelectedIndex == -1) return;
-                customer.description = TbSettingsCustomerDescription.Text;
+                    foreach (var customer in Settings._instance.customers.customers)
+                        if (customer.ID == Settings._instance.customers.current.ID)
+                            customer.description = TbSettingsCustomerDescription.Text;
+        } catch (Exception ex)
+        {
+            Logger.Log($"Error while changing customer description: {ex.Message}", Logger.LogType.Error);
+        }
+        
         //Settings.Save();
     }
 
@@ -545,63 +559,72 @@ public partial class MainWindow : Window
 
     private async void BtnCustomerAddressSetImport_OnClick(object? sender, RoutedEventArgs e)
     {
-        MakeCalcManVisible();
+        try
-        var opts = new FilePickerOpenOptions();
+        {
-        opts.Title = "Address-Set importieren...";
+            MakeCalcManVisible();
-        opts.AllowMultiple = false;
+             var opts = new FilePickerOpenOptions();
+             opts.Title = "Address-Set importieren...";
+             opts.AllowMultiple = false;
      
-        var type = new FilePickerFileType("CSV-Dateien (*.csv)");
+             var type = new FilePickerFileType("CSV-Dateien (*.csv)");
-        type.Patterns = new[] { "*.csv" };
+             type.Patterns = new[] { "*.csv" };
-        opts.FileTypeFilter = new[] { type };
+             opts.FileTypeFilter = new[] { type };
      
-        var paths = await StorageProvider.OpenFilePickerAsync(opts);
+             var paths = await StorageProvider.OpenFilePickerAsync(opts);
      
-        if (paths?.Count <= 0) return;
+             if (paths?.Count <= 0) return;
      
-        if (LstCustomers.SelectedIndex < 0) return;
+             if (LstCustomers.SelectedIndex < 0) return;
      
      
-        var selected_path = paths[0].Path;
+             var selected_path = paths[0].Path;
      
-        foreach (var customer in Settings._instance.customers.customers)
+             foreach (var customer in Settings._instance.customers.customers)
-            if (customer == ((Customer)LstCustomers.SelectedItems[0]))
+                 if (customer == ((Customer)LstCustomers.SelectedItems[0]))
-            {
+                 {
-                if (customer.patch == null)
+                     if (customer.patch == null)
-                {
+                     {
-                    var got = await DataImport.ImportKasAddressList(selected_path, null, customer.separator);
+                         var got = await DataImport.ImportKasAddressList(selected_path, null, customer.separator);
-                    if (!got.Item1)
+                         if (!got.Item1)
-                    {
+                         {
-                        Console.WriteLine("Error while importing. Please try another file.");
+                             Console.WriteLine("Error while importing. Please try another file.");
-                    }
+                         }
-                    else
+                         else
-                    {
+                         {
-                        got.Item2.SetOwner(customer.ID);
+                             got.Item2.SetOwner(customer.ID);
-                        Settings._instance.addressSets.addresses.Add(got.Item2);
+                             Settings._instance.addressSets.addresses.Add(got.Item2);
-                    }
+                         }
      
-                    //var customer_id = int.Parse(LstCustomers.SelectedItem.ToString().Split(" - ")[0]);
+                         //var customer_id = int.Parse(LstCustomers.SelectedItem.ToString().Split(" - ")[0]);
-                    RefreshAddressSetListItems(customer.ID);
+                         RefreshAddressSetListItems(customer.ID);
-                }
+                     }
-                else
+                     else
-                {
+                     {
-                    var got = await DataImport.ImportKasAddressList(selected_path, customer.patch, customer.separator);
+                         var got = await DataImport.ImportKasAddressList(selected_path, customer.patch, customer.separator);
-                    if (!got.Item1)
+                         if (!got.Item1)
-                    {
+                         {
-                        Console.WriteLine("Error while importing. Please try another file.");
+                             Console.WriteLine("Error while importing. Please try another file.");
-                    }
+                         }
-                    else
+                         else
-                    {
+                         {
-                        got.Item2.SetOwner(customer.ID);
+                             got.Item2.SetOwner(customer.ID);
-                        Settings._instance.addressSets.addresses.Add(got.Item2);
+                             Settings._instance.addressSets.addresses.Add(got.Item2);
-                    }
+                         }
      
-                    //var customer_id = int.Parse(LstCustomers.SelectedItem.ToString().Split(" - ")[0]);
+                         //var customer_id = int.Parse(LstCustomers.SelectedItem.ToString().Split(" - ")[0]);
-                    RefreshAddressSetListItems(customer.ID);
+                         RefreshAddressSetListItems(customer.ID);
-                }
+                     }
-            }
+                 }
      
      
-        Settings.Save();
+             Settings.Save();
+            
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while importing: {ex.Message}", Logger.LogType.Error);
+        }
+        
     }
 
     public void RefreshAddressSetListItems(int customer_id)
@@ -647,31 +670,38 @@ public partial class MainWindow : Window
 
     private async void BtnSettingsImportCustomerAddressPatch_OnClick(object? sender, RoutedEventArgs e)
     {
-        if (LstSettingsCustomers.SelectedIndex == null || LstSettingsCustomers.SelectedIndex == -1) return;
+        try
+        {
+            if (LstSettingsCustomers.SelectedIndex == null || LstSettingsCustomers.SelectedIndex == -1) return;
             
-        var opts = new FilePickerOpenOptions();
+            var opts = new FilePickerOpenOptions();
-        opts.Title = "Address-Patch für " + LstSettingsCustomers.SelectedItems[0] + "importieren...";
+            opts.Title = "Address-Patch für " + LstSettingsCustomers.SelectedItems[0] + "importieren...";
-        opts.AllowMultiple = false;
+            opts.AllowMultiple = false;
     
-        var type = new FilePickerFileType("ADPAC-Dateien (*.adpac)");
+            var type = new FilePickerFileType("ADPAC-Dateien (*.adpac)");
-        type.Patterns = new[] { "*.adpac" };
+            type.Patterns = new[] { "*.adpac" };
-        opts.FileTypeFilter = new[] { type };
+            opts.FileTypeFilter = new[] { type };
     
-        var paths = await StorageProvider.OpenFilePickerAsync(opts);
+            var paths = await StorageProvider.OpenFilePickerAsync(opts);
     
-        if (paths?.Count <= 0) return;
+            if (paths?.Count <= 0) return;
     
-        //if (LstSettingsCustomers.SelectedIndex < 0) return;
+            //if (LstSettingsCustomers.SelectedIndex < 0) return;
     
     
-        var selected_path = paths[0].Path;
+            var selected_path = paths[0].Path;
     
-        foreach (var customer in Settings._instance.customers.customers)
+            foreach (var customer in Settings._instance.customers.customers)
-            if (customer.ID == ((Customer)LstSettingsCustomers.SelectedItems[0]).ID)
+                if (customer.ID == ((Customer)LstSettingsCustomers.SelectedItems[0]).ID)
-                customer.patch = AddressPatch.Import(selected_path);
+                    customer.patch = AddressPatch.Import(selected_path);
     
     
-        Settings.Save();
+            Settings.Save();
+        } catch (Exception ex)
+        {
+            Logger.Log($"Error while importing customer address patch: {ex.Message}", Logger.LogType.Error);
+        }
+        
     }
 
     private void TbSettingsCustomerCsvSeparator_OnTextChanged(object? sender, TextChangedEventArgs e)
@@ -689,11 +719,14 @@ public partial class MainWindow : Window
                 catch (FormatException ex)
                 {
                     MessageBox.Show(this, "Error while converting: " + ex.Message, "Could not parse");
+                    Logger.Log($"Error while converting: {ex.Message}", Logger.LogType.Error);
                 }
                 catch (Exception ex)
                 {
                     MessageBox.Show(this, "Unknown Error: " + ex.Message, "Error");
+                    Logger.Log($"Error while converting: {ex.Message}", Logger.LogType.Error);
                 }
+
     }
 
     private void BtnCombineDifference_OnClick(object? sender, RoutedEventArgs e)
@@ -704,10 +737,9 @@ public partial class MainWindow : Window
         try
         {
             StartCombine(list, Convert.ToInt32(LstCustomers.SelectedItem), "difference", GetCombiningTyp());
-        }
+        }catch (Exception ex)
-        catch
         {
-            
+            Logger.Log($"Error while combining (difference): {ex.Message}", Logger.LogType.Error);
         }
     }
 
@@ -720,9 +752,9 @@ public partial class MainWindow : Window
         {
             StartCombine(list, Convert.ToInt32((LstCustomers.SelectedItem as Customer).ID), "union", GetCombiningTyp());
         }
-        catch
+        catch (Exception ex)
         {
-            
+            Logger.Log($"Error while combining (union): {ex.Message}", Logger.LogType.Error);
         }
         
     }
@@ -736,9 +768,9 @@ public partial class MainWindow : Window
         {
             StartCombine(list, Convert.ToInt32(LstCustomers.SelectedItem.ToString().Split(" - ")[0]), "intersection", GetCombiningTyp());
         }
-        catch
+        catch (Exception ex)
         {
-                
+            Logger.Log($"Error while combining (intersection): {ex.Message}", Logger.LogType.Error);
         }
     }
 
@@ -774,52 +806,94 @@ public partial class MainWindow : Window
 
     private async void BtnStartGenerateLabels_OnClick(object? sender, RoutedEventArgs e)
     {
-        SavePdfExportOptions();
+        try
-
-        var saveDialog = new SaveFileDialog
         {
-            DefaultExtension = "pdf",
+            SavePdfExportOptions();
-            Filters = { new FileDialogFilter { Name = "PDF-Dateien", Extensions = { "pdf" } } }
-        };
-        var filePath = await saveDialog.ShowAsync(this);
 
-        if (!string.IsNullOrEmpty(filePath))
+            var saveDialog = new SaveFileDialog
+            {
+                DefaultExtension = "pdf",
+                Filters =
+                {
+                    new FileDialogFilter
+                    {
+                        Name = "PDF-Dateien",
+                        Extensions = { "pdf" }
+                    }
+                }
+            };
+
+            var filePath = await saveDialog.ShowAsync(this);
+
+            Console.WriteLine($"RAW: {filePath}");
+
+            if (!string.IsNullOrWhiteSpace(filePath))
+            {
+                if (filePath.StartsWith("file://"))
+                {
+                    filePath = new Uri(filePath).LocalPath;
+                }
+
+                Console.WriteLine($"PATH: {filePath}");
+
+                var builder = new PdfBuilder(Settings._instance.pdfExport);
+
+                builder.CreateAddressLabelPdfFromAddressSetWithPlaceholder(
+                    ((KasAddressList)LstCustomerAdressSets.SelectedItem).ID,
+                    "Company Logo/Info",
+                    filePath
+                );
+
+                Console.WriteLine("PDF OK");
+            }
+        }
+
+        catch (Exception ex)
         {
-            var builder = new PdfBuilder(Settings._instance.pdfExport);
+            Logger.Log($"Error while generating labels: {ex.Message}", Logger.LogType.Error);
-
-            builder.CreateAddressLabelPdfFromAddressSetWithPlaceholder((
-                    (KasAddressList)LstCustomerAdressSets.SelectedItem).ID,
-                "Company Logo/Info",
-                filePath
-            );
-            //return true;
         }
     }
 
     private void HookPdfExportOptionEvents()
     {
-        var controls = GetPdfExportControls();
+        try
-        foreach (var control in controls) control.ValueChanged += PdfExportOption_OnValueChanged;
+        {
+            var controls = GetPdfExportControls();
+            foreach (var control in controls) control.ValueChanged += PdfExportOption_OnValueChanged;
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while hooking pdf export margins: {ex.Message}", Logger.LogType.Error);
+        }
+
     }
 
     private void LoadPdfExportOptions()
     {
-        var options = Settings._instance.pdfExport ?? new PdfExportSettings();
+        try
+        {
+            var options = Settings._instance.pdfExport ?? new PdfExportSettings();
 
-        NudExpMargCellPaddingTop.Value = (decimal)options.cellPaddingTopMm;
+            NudExpMargCellPaddingTop.Value = (decimal)options.cellPaddingTopMm;
-        NudExpMargCellPaddingBot.Value = (decimal)options.cellPaddingBottomMm;
+            NudExpMargCellPaddingBot.Value = (decimal)options.cellPaddingBottomMm;
-        NudExpMargCellPaddingLeft.Value = (decimal)options.cellPaddingLeftMm;
+            NudExpMargCellPaddingLeft.Value = (decimal)options.cellPaddingLeftMm;
-        NudExpMargCellPaddingRight.Value = (decimal)options.cellPaddingRightMm;
+            NudExpMargCellPaddingRight.Value = (decimal)options.cellPaddingRightMm;
 
-        TbExpMargMarginTop.Value = (decimal)options.pageMarginTopMm;
+            TbExpMargMarginTop.Value = (decimal)options.pageMarginTopMm;
-        TbExpMargMarginBottom.Value = (decimal)options.pageMarginBottomMm;
+            TbExpMargMarginBottom.Value = (decimal)options.pageMarginBottomMm;
-        TbExpMargMarginLeft.Value = (decimal)options.pageMarginLeftMm;
+            TbExpMargMarginLeft.Value = (decimal)options.pageMarginLeftMm;
-        TbExpMargMarginRight.Value = (decimal)options.pageMarginRightMm;
+            TbExpMargMarginRight.Value = (decimal)options.pageMarginRightMm;
+
+            TbExpMargRowsPerPage.Value = options.rowsPerPage;
+            NudExpMargColumnsPerPage.Value = options.columnsPerPage;
+            NudExpMargFontSize.Value = (decimal)options.fontSize;
+            NudExpMargSmallFontSize.Value = (decimal)options.smallFontSize;
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while loading PDF Export Options: {ex.Message}", Logger.LogType.Error);
+        }
         
-        TbExpMargRowsPerPage.Value = options.rowsPerPage;
-        NudExpMargColumnsPerPage.Value = options.columnsPerPage;
-        NudExpMargFontSize.Value = (decimal)options.fontSize;
-        NudExpMargSmallFontSize.Value = (decimal)options.smallFontSize;
     }
 
     private void SavePdfExportOptions()
@@ -900,6 +974,7 @@ public partial class MainWindow : Window
 
     public void RefreshCountryView()
     {
+        
         if (_suppressCountryRefresh) return;
         try
         {
@@ -914,8 +989,9 @@ public partial class MainWindow : Window
             {
                 CountryList.Items.Clear();
             }
-            catch
+            catch (Exception ex)
             {
+                Logger.Log($"Error while clearing country list: {ex.Message}", Logger.LogType.Error);
             }
 
             foreach (var c in Global._instance.countries) CountryList.Items.Add(c.name);
@@ -923,17 +999,19 @@ public partial class MainWindow : Window
             {
                 CountryList.SelectedIndex = country_index;
             }
-            catch
+            catch (Exception ex)
             {
+                Logger.Log($"Error while setting selected item: {ex.Message}", Logger.LogType.Error);
             }
 
             try
             {
                 _selectedCountry = Country.GetByName(CountryList.SelectedItems[0].ToString());
             }
-            catch
+            catch (Exception ex)
             {
                 _selectedCountry = null;
+                Logger.Log($"Error while getting country: {ex.Message}", Logger.LogType.Error);
             }
 
             if (_selectedCountry == null) return;
@@ -947,8 +1025,9 @@ public partial class MainWindow : Window
             {
                 LbSettingsAlternatives.SelectedIndex = alt_index;
             }
-            catch
+            catch (Exception ex)
             {
+                Logger.Log($"Error while setting selected item: {ex.Message}", Logger.LogType.Error);
             }
         }
         finally
@@ -964,9 +1043,9 @@ public partial class MainWindow : Window
             {
                 _selectedCountry.alternatives.Remove(selected.ToString());
             }
-            catch
+            catch (Exception ex)
             {
-                Console.WriteLine("Error while removing country alternative");
+                Logger.Log($"Error while removing alternatives: {ex.Message}", Logger.LogType.Error);
             }
 
         RefreshCountryView();
@@ -1041,6 +1120,7 @@ public partial class MainWindow : Window
         catch (Exception ex)
         {
             MessageBox.Show(this, ex.StackTrace, "Fehler");
+            Logger.Log($"Error while adding wiki file: {ex.Message}", Logger.LogType.Error);
         }
     }
     
@@ -1064,34 +1144,39 @@ public partial class MainWindow : Window
         catch (Exception ex)
         {
             MessageBox.Show(this, ex.StackTrace, "Fehler");
+            Logger.Log($"Error while adding wiki folder: {ex.Message}", Logger.LogType.Error);
         }
     }
 
 
     private async void MnIAdSetRename_OnClick(object? sender, RoutedEventArgs e)
     {
+        var id = (LstCustomerAdressSets.SelectedItems[0] as KasAddressList).ID;
+        var cus_id = ((Customer)LstCustomers.SelectedItems[0]).ID;
+        var curr_set = new KasAddressList("");
+        foreach (var set in Settings._instance.addressSets.addresses)
+            if (set.ID == id)
+            {
+                curr_set = set;
+
+            }
+        
+        
         if (LstCustomerAdressSets.SelectedItems.Count > 0)
             try
             {
-                var result = await NamingWindow.Show(this);
+                var result = await NamingWindow.Show(this, curr_set.Name);
                 if (result != null)
                 {
-                    var id = KasAddressList.GetIDByAddressSetListItem(LstCustomerAdressSets.SelectedItems[0]
+                    curr_set.Name = result;
-                        .ToString());
+                    Settings.Save();
-                    var cus_id = ((Customer)LstCustomers.SelectedItems[0]).ID;
+                    RefreshAddressSetListItems(cus_id);
-                    foreach (var set in Settings._instance.addressSets.addresses)
-                        if (set.ID == id)
-                        {
-                            set.Name = result;
-                            Settings.Save();
-                            RefreshAddressSetListItems(cus_id);
-                            break;
-                        }
                 }
             }
             catch (Exception ex)
             {
                 MessageBox.Show(this, ex.StackTrace, "Fehler");
+                Logger.Log($"Error while renaming address set: {ex.Message}", Logger.LogType.Error);
             }
     }
 
@@ -1105,6 +1190,7 @@ public partial class MainWindow : Window
         catch (Exception ex)
         {
             MessageBox.Show(this, "Unknown Error: " + ex.Message, "Error");
+            Logger.Log($"Error while deleting customer: {ex.Message}", Logger.LogType.Error);
         }
         RefreshCustomerItems();
     }
@@ -1131,9 +1217,9 @@ public partial class MainWindow : Window
             TbSettingsCustomerCsvSeparator.Text = div;
             Settings.Save();
         }
-        catch
+        catch (Exception ex)
         {
-            Console.WriteLine("Error while parsing");
+            Logger.Log($"Error while changing CSV divider: {ex.Message}", Logger.LogType.Error);
         }
          
     }

NamingWindow.axaml.cs

@@ -52,7 +52,7 @@ public partial class NamingWindow : Window
         }
         catch (Exception ex)
         {
-            Console.WriteLine("Error while showing naming window: " + ex.Message);
+            Logger.Log("Error while showing naming window: " + ex.Message, Logger.LogType.Warning);
             return Task.FromResult<string>(null!);
         }
     }

ResultWindow.axaml.cs

@@ -28,16 +28,25 @@ public partial class ResultWindow : Window
 
     private void GenerateView(List<KasPerson> result)
     {
-        // Filter to only show persons with errors
+        try
-        var result_with_errors = result.Where(p => p.PersonError != null).ToList();
+        {
-        LblResultCount.Content = $"{result_with_errors.Count}/{ur_result.Count} Ergebnisse";
 
-        // TbResults.Text = "";
+
-        // foreach (var person in result_with_errors) TbResults.Text += person.PersonError.GetString()+"\n";
+            // Filter to only show persons with errors
-        LbResults.Items.Clear();
+            var result_with_errors = result.Where(p => p.PersonError != null).ToList();
-        foreach (var person in result_with_errors) LbResults.Items.Add(person.PersonError.ToString(person));
+            LblResultCount.Content = $"{result_with_errors.Count}/{ur_result.Count} Ergebnisse";
-        // StkResults.Children.Clear();
+
-        // foreach (var person in result_with_errors) StkResults.Children.Add(CreatePersonGrid(person));
+            // TbResults.Text = "";
+            // foreach (var person in result_with_errors) TbResults.Text += person.PersonError.GetString()+"\n";
+            LbResults.Items.Clear();
+            foreach (var person in result_with_errors) LbResults.Items.Add(person.PersonError.ToString(person));
+            // StkResults.Children.Clear();
+            // foreach (var person in result_with_errors) StkResults.Children.Add(CreatePersonGrid(person));
+        }
+        catch (Exception ex)
+        {
+            Logger.Log("Error while generating result view: " + ex.Message, Logger.LogType.Warning);
+        }
     }
 
     private Grid CreatePersonGrid(KasPerson person)
@@ -106,44 +115,52 @@ public partial class ResultWindow : Window
 
     private void Load(List<KasPerson> result)
     {
-        var knownErrors = new List<AddressCheck.ErrorTypes>();
+        try
-        var knownWarnings = new List<AddressCheck.WarningTypes>();
-
-        foreach (var person in result)
         {
-            if (person.PersonError == null) continue;
+            var knownErrors = new List<AddressCheck.ErrorTypes>();
+            var knownWarnings = new List<AddressCheck.WarningTypes>();
 
-            foreach (var errtyp in person.PersonError.errors)
+            foreach (var person in result)
-                if (!knownErrors.Contains(errtyp))
+            {
-                    knownErrors.Add(errtyp);
+                if (person.PersonError == null) continue;
 
-            foreach (var wartyp in person.PersonError.warnings)
+                foreach (var errtyp in person.PersonError.errors)
-                if (!knownWarnings.Contains(wartyp))
+                    if (!knownErrors.Contains(errtyp))
-                    knownWarnings.Add(wartyp);
+                        knownErrors.Add(errtyp);
+
+                foreach (var wartyp in person.PersonError.warnings)
+                    if (!knownWarnings.Contains(wartyp))
+                        knownWarnings.Add(wartyp);
+            }
+
+
+            foreach (var errtype in knownErrors)
+            {
+                var cb = new CheckBox();
+                cb.IsChecked = true;
+                cb.Content = errtype.ToString();
+                //cb.Click += (sender, e) => UpdateFilter();
+                errortypecheckboxes.Add(cb);
+                StpFilterOptions.Children.Add(cb);
+            }
+
+            foreach (var wartype in knownWarnings)
+            {
+                var cb = new CheckBox();
+                cb.IsChecked = true;
+                cb.Content = wartype.ToString();
+                //cb.Click += (sender, e) => UpdateFilter();
+                warningtypecheckboxes.Add(cb);
+                StpFilterOptions.Children.Add(cb);
+            }
+
+            GenerateView(result);
+        }
+        catch (Exception ex)
+        {
+            Logger.Log("Error while showing naming window: " + ex.Message, Logger.LogType.Warning);
         }
         
-
-        foreach (var errtype in knownErrors)
-        {
-            var cb = new CheckBox();
-            cb.IsChecked = true;
-            cb.Content = errtype.ToString();
-            //cb.Click += (sender, e) => UpdateFilter();
-            errortypecheckboxes.Add(cb);
-            StpFilterOptions.Children.Add(cb);
-        }
-
-        foreach (var wartype in knownWarnings)
-        {
-            var cb = new CheckBox();
-            cb.IsChecked = true;
-            cb.Content = wartype.ToString();
-            //cb.Click += (sender, e) => UpdateFilter();
-            warningtypecheckboxes.Add(cb);
-            StpFilterOptions.Children.Add(cb);
-        }
-
-        GenerateView(result);
     }
 
     private void BtnUpdateFilter_OnClick(object? sender, RoutedEventArgs e)
@@ -152,7 +169,9 @@ public partial class ResultWindow : Window
 
     private void UpdateFilter()
     {
-        var temp_result = new List<KasPerson>();
+        try
+        {
+            var temp_result = new List<KasPerson>();
 
         var checkedErrors = new HashSet<AddressCheck.ErrorTypes>();
         var checkedWarnings = new HashSet<AddressCheck.WarningTypes>();
@@ -211,6 +230,11 @@ public partial class ResultWindow : Window
         // foreach (var person in temp_result) TbResults.Text += person.PersonError.GetString() +"\n";
         // StkResults.Children.Clear();
         // foreach (var person in temp_result) StkResults.Children.Add(CreatePersonGrid(person));
+        } catch (Exception ex)
+        {
+            Logger.Log("Error while updating filter: " + ex.Message, Logger.LogType.Warning);
+        }
+        
     }
 
 

Tasks/AddressCheck.cs

@@ -39,227 +39,240 @@ public class AddressCheck
 
     public async Task<List<KasPerson>> Perform(int id)
     {
-        // Find the index of the address set with the given id
+        try
-        var adset_index = -1;
-        for (var i = 0; i < Settings._instance.addressSets.addresses.Count; i++)
-            if (Settings._instance.addressSets.addresses[i].ID == id)
-            {
-                adset_index = i;
-                break;
-            }
-
-        if (adset_index == -1) return new List<KasPerson>();
-
-        var adset = Settings._instance.addressSets.addresses[adset_index];
-        var total = adset.KasPersons.Count;
-        var current = 0;
-
-        await Task.Run(async () =>
         {
-            foreach (var person in adset.KasPersons)
+            // Find the index of the address set with the given id
+            var adset_index = -1;
+            for (var i = 0; i < Settings._instance.addressSets.addresses.Count; i++)
+                if (Settings._instance.addressSets.addresses[i].ID == id)
+                {
+                    adset_index = i;
+                    break;
+                }
+
+            if (adset_index == -1) return new List<KasPerson>();
+
+            var adset = Settings._instance.addressSets.addresses[adset_index];
+            var total = adset.KasPersons.Count;
+            var current = 0;
+
+            await Task.Run(async () =>
             {
-                var errors = new List<ErrorTypes>();
+                foreach (var person in adset.KasPersons)
-                var warnings = new List<WarningTypes>();
-                var hasFaults = false;
-
-                var address_component_count = 2; // cause anrede and name are first
-
-                // PLZ-Prüfung
-                if (person.plz == "" || person.plz == null)
                 {
-                    hasFaults = true;
+                    var errors = new List<ErrorTypes>();
-                    warnings.Add(WarningTypes.NoPLZ);
+                    var warnings = new List<WarningTypes>();
-                }
+                    var hasFaults = false;
-                else
+
-                {
+                    var address_component_count = 2; // cause anrede and name are first
-                    if (!AddressCreator.CheckPLZ(person.plz, person.land))
+
+                    // PLZ-Prüfung
+                    if (person.plz == "" || person.plz == null)
                     {
                         hasFaults = true;
-                        errors.Add(ErrorTypes.PlzNotUsable);
+                        warnings.Add(WarningTypes.NoPLZ);
+                    }
+                    else
+                    {
+                        if (!AddressCreator.CheckPLZ(person.plz, person.land))
+                        {
+                            hasFaults = true;
+                            errors.Add(ErrorTypes.PlzNotUsable);
+                        }
+                        // if ((person.plz < 10000 && string.IsNullOrWhiteSpace(person.land)) ||
+                        //     (person.plz < 10000 && person.land == "GER") ||
+                        //     (person.plz < 10000 && person.land == "DE"))
+                        // {
+                        //     hasFaults = true;
+                        //     errors.Add(ErrorTypes.PlzTooShort);
+                        // }
+                        // else if ((person.plz > 99999 && string.IsNullOrWhiteSpace(person.land)) ||
+                        //          (person.plz > 99999 && person.land == "GER") ||
+                        //          (person.plz > 99999 && person.land == "DE"))
+                        // {
+                        //     hasFaults = true;
+                        //     errors.Add(ErrorTypes.PlzTooLong);
+                        // }
                     }
-                    // if ((person.plz < 10000 && string.IsNullOrWhiteSpace(person.land)) ||
-                    //     (person.plz < 10000 && person.land == "GER") ||
-                    //     (person.plz < 10000 && person.land == "DE"))
-                    // {
-                    //     hasFaults = true;
-                    //     errors.Add(ErrorTypes.PlzTooShort);
-                    // }
-                    // else if ((person.plz > 99999 && string.IsNullOrWhiteSpace(person.land)) ||
-                    //          (person.plz > 99999 && person.land == "GER") ||
-                    //          (person.plz > 99999 && person.land == "DE"))
-                    // {
-                    //     hasFaults = true;
-                    //     errors.Add(ErrorTypes.PlzTooLong);
-                    // }
-                }
 
 
-                // PPLZ-Prüfung
+                    // PPLZ-Prüfung
-                if (person.pplz == "" || person.pplz == null)
+                    if (person.pplz == "" || person.pplz == null)
-                {
-                    hasFaults = true;
-                    warnings.Add(WarningTypes.NoPPLZ);
-                }
-                else
-                {
-                    if (!AddressCreator.CheckPLZ(person.pplz, person.land))
                     {
                         hasFaults = true;
-                        errors.Add(ErrorTypes.PPlzNotUsable);
+                        warnings.Add(WarningTypes.NoPPLZ);
+                    }
+                    else
+                    {
+                        if (!AddressCreator.CheckPLZ(person.pplz, person.land))
+                        {
+                            hasFaults = true;
+                            errors.Add(ErrorTypes.PPlzNotUsable);
+                        }
+
+                        // if ((person.pplz < 10000 && string.IsNullOrWhiteSpace(person.land)) ||
+                        //     (person.pplz < 10000 && person.land == "GER") ||
+                        //     (person.pplz < 10000 && person.land == "DE"))
+                        // {
+                        //     hasFaults = true;
+                        //     errors.Add(ErrorTypes.PPlzTooShort);
+                        // }
+                        // else if ((person.pplz > 99999 && string.IsNullOrWhiteSpace(person.land)) ||
+                        //          (person.pplz > 99999 && person.land == "GER") ||
+                        //          (person.pplz > 99999 && person.land == "DE"))
+                        // {
+                        //     hasFaults = true;
+                        //     errors.Add(ErrorTypes.PPlzTooLong);
+                        // }
                     }
 
-                    // if ((person.pplz < 10000 && string.IsNullOrWhiteSpace(person.land)) ||
+                    if (warnings.Contains(WarningTypes.NoPLZ) && warnings.Contains(WarningTypes.NoPPLZ))
-                    //     (person.pplz < 10000 && person.land == "GER") ||
-                    //     (person.pplz < 10000 && person.land == "DE"))
-                    // {
-                    //     hasFaults = true;
-                    //     errors.Add(ErrorTypes.PPlzTooShort);
-                    // }
-                    // else if ((person.pplz > 99999 && string.IsNullOrWhiteSpace(person.land)) ||
-                    //          (person.pplz > 99999 && person.land == "GER") ||
-                    //          (person.pplz > 99999 && person.land == "DE"))
-                    // {
-                    //     hasFaults = true;
-                    //     errors.Add(ErrorTypes.PPlzTooLong);
-                    // }
-                }
-
-                if (warnings.Contains(WarningTypes.NoPLZ) && warnings.Contains(WarningTypes.NoPPLZ))
-                {
-                    hasFaults = true;
-                    errors.Add(ErrorTypes.NoPLZorPPLZ);
-                }
-
-                // Ort-Prüfung
-                if (string.IsNullOrWhiteSpace(person.ort))
-                {
-                    hasFaults = true;
-                    warnings.Add(WarningTypes.NoCity);
-                }
-                else
-                {
-                    address_component_count++;
-                }
-
-                // Street-Number
-                var street = person.strasse.ToCharArray();
-                var intcount = 0;
-                foreach (var c in street)
-                {
-                    int maybe;
-                    if (int.TryParse(c.ToString(), out maybe)) intcount++;
-                }
-
-                if (intcount == 0)
-                {
-                    hasFaults = true;
-                    warnings.Add(WarningTypes.NoStreetNumber);
-                }
-
-                // Last-Name
-                if (string.IsNullOrWhiteSpace(person.name))
-                {
-                    hasFaults = true;
-                    warnings.Add(WarningTypes.NoLastName);
-                }
-
-                // First-Name
-                if (string.IsNullOrWhiteSpace(person.vorname))
-                {
-                    hasFaults = true;
-                    warnings.Add(WarningTypes.NoFirstName);
-                }
-
-                // Street-Check
-                if (string.IsNullOrWhiteSpace(person.strasse))
-                {
-                    hasFaults = true;
-                    warnings.Add(WarningTypes.NoStreet);
-                }
-                else
-                {
-                    address_component_count++;
-                }
-
-                // Address-Component-Count
-                if (!string.IsNullOrWhiteSpace(person.strasse2)) address_component_count++;
-                if (!string.IsNullOrWhiteSpace(person.land)) address_component_count++;
-                if (!string.IsNullOrWhiteSpace(person.name1)) address_component_count++;
-                if (!string.IsNullOrWhiteSpace(person.name2)) address_component_count++;
-                if (!string.IsNullOrWhiteSpace(person.name3)) address_component_count++;
-                if (!string.IsNullOrWhiteSpace(person.name4)) address_component_count++;
-                if (!string.IsNullOrWhiteSpace(person.name5)) address_component_count++;
-                if (!string.IsNullOrWhiteSpace(person.funktion)) address_component_count++;
-                if (!string.IsNullOrWhiteSpace(person.funktion2)) address_component_count++;
-                if (!string.IsNullOrWhiteSpace(person.funktionad)) address_component_count++;
-                if (!string.IsNullOrWhiteSpace(person.abteilung)) address_component_count++;
-
-                // Double-Refsid or DoubleAddresses
-                foreach (var person2 in adset.KasPersons)
-                {
-                    if (adset.KasPersons.IndexOf(person) == adset.KasPersons.IndexOf(person2)) continue;
-
-                    if (person.refsid == person2.refsid) // trifft auf Patch-Addressen nicht zu
                     {
                         hasFaults = true;
-                        warnings.Add(WarningTypes.DoubledRefsid);
+                        errors.Add(ErrorTypes.NoPLZorPPLZ);
                     }
 
-                    if (person.name == person2.name &&
+                    // Ort-Prüfung
-                        person.strasse == person2.strasse &&
+                    if (string.IsNullOrWhiteSpace(person.ort))
-                        person.vorname == person2.vorname &&
-                        person.ort == person2.ort &&
-                        person.funktion == person2.funktion &&
-                        person.funktion2 == person2.funktion2 &&
-                        person.funktionad == person2.funktionad &&
-                        person.abteilung == person2.abteilung &&
-                        person.name1 == person2.name1 &&
-                        person.name2 == person2.name2 &&
-                        person.name3 == person2.name3 &&
-                        person.name4 == person2.name4 &&
-                        person.name5 == person2.name5) // 
-
                     {
                         hasFaults = true;
-                        errors.Add(ErrorTypes.MayBeSameAddress);
+                        warnings.Add(WarningTypes.NoCity);
                     }
-                }
+                    else
-
-                // Adressen-Länge
-                if (address_component_count > 10)
-                {
-                    hasFaults = true;
-                    warnings.Add(WarningTypes.FullAddressTooLong);
-                }
-
-                if (hasFaults)
-                    lock (Settings._instance.addressSets.addresses)
                     {
-                        // Directly set PersonError in the address set
+                        address_component_count++;
-                        var person_index = adset.KasPersons.IndexOf(person);
+                    }
-                        if (person_index >= 0)
+
-                            Settings._instance.addressSets.addresses[adset_index].KasPersons[person_index].PersonError =
+                    // Street-Number
-                                new KasPersonError((errors, warnings));
+                    var street = person.strasse.ToCharArray();
+                    var intcount = 0;
+                    foreach (var c in street)
+                    {
+                        int maybe;
+                        if (int.TryParse(c.ToString(), out maybe)) intcount++;
+                    }
+
+                    if (intcount == 0)
+                    {
+                        hasFaults = true;
+                        warnings.Add(WarningTypes.NoStreetNumber);
+                    }
+
+                    // Last-Name
+                    if (string.IsNullOrWhiteSpace(person.name))
+                    {
+                        hasFaults = true;
+                        warnings.Add(WarningTypes.NoLastName);
+                    }
+
+                    // First-Name
+                    if (string.IsNullOrWhiteSpace(person.vorname))
+                    {
+                        hasFaults = true;
+                        warnings.Add(WarningTypes.NoFirstName);
+                    }
+
+                    // Street-Check
+                    if (string.IsNullOrWhiteSpace(person.strasse))
+                    {
+                        hasFaults = true;
+                        warnings.Add(WarningTypes.NoStreet);
+                    }
+                    else
+                    {
+                        address_component_count++;
+                    }
+
+                    // Address-Component-Count
+                    if (!string.IsNullOrWhiteSpace(person.strasse2)) address_component_count++;
+                    if (!string.IsNullOrWhiteSpace(person.land)) address_component_count++;
+                    if (!string.IsNullOrWhiteSpace(person.name1)) address_component_count++;
+                    if (!string.IsNullOrWhiteSpace(person.name2)) address_component_count++;
+                    if (!string.IsNullOrWhiteSpace(person.name3)) address_component_count++;
+                    if (!string.IsNullOrWhiteSpace(person.name4)) address_component_count++;
+                    if (!string.IsNullOrWhiteSpace(person.name5)) address_component_count++;
+                    if (!string.IsNullOrWhiteSpace(person.funktion)) address_component_count++;
+                    if (!string.IsNullOrWhiteSpace(person.funktion2)) address_component_count++;
+                    if (!string.IsNullOrWhiteSpace(person.funktionad)) address_component_count++;
+                    if (!string.IsNullOrWhiteSpace(person.abteilung)) address_component_count++;
+
+                    // Double-Refsid or DoubleAddresses
+                    foreach (var person2 in adset.KasPersons)
+                    {
+                        if (adset.KasPersons.IndexOf(person) == adset.KasPersons.IndexOf(person2)) continue;
+
+                        if (person.refsid == person2.refsid) // trifft auf Patch-Addressen nicht zu
+                        {
+                            hasFaults = true;
+                            warnings.Add(WarningTypes.DoubledRefsid);
+                        }
+
+                        if (person.name == person2.name &&
+                            person.strasse == person2.strasse &&
+                            person.vorname == person2.vorname &&
+                            person.ort == person2.ort &&
+                            person.funktion == person2.funktion &&
+                            person.funktion2 == person2.funktion2 &&
+                            person.funktionad == person2.funktionad &&
+                            person.abteilung == person2.abteilung &&
+                            person.name1 == person2.name1 &&
+                            person.name2 == person2.name2 &&
+                            person.name3 == person2.name3 &&
+                            person.name4 == person2.name4 &&
+                            person.name5 == person2.name5) // 
+
+                        {
+                            hasFaults = true;
+                            errors.Add(ErrorTypes.MayBeSameAddress);
+                        }
+                    }
+
+                    // Adressen-Länge
+                    if (address_component_count > 10)
+                    {
+                        hasFaults = true;
+                        warnings.Add(WarningTypes.FullAddressTooLong);
                     }
 
-                // Fortschritt aktualisieren
-                Interlocked.Increment(ref current);
-                var percent = current / (double)total * 100;
-                await Dispatcher.UIThread.InvokeAsync(() =>
-                {
                     if (hasFaults)
-                        _progress.AddToLog($"Person mit id {person.id} ist fehlerhaft", Convert.ToInt32(percent).ToString());
+                        lock (Settings._instance.addressSets.addresses)
+                        {
+                            // Directly set PersonError in the address set
+                            var person_index = adset.KasPersons.IndexOf(person);
+                            if (person_index >= 0)
+                                Settings._instance.addressSets.addresses[adset_index].KasPersons[person_index]
+                                        .PersonError =
+                                    new KasPersonError((errors, warnings));
+                        }
 
-                    _progress.ChangePercentage(percent);
+                    // Fortschritt aktualisieren
-                });
+                    Interlocked.Increment(ref current);
-            }
+                    var percent = current / (double)total * 100;
-        });
+                    await Dispatcher.UIThread.InvokeAsync(() =>
+                    {
+                        if (hasFaults)
+                            _progress.AddToLog($"Person mit id {person.id} ist fehlerhaft",
+                                Convert.ToInt32(percent).ToString());
 
-        Settings.Save();
+                        _progress.ChangePercentage(percent);
+                    });
+                }
+            });
 
-        // Return only the persons with errors from the address set
+            Settings.Save();
-        return Settings._instance.addressSets.addresses[adset_index].KasPersons
+
-            .Where(p => p.PersonError != null)
+            // Return only the persons with errors from the address set
-            .ToList();
+            return Settings._instance.addressSets.addresses[adset_index].KasPersons
+                .Where(p => p.PersonError != null)
+                .ToList();
+
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while performing address check: {ex.Message}", Logger.LogType.Error);
+        }
+
+        return null;
     }
+
 }

Tasks/AddressCreation.cs

@@ -43,177 +43,195 @@ public static class AddressCreator
     public static string? CreateFinalMarkdownString(int id)
     {
         // Maximum seven lines of information
-
+        try
-        // find the address
-        KasPerson? address = null;
-        var string_address = "";
-        var address_line_count = 0;
-        foreach (var set in Settings._instance.addressSets.addresses)
         {
-            var temp = set.KasPersons.FirstOrDefault(obj => obj.id == id);
+            // find the address
-            if (temp != null)
+                    KasPerson? address = null;
-            {
+                    var string_address = "";
-                address = temp;
+                    var address_line_count = 0;
-                break;
+                    foreach (var set in Settings._instance.addressSets.addresses)
-            }
+                    {
-        }
+                        var temp = set.KasPersons.FirstOrDefault(obj => obj.id == id);
-
+                        if (temp != null)
-        // no address found
-        if (address == null) return null;
-
-        // let's get started: we start from the bottom
-        // if the country is not Germany, set it; try to map via Global countries alternatives -> translation
-        var trimmedLand = (address.land ?? "").Trim();
-        var trimmedLowerLand = trimmedLand.ToLower();
-        var isGermany = trimmedLowerLand == "germany" || trimmedLowerLand == "ger" ||
-                        trimmedLowerLand == "" || trimmedLowerLand == "de" ||
-                        trimmedLowerLand == "deutschland";
-
-        if (!isGermany)
-        {
-            var countryToShow = trimmedLand; // default: use raw land value
-
-            if (!string.IsNullOrEmpty(trimmedLand))
-                // search for matching country via alternatives using for-loops
-                for (var ci = 0; ci < Global._instance.countries.Count; ci++)
-                {
-                    var country = Global._instance.countries[ci];
-                    for (var ai = 0; ai < country.alternatives.Count; ai++)
-                        try
                         {
-                            var alt = (country.alternatives[ai] ?? "").Trim();
+                            address = temp;
-                            if (string.Equals(alt, trimmedLand, StringComparison.OrdinalIgnoreCase))
+                            break;
+                        }
+                    }
+            
+                    // no address found
+                    if (address == null) return null;
+            
+                    // let's get started: we start from the bottom
+                    // if the country is not Germany, set it; try to map via Global countries alternatives -> translation
+                    var trimmedLand = (address.land ?? "").Trim();
+                    var trimmedLowerLand = trimmedLand.ToLower();
+                    var isGermany = trimmedLowerLand == "germany" || trimmedLowerLand == "ger" ||
+                                    trimmedLowerLand == "" || trimmedLowerLand == "de" ||
+                                    trimmedLowerLand == "deutschland";
+            
+                    if (!isGermany)
+                    {
+                        var countryToShow = trimmedLand; // default: use raw land value
+            
+                        if (!string.IsNullOrEmpty(trimmedLand))
+                            // search for matching country via alternatives using for-loops
+                            for (var ci = 0; ci < Global._instance.countries.Count; ci++)
                             {
-                                countryToShow = string.IsNullOrWhiteSpace(country.translation)
+                                var country = Global._instance.countries[ci];
-                                    ? country.name
+                                for (var ai = 0; ai < country.alternatives.Count; ai++)
-                                    : country.translation;
+                                    try
-                                goto CountryFound;
+                                    {
+                                        var alt = (country.alternatives[ai] ?? "").Trim();
+                                        if (string.Equals(alt, trimmedLand, StringComparison.OrdinalIgnoreCase))
+                                        {
+                                            countryToShow = string.IsNullOrWhiteSpace(country.translation)
+                                                ? country.name
+                                                : country.translation;
+                                            goto CountryFound;
+                                        }
+                                    }
+                                    catch
+                                    {
+                                        // ignore malformed alternative
+                                    }
                             }
-                        }
-                        catch
-                        {
-                            // ignore malformed alternative
-                        }
-                }
             
-            CountryFound:
+                        CountryFound:
-            string_address = "**" + countryToShow + "**"; // Needs to be bold
+                        string_address = "**" + countryToShow + "**"; // Needs to be bold
-            address_line_count++;
+                        address_line_count++;
+                    }
+            
+                    // Alternative A: pplz valid and city existing
+                    if (!string.IsNullOrEmpty(address.ort) && CheckPLZ(address.pplz, address.land))
+                    {
+                        string_address = address.pplz + " " + address.ort + "\n" + string_address;
+                        address_line_count++;
+                        if (!string.IsNullOrWhiteSpace(address.postfach))
+                        {
+                            string_address = "Postfach " + address.postfach.Trim() + "\n" + string_address;
+                            address_line_count++;
+                        }
+                        else if (!string.IsNullOrWhiteSpace(address.strasse.Trim()))
+                        {
+                            string_address = address.strasse.Trim() + "\n" + string_address;
+                            address_line_count++;
+                        }
+            
+                        var nameline = CreateNameLine(address.anredzus, address.anrede, address.titel, address.vorname,
+                            address.adel, address.name, address.namezus);
+            
+                        if (!string.IsNullOrWhiteSpace(nameline))
+                        {
+                            string_address = nameline + "\n" + string_address;
+                            address_line_count++;
+                        }
+            
+                        // REIHENFOLGE
+                        var nameattribs = new[]
+                            { address.name1, address.name2, address.name3, address.name4, address.name5, address.abteilung };
+            
+                        var names = "";
+                        for (var i = 0; i < nameattribs.Length; i++)
+                            try
+                            {
+                                if (address_line_count >= 7) break;
+                                if (!string.IsNullOrWhiteSpace(nameattribs[i]))
+                                {
+                                    names += "\n" + nameattribs[i];
+                                    address_line_count++;
+                                }
+                            }
+                            catch
+                            {
+                                Console.WriteLine("ERROR 15821");
+                            }
+            
+                        string_address = names + "\n" + string_address;
+                    } // Alternative B: plz valid and city existing 
+                    else if (!string.IsNullOrEmpty(address.ort) && CheckPLZ(address.plz, address.land))
+                    {
+                        string_address = address.plz + " " + address.ort + "\n" + string_address;
+                        address_line_count++;
+                        if (!string.IsNullOrWhiteSpace(address.strasse))
+                        {
+                            string_address = address.strasse.Trim() + "\n" + string_address;
+                            address_line_count++;
+                        }
+                        else if (!string.IsNullOrWhiteSpace(address.postfach.Trim()))
+                        {
+                            string_address = "Postfach " + address.postfach.Trim() + "\n" + string_address;
+                            address_line_count++;
+                        }
+            
+                        var nameline = CreateNameLine(address.anredzus, address.anrede, address.titel, address.vorname,
+                            address.adel, address.name, address.namezus);
+            
+                        if (!string.IsNullOrWhiteSpace(nameline))
+                        {
+                            string_address = nameline + "\n" + string_address;
+                            address_line_count++;
+                        }
+            
+                        var nameattribs = new[]
+                            { address.name1, address.name2, address.name3, address.name4, address.name5, address.abteilung };
+            
+                        var names = "";
+                        for (var i = 0; i < nameattribs.Length; i++)
+                            try
+                            {
+                                if (address_line_count >= 7) break;
+                                if (!string.IsNullOrWhiteSpace(nameattribs[i]))
+                                {
+                                    names += "\n" + nameattribs[i];
+                                    address_line_count++;
+                                }
+                            }
+                            catch
+                            {
+                                Console.WriteLine("ERROR 15821");
+                            }
+            
+                        string_address = names + "\n" + string_address;
+                    } // Error-Handling?
+            
+                    if (address_line_count > 1) return string_address;
+                    return null;
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while creating markdown string: {ex.Message}", Logger.LogType.Error);
         }
 
-        // Alternative A: pplz valid and city existing
-        if (!string.IsNullOrEmpty(address.ort) && CheckPLZ(address.pplz, address.land))
-        {
-            string_address = address.pplz + " " + address.ort + "\n" + string_address;
-            address_line_count++;
-            if (!string.IsNullOrWhiteSpace(address.postfach))
-            {
-                string_address = "Postfach " + address.postfach.Trim() + "\n" + string_address;
-                address_line_count++;
-            }
-            else if (!string.IsNullOrWhiteSpace(address.strasse.Trim()))
-            {
-                string_address = address.strasse.Trim() + "\n" + string_address;
-                address_line_count++;
-            }
-
-            var nameline = CreateNameLine(address.anredzus, address.anrede, address.titel, address.vorname,
-                address.adel, address.name, address.namezus);
-
-            if (!string.IsNullOrWhiteSpace(nameline))
-            {
-                string_address = nameline + "\n" + string_address;
-                address_line_count++;
-            }
-
-            // REIHENFOLGE
-            var nameattribs = new[]
-                { address.name1, address.name2, address.name3, address.name4, address.name5, address.abteilung };
-
-            var names = "";
-            for (var i = 0; i < nameattribs.Length; i++)
-                try
-                {
-                    if (address_line_count >= 7) break;
-                    if (!string.IsNullOrWhiteSpace(nameattribs[i]))
-                    {
-                        names += "\n" + nameattribs[i];
-                        address_line_count++;
-                    }
-                }
-                catch
-                {
-                    Console.WriteLine("ERROR 15821");
-                }
-
-            string_address = names + "\n" + string_address;
-        } // Alternative B: plz valid and city existing 
-        else if (!string.IsNullOrEmpty(address.ort) && CheckPLZ(address.plz, address.land))
-        {
-            string_address = address.plz + " " + address.ort + "\n" + string_address;
-            address_line_count++;
-            if (!string.IsNullOrWhiteSpace(address.strasse))
-            {
-                string_address = address.strasse.Trim() + "\n" + string_address;
-                address_line_count++;
-            }
-            else if (!string.IsNullOrWhiteSpace(address.postfach.Trim()))
-            {
-                string_address = "Postfach " + address.postfach.Trim() + "\n" + string_address;
-                address_line_count++;
-            }
-
-            var nameline = CreateNameLine(address.anredzus, address.anrede, address.titel, address.vorname,
-                address.adel, address.name, address.namezus);
-
-            if (!string.IsNullOrWhiteSpace(nameline))
-            {
-                string_address = nameline + "\n" + string_address;
-                address_line_count++;
-            }
-
-            var nameattribs = new[]
-                { address.name1, address.name2, address.name3, address.name4, address.name5, address.abteilung };
-
-            var names = "";
-            for (var i = 0; i < nameattribs.Length; i++)
-                try
-                {
-                    if (address_line_count >= 7) break;
-                    if (!string.IsNullOrWhiteSpace(nameattribs[i]))
-                    {
-                        names += "\n" + nameattribs[i];
-                        address_line_count++;
-                    }
-                }
-                catch
-                {
-                    Console.WriteLine("ERROR 15821");
-                }
-
-            string_address = names + "\n" + string_address;
-        } // Error-Handling?
-
-        if (address_line_count > 1) return string_address;
         return null;
     }
 
     public static string CreateNameLine(string anredezus, string anrede, string titel, string vorname, string adel,
         string name, string namezus)
     {
-        if (!string.IsNullOrWhiteSpace(anredezus))
+        try
+        {
+            if (!string.IsNullOrWhiteSpace(anredezus))
+                    return string.Join(" ",
+                               new[] { anredezus, titel, vorname, adel, name }
+                                   .Where(s => !string.IsNullOrWhiteSpace(s))
+                           )
+                           + (string.IsNullOrWhiteSpace(namezus) ? "" : $" ({namezus.Trim()})");
+        
+            // else 
             return string.Join(" ",
-                       new[] { anredezus, titel, vorname, adel, name }
+                       new[] { anrede, titel, vorname, adel, name }
                            .Where(s => !string.IsNullOrWhiteSpace(s))
                    )
                    + (string.IsNullOrWhiteSpace(namezus) ? "" : $" ({namezus.Trim()})");
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while performing address name line creation: {ex.Message}", Logger.LogType.Error);
+        }
+
+        return null;
         
-        // else 
-        return string.Join(" ",
-                   new[] { anrede, titel, vorname, adel, name }
-                       .Where(s => !string.IsNullOrWhiteSpace(s))
-               )
-               + (string.IsNullOrWhiteSpace(namezus) ? "" : $" ({namezus.Trim()})");
     }
 
     /// <summary>
@@ -224,29 +242,39 @@ public static class AddressCreator
     /// <returns>true or faslse, depending on result :+1:</returns>
     public static bool CheckPLZ(string plz, string land)
     {
-        if (string.IsNullOrWhiteSpace(plz)) return false;
+        try
+        {
+            if (string.IsNullOrWhiteSpace(plz)) return false;
             
-        var trimmedPlz = plz.Trim();
+            var trimmedPlz = plz.Trim();
-        var trimmedLand = land.ToLower().Trim();
+            var trimmedLand = land.ToLower().Trim();
     
-        // Check if it's a German country code
+            // Check if it's a German country code
-        var isGermany = trimmedLand == "germany" || trimmedLand == "ger" || trimmedLand == "de" ||
+            var isGermany = trimmedLand == "germany" || trimmedLand == "ger" || trimmedLand == "de" ||
-                        trimmedLand == "deutschland" || trimmedLand == "";
+                            trimmedLand == "deutschland" || trimmedLand == "";
     
-        if (isGermany)
+            if (isGermany)
-            // For Germany (including empty land), accept numeric postal codes with 5 digits
+                // For Germany (including empty land), accept numeric postal codes with 5 digits
-            try
+                try
-            {
+                {
-                var iplz = Convert.ToInt32(trimmedPlz);
+                    var iplz = Convert.ToInt32(trimmedPlz);
-                if (trimmedPlz.Length == 5) return true;
+                    if (trimmedPlz.Length == 5) return true;
-                return false;
+                    return false;
-            }
+                }
-            catch
+                catch
-            {
+                {
-                return false;
+                    return false;
-            }
+                }
+    
+            // For non-German countries, accept any non-empty postal code
+            return true;
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while performing address plz check: {ex.Message}", Logger.LogType.Error);
+        }
+
+        return false;
         
-        // For non-German countries, accept any non-empty postal code
-        return true;
     }
 }

Tasks/AddressPatch.cs

@@ -3,6 +3,7 @@ using System.IO;
 using System.Linq;
 using System.Reflection;
 using System.Text;
+using Logof_Client;
 
 public class AddressPatch
 {
@@ -58,61 +59,82 @@ public class AddressPatch
 
     public static AddressPatch Import(Uri filename)
     {
-        var patch = new AddressPatch();
+        try
-
-        // Alle Zeilen aus der Datei laden
-        var lines = File.ReadAllLines(filename.LocalPath);
-
-        // Alle Properties der Klasse (Strings und bools)
-        var properties = typeof(AddressPatch).GetProperties(BindingFlags.Public | BindingFlags.Instance);
-
-        // Nur die Properties, die mit _is enden (also die String-Werte)
-        var stringProps = properties.Where(p => p.PropertyType == typeof(string) && p.Name.EndsWith("_is"));
-
-        foreach (var prop in stringProps)
         {
-            // Beispiel: prop.Name = "name_is"
+            var patch = new AddressPatch();
-            var baseName = prop.Name.Substring(0, prop.Name.Length - 3); // "name"
             
-            // In der Datei wird nach "name:" gesucht (ohne _is)
+                    // Alle Zeilen aus der Datei laden
-            var line = lines.FirstOrDefault(l => l.StartsWith(baseName + ":"));
+                    var lines = File.ReadAllLines(filename.LocalPath);
-            if (line != null)
-            {
-                // Wert extrahieren (alles nach dem Doppelpunkt)
-                var value = line.Substring(line.IndexOf(':') + 1).Trim();
             
-                // Wert im Patch-Objekt setzen
+                    // Alle Properties der Klasse (Strings und bools)
-                prop.SetValue(patch, value);
+                    var properties = typeof(AddressPatch).GetProperties(BindingFlags.Public | BindingFlags.Instance);
             
-                // Passendes has_ Feld aktivieren, z.B. "has_name"
+                    // Nur die Properties, die mit _is enden (also die String-Werte)
-                var hasProp = properties.FirstOrDefault(p => p.Name == "has_" + baseName);
+                    var stringProps = properties.Where(p => p.PropertyType == typeof(string) && p.Name.EndsWith("_is"));
-                if (hasProp != null && hasProp.PropertyType == typeof(bool)) hasProp.SetValue(patch, true);
+            
-            }
+                    foreach (var prop in stringProps)
+                    {
+                        // Beispiel: prop.Name = "name_is"
+                        var baseName = prop.Name.Substring(0, prop.Name.Length - 3); // "name"
+            
+                        // In der Datei wird nach "name:" gesucht (ohne _is)
+                        var line = lines.FirstOrDefault(l => l.StartsWith(baseName + ":"));
+                        if (line != null)
+                        {
+                            // Wert extrahieren (alles nach dem Doppelpunkt)
+                            var value = line.Substring(line.IndexOf(':') + 1).Trim();
+            
+                            // Wert im Patch-Objekt setzen
+                            prop.SetValue(patch, value);
+            
+                            // Passendes has_ Feld aktivieren, z.B. "has_name"
+                            var hasProp = properties.FirstOrDefault(p => p.Name == "has_" + baseName);
+                            if (hasProp != null && hasProp.PropertyType == typeof(bool)) hasProp.SetValue(patch, true);
+                        }
+                    }
+            
+                    return patch;
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while importing address patch: {ex.Message}", Logger.LogType.Error);
         }
 
-        return patch;
+        return null;
+
+
     }
 
 
     public override string ToString()
     {
-        var properties = GetType().GetProperties(BindingFlags.Public | BindingFlags.Instance);
+        try
-        var stringProps = properties.Where(p => p.PropertyType == typeof(string) && p.Name.EndsWith("_is"));
-
-        var lines = new StringBuilder();
-
-        foreach (var prop in stringProps)
         {
-            // passendes has_ Feld
+            var properties = GetType().GetProperties(BindingFlags.Public | BindingFlags.Instance);
-            var hasProp =
+                    var stringProps = properties.Where(p => p.PropertyType == typeof(string) && p.Name.EndsWith("_is"));
-                properties.FirstOrDefault(p => p.Name == "has_" + prop.Name.Substring(0, prop.Name.Length - 3));
+            
-            if (hasProp != null && (bool)hasProp.GetValue(this))
+                    var lines = new StringBuilder();
-            {
+            
-                var value = (string)prop.GetValue(this);
+                    foreach (var prop in stringProps)
-                lines.AppendLine($"{prop.Name} => {value}");
+                    {
-            }
+                        // passendes has_ Feld
+                        var hasProp =
+                            properties.FirstOrDefault(p => p.Name == "has_" + prop.Name.Substring(0, prop.Name.Length - 3));
+                        if (hasProp != null && (bool)hasProp.GetValue(this))
+                        {
+                            var value = (string)prop.GetValue(this);
+                            lines.AppendLine($"{prop.Name} => {value}");
+                        }
+                    }
+            
+                    return lines.ToString().TrimEnd();
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while parsing: {ex.Message}", Logger.LogType.Error);
         }
 
-        return lines.ToString().TrimEnd();
+        return "Error while parsing";
+        
     }
 }

Tasks/AddressRepair.cs

@@ -1,3 +1,4 @@
+using System;
 using System.Collections.Generic;
 
 namespace Logof_Client;
@@ -9,15 +10,16 @@ public class AddressRepair(ProgressWindow progressWindow)
     public KasAddressList Perform(KasAddressList all_addresses,
         List<(int, List<AddressCheck.ErrorTypes>)> failed_addresses)
     {
-        // foreach (var k in all_addresses.KasPersons)
+        try
-        // foreach (var p in failed_addresses)
+        {
-        // {
+            
-        //     if (k.refsid != p.Item1) continue;
+        }
-        //
+        catch (Exception ex)
-        //     if (p.Item1.Contains(AddressCheck.WarningTypes.DoubledRefsid))
+        {
-        //     {
+            Logger.Log($"Error while performing address repair: {ex.Message}", Logger.LogType.Error);
-        //     }
+        }
-        // }
+
+        return null;
 
         return null;
     }

Tasks/AddressShortener.cs

@@ -12,68 +12,73 @@ public class AddressShortener(ProgressWindow progressWindow)
 
     public async Task Perform(KasAddressList list)
     {
-        // find doubled addresses by refsid
+        try
-        List<int> doubled_ids = new List<int>();
-        for (int i = 0; i < list.KasPersons.Count; i++)
         {
-            var address =  list.KasPersons[i];
+            List<int> doubled_ids = new List<int>();
-            for (int j = 0; j < list.KasPersons.Count; j++)
+                    for (int i = 0; i < list.KasPersons.Count; i++)
-            {
+                    {
-                if (i == j) continue;
+                        var address =  list.KasPersons[i];
-                var sec_address =  list.KasPersons[j];
+                        for (int j = 0; j < list.KasPersons.Count; j++)
+                        {
+                            if (i == j) continue;
+                            var sec_address =  list.KasPersons[j];
                             
-                if (address.refsid == sec_address.refsid && !doubled_ids.Contains(address.refsid) && address.refsid != 0)
+                            if (address.refsid == sec_address.refsid && !doubled_ids.Contains(address.refsid) && address.refsid != 0)
-                {
+                            {
-                    doubled_ids.Add(address.refsid);
+                                doubled_ids.Add(address.refsid);
-                }
+                            }
-            }
+                        }
+                    }
+                    
+                    
+                    // delete doubled addresses by refsid
+                    foreach (int id in doubled_ids)
+                    {
+                        // does this remove both of the doubled addresses?
+                        list.KasPersons.Remove(list.KasPersons.FirstOrDefault(x => x.refsid == id));
+                    }
+            
+                    List<int> toRemove = new List<int>();
+                    foreach (var address in list.KasPersons)
+                    {
+                        try
+                        {
+                            if (address.PersonError.errors.Contains(AddressCheck.ErrorTypes.NoPLZorPPLZ))
+                            {
+                                toRemove.Add(address.id);
+                            }
+                            else if (address.PersonError.errors.Contains(AddressCheck.ErrorTypes.PlzNotUsable) &&
+                                     address.PersonError.errors.Contains(AddressCheck.ErrorTypes.PPlzNotUsable))
+                            {
+                                toRemove.Add(address.id);
+                            } else if (address.PersonError.errors.Contains(AddressCheck.ErrorTypes.PlzNotUsable) &&
+                                       address.PersonError.warnings.Contains(AddressCheck.WarningTypes.NoPPLZ))
+                            {
+                                toRemove.Add(address.id);
+                            }
+                            else if (address.PersonError.errors.Contains(AddressCheck.ErrorTypes.PPlzNotUsable) &&
+                                     address.PersonError.warnings.Contains(AddressCheck.WarningTypes.NoPLZ))
+                            {
+                                toRemove.Add(address.id);
+                            }
+                        }
+                        catch
+                        {
+                            Console.WriteLine("PersonError not accessible: " + address.id);
+                        }
+                    }
+                    
+                    // delete doubled addresses by refsid
+                    foreach (int id in toRemove)
+                    {
+                        // does this remove both of the doubled addresses?
+                        list.KasPersons.Remove(list.KasPersons.Find(x => x.id == id));
+                    }
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while performing address shortener: {ex.Message}", Logger.LogType.Error);
         }
         
-        
-        // delete doubled addresses by refsid
-        foreach (int id in doubled_ids)
-        {
-            // does this remove both of the doubled addresses?
-            list.KasPersons.Remove(list.KasPersons.FirstOrDefault(x => x.refsid == id));
-        }
-
-        List<int> toRemove = new List<int>();
-        foreach (var address in list.KasPersons)
-        {
-            try
-            {
-                if (address.PersonError.errors.Contains(AddressCheck.ErrorTypes.NoPLZorPPLZ))
-                {
-                    toRemove.Add(address.id);
-                }
-                else if (address.PersonError.errors.Contains(AddressCheck.ErrorTypes.PlzNotUsable) &&
-                         address.PersonError.errors.Contains(AddressCheck.ErrorTypes.PPlzNotUsable))
-                {
-                    toRemove.Add(address.id);
-                } else if (address.PersonError.errors.Contains(AddressCheck.ErrorTypes.PlzNotUsable) &&
-                           address.PersonError.warnings.Contains(AddressCheck.WarningTypes.NoPPLZ))
-                {
-                    toRemove.Add(address.id);
-                }
-                else if (address.PersonError.errors.Contains(AddressCheck.ErrorTypes.PPlzNotUsable) &&
-                         address.PersonError.warnings.Contains(AddressCheck.WarningTypes.NoPLZ))
-                {
-                    toRemove.Add(address.id);
-                }
-            }
-            catch
-            {
-                Console.WriteLine("PersonError not accessible: " + address.id);
-            }
-        }
-        
-        // delete doubled addresses by refsid
-        foreach (int id in toRemove)
-        {
-            // does this remove both of the doubled addresses?
-            list.KasPersons.Remove(list.KasPersons.Find(x => x.id == id));
-        }
-        
-        //return null;
     }
 }

Tasks/CombineAddresses.cs

@@ -26,28 +26,48 @@ public class CombineAddresses
     public async Task<(KasAddressList, KasAddressList)> Perform(List<KasAddressList> address_lists, string type, CombineType comb_type,
         bool? exportUnused, bool? deleteOld)
     {
-        var result = await Execute(address_lists,type,comb_type,exportUnused);
+        try
-
-        if (deleteOld == true)
         {
-            foreach (var list in address_lists)
+            var result = await Execute(address_lists,type,comb_type,exportUnused);
-            {
+            
-                Settings._instance.addressSets.addresses.Remove(list);
+                    if (deleteOld == true)
-            }
+                    {
+                        foreach (var list in address_lists)
+                        {
+                            Settings._instance.addressSets.addresses.Remove(list);
+                        }
+                    }
+            
+                    return result;
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while performing address combining: {ex.Message}", Logger.LogType.Error);
         }
 
-        return result;
+        return (null,null);
+        
 
     }
 
     private async Task<(KasAddressList, KasAddressList)> Execute(List<KasAddressList> address_lists, string type, CombineType comb_type,
         bool? exportUnused)
     {
-        if (type == "difference") return await Difference(address_lists, comb_type, exportUnused);
+        try
-        if (type == "union") return await Union(address_lists, comb_type, exportUnused);
+        {
-        if (type == "intersection") return await Intersection(address_lists, comb_type, exportUnused);
+            if (type == "difference") return await Difference(address_lists, comb_type, exportUnused);
-        if (type == "symdiff") return await SymmetricDifference(address_lists, comb_type, exportUnused);
+            if (type == "union") return await Union(address_lists, comb_type, exportUnused);
-        return (null, null);
+            if (type == "intersection") return await Intersection(address_lists, comb_type, exportUnused);
+            if (type == "symdiff") return await SymmetricDifference(address_lists, comb_type, exportUnused);
+            return (null, null);
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while executing address combining: {ex.Message}", Logger.LogType.Error);
+        }
+
+        return (null,null);
+        
     }
 
 
@@ -60,6 +80,7 @@ public class CombineAddresses
     /// <returns></returns>
     public bool CompareAddresses(KasPerson first, KasPerson second, CombineType comb_type)
     {
+        
         // A refsid of 0 means "missing", so it must not collapse unrelated entries.
         if(comb_type == CombineType.refsid)
             if (first.refsid != 0 && second.refsid != 0 && first.refsid == second.refsid) return true;
@@ -96,81 +117,101 @@ public class CombineAddresses
         bool? return_unused,
         Progress? progress = null)
     {
-        if (address_lists == null || address_lists.Count == 0)
+        try
-            return (new KasAddressList(await KasAddressList.GenerateName("difference")), null);
-
-        progress ??= new Progress
         {
-            TotalPersons = address_lists.Sum(l => l.KasPersons.Count),
+            if (address_lists == null || address_lists.Count == 0)
-            ComparedPersons = 0
+                return (new KasAddressList(await KasAddressList.GenerateName("difference")), null);
-        };
     
-        // Vereinigung aller Listen außer der ersten
+            progress ??= new Progress
-        var restUnion = new List<KasPerson>();
+            {
-        for (var i = 1; i < address_lists.Count; i++)
+                TotalPersons = address_lists.Sum(l => l.KasPersons.Count),
-            restUnion.AddRange(address_lists[i].KasPersons);
+                ComparedPersons = 0
-        var result = new KasAddressList(await KasAddressList.GenerateName("difference"));
+            };
-        var second_result = new KasAddressList("none");
-        if(return_unused == true)
-            second_result = new KasAddressList(await KasAddressList.GenerateName("difference_rest", false));
     
-        foreach (var person in address_lists[0].KasPersons)
+            // Vereinigung aller Listen außer der ersten
+            var restUnion = new List<KasPerson>();
+            for (var i = 1; i < address_lists.Count; i++)
+                restUnion.AddRange(address_lists[i].KasPersons);
+            var result = new KasAddressList(await KasAddressList.GenerateName("difference"));
+            var second_result = new KasAddressList("none");
+            if(return_unused == true)
+                second_result = new KasAddressList(await KasAddressList.GenerateName("difference_rest", false));
+    
+            foreach (var person in address_lists[0].KasPersons)
+            {
+                var isDouble = restUnion.Any(p => CompareAddresses(person, p, comb_type));
+                if (!isDouble)
+                    result.KasPersons.Add(person);
+                else
+                    second_result.KasPersons.Add(person);
+    
+    
+                progress.Increment();
+                if (progress.LogAction == null) continue;
+                var logMessage =
+                    $"Person mit id {person.id} verglichen mit {restUnion.Count} Personen des Restes.";
+                await Dispatcher.UIThread.InvokeAsync(() => progress.LogAction?.Invoke(logMessage));
+            }
+    
+            if (return_unused == true) return (result, second_result);
+            return (result, null);
+        }
+        catch (Exception ex)
         {
-            var isDouble = restUnion.Any(p => CompareAddresses(person, p, comb_type));
+            Logger.Log($"Error while performing difference-combining: {ex.Message}", Logger.LogType.Error);
-            if (!isDouble)
-                result.KasPersons.Add(person);
-            else
-                second_result.KasPersons.Add(person);
-
-
-            progress.Increment();
-            if (progress.LogAction == null) continue;
-            var logMessage =
-                $"Person mit id {person.id} verglichen mit {restUnion.Count} Personen des Restes.";
-            await Dispatcher.UIThread.InvokeAsync(() => progress.LogAction?.Invoke(logMessage));
         }
 
-        if (return_unused == true) return (result, second_result);
+        return (null,null);
-        return (result, null);
+        
     }
 
 
     public async Task<(KasAddressList, KasAddressList)> Union(List<KasAddressList> address_lists, CombineType comb_type, bool? return_unused,
         Progress progress = null)
     {
-        var result = new KasAddressList(await KasAddressList.GenerateName("union"));
+        try
-        var second_result = new KasAddressList("none");
-        if(return_unused == true)
-            second_result = new KasAddressList(await KasAddressList.GenerateName("union_rest", false));
-
-        if (address_lists == null || address_lists.Count == 0)
-            return (result, null);
-
-        var total = address_lists.Sum(l => l.KasPersons.Count);
-        var processed = 0;
-
-        foreach (var list in address_lists)
-        foreach (var person in list.KasPersons)
         {
-            if (!result.KasPersons.Any(existing => CompareAddresses(existing, person, comb_type)))
+            var result = new KasAddressList(await KasAddressList.GenerateName("union"));
-                result.KasPersons.Add(person);
+            var second_result = new KasAddressList("none");
-            else
+            if(return_unused == true)
-                second_result.KasPersons.Add(person);
+                second_result = new KasAddressList(await KasAddressList.GenerateName("union_rest", false));
     
-            processed++;
+            if (address_lists == null || address_lists.Count == 0)
-            var percent = processed / (double)total * 100;
+                return (result, null);
-            var logMessage =
-                $"{percent:F1}%: Person mit {person.id} hinzugefügt (aktuell {result.KasPersons.Count} Einträge)";
     
-            if (progress == null) continue;
+            var total = address_lists.Sum(l => l.KasPersons.Count);
-            if (Dispatcher.UIThread.CheckAccess())
+            var processed = 0;
-                progress.LogAction?.Invoke(logMessage);
+    
-            else
+            foreach (var list in address_lists)
-                Dispatcher.UIThread.Post(() => progress.LogAction?.Invoke(logMessage));
+            foreach (var person in list.KasPersons)
+            {
+                if (!result.KasPersons.Any(existing => CompareAddresses(existing, person, comb_type)))
+                    result.KasPersons.Add(person);
+                else
+                    second_result.KasPersons.Add(person);
+    
+                processed++;
+                var percent = processed / (double)total * 100;
+                var logMessage =
+                    $"{percent:F1}%: Person mit {person.id} hinzugefügt (aktuell {result.KasPersons.Count} Einträge)";
+    
+                if (progress == null) continue;
+                if (Dispatcher.UIThread.CheckAccess())
+                    progress.LogAction?.Invoke(logMessage);
+                else
+                    Dispatcher.UIThread.Post(() => progress.LogAction?.Invoke(logMessage));
+            }
+    
+            if (return_unused == true) return (result, second_result);
+            return (result, null);
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while performing union-combining: {ex.Message}", Logger.LogType.Error);
         }
 
-        if (return_unused == true) return (result, second_result);
+        return (null,null);
-        return (result, null);
+        
     }
 
 
@@ -182,107 +223,127 @@ public class CombineAddresses
     public async Task<(KasAddressList, KasAddressList)> Intersection(List<KasAddressList> address_lists, CombineType comb_type,
         bool? return_unused, Progress progress = null)
     {
-        var result = new KasAddressList(await KasAddressList.GenerateName("intersection"));
+        try
-        var second_result = new KasAddressList("none");
-        if(return_unused == true) second_result = new KasAddressList(await KasAddressList.GenerateName("intersection_rest", false));
-
-        if (address_lists == null || address_lists.Count == 0)
-            return (result, null);
-
-        // Nur die erste Liste als Ausgangspunkt verwenden
-        var baseList = address_lists[0];
-        var otherLists = address_lists.Skip(1).ToList();
-
-        var total = baseList.KasPersons.Count;
-        var processed = 0;
-
-        foreach (var person in baseList.KasPersons)
         {
-            // Prüfen, ob Person in *allen* anderen Listen vorkommt
+            var result = new KasAddressList(await KasAddressList.GenerateName("intersection"));
-            var isInAll = otherLists.All(list =>
+            var second_result = new KasAddressList("none");
-                list.KasPersons.Any(existing => CompareAddresses(existing, person, comb_type)));
+            if(return_unused == true) second_result = new KasAddressList(await KasAddressList.GenerateName("intersection_rest", false));
     
-            if (isInAll)
+            if (address_lists == null || address_lists.Count == 0)
-                result.KasPersons.Add(person);
+                return (result, null);
-            else
-                second_result.KasPersons.Add(person);
     
-            processed++;
+            // Nur die erste Liste als Ausgangspunkt verwenden
-            var percent = processed / (double)total * 100;
+            var baseList = address_lists[0];
-            var logMessage =
+            var otherLists = address_lists.Skip(1).ToList();
-                $"{percent:F1}%: Person mit {person.id} geprüft – {(isInAll ? "in allen enthalten" : "nicht überall vorhanden")}";
     
-            // Sicher und nicht blockierend loggen
+            var total = baseList.KasPersons.Count;
-            if (progress != null)
+            var processed = 0;
+    
+            foreach (var person in baseList.KasPersons)
             {
-                if (Dispatcher.UIThread.CheckAccess())
+                // Prüfen, ob Person in *allen* anderen Listen vorkommt
-                    progress.LogAction?.Invoke(logMessage);
+                var isInAll = otherLists.All(list =>
+                    list.KasPersons.Any(existing => CompareAddresses(existing, person, comb_type)));
+    
+                if (isInAll)
+                    result.KasPersons.Add(person);
                 else
-                    Dispatcher.UIThread.Post(() => progress.LogAction?.Invoke(logMessage));
+                    second_result.KasPersons.Add(person);
+    
+                processed++;
+                var percent = processed / (double)total * 100;
+                var logMessage =
+                    $"{percent:F1}%: Person mit {person.id} geprüft – {(isInAll ? "in allen enthalten" : "nicht überall vorhanden")}";
+    
+                // Sicher und nicht blockierend loggen
+                if (progress != null)
+                {
+                    if (Dispatcher.UIThread.CheckAccess())
+                        progress.LogAction?.Invoke(logMessage);
+                    else
+                        Dispatcher.UIThread.Post(() => progress.LogAction?.Invoke(logMessage));
+                }
             }
+    
+            if (return_unused == true) return (result, second_result);
+            return (result, null);
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while performing intersection-combining: {ex.Message}", Logger.LogType.Error);
         }
 
-        if (return_unused == true) return (result, second_result);
+        return (null,null);
-        return (result, null);
+        
     }
 
 
     public async Task<(KasAddressList, KasAddressList)> SymmetricDifference(List<KasAddressList> address_lists, CombineType comb_type,
         bool? return_unused, Progress progress = null)
     {
-        var result = new KasAddressList(await KasAddressList.GenerateName("symmetric_difference"));
+        try
-        var second_result = new KasAddressList("none");
-        if(return_unused == true) second_result = new KasAddressList(await KasAddressList.GenerateName("symmetric_rest", false));
-
-        if (address_lists == null || address_lists.Count == 0)
-            return (result, null);
-
-        var total = address_lists.Sum(l => l.KasPersons.Count);
-        var processed = 0;
-
-        // Wir sammeln alle Personen + zählen, wie oft sie vorkommen
-        var allPersons = new List<(KasPerson person, int count)>();
-
-        foreach (var list in address_lists)
-        foreach (var person in list.KasPersons)
         {
-            // Prüfen, ob schon vorhanden
+            var result = new KasAddressList(await KasAddressList.GenerateName("symmetric_difference"));
-            var existing = allPersons.FirstOrDefault(p => CompareAddresses(p.person, person, comb_type));
+            var second_result = new KasAddressList("none");
-            if (existing.person != null)
+            if(return_unused == true) second_result = new KasAddressList(await KasAddressList.GenerateName("symmetric_rest", false));
-            {
-                // Falls schon drin → Vorkommen erhöhen
-                var index = allPersons.IndexOf(existing);
-                allPersons[index] = (existing.person, existing.count + 1);
-            }
-            else
-            {
-                // Neu hinzufügen
-                allPersons.Add((person, 1));
-            }
     
-            processed++;
+            if (address_lists == null || address_lists.Count == 0)
-            var percent = processed / (double)total * 100;
+                return (result, null);
-            var logMessage =
-                $"{percent:F1}%: Person mit {person.id} verarbeitet (Zwischengröße {allPersons.Count})";
     
-            if (progress != null)
+            var total = address_lists.Sum(l => l.KasPersons.Count);
+            var processed = 0;
+    
+            // Wir sammeln alle Personen + zählen, wie oft sie vorkommen
+            var allPersons = new List<(KasPerson person, int count)>();
+    
+            foreach (var list in address_lists)
+            foreach (var person in list.KasPersons)
             {
-                if (Dispatcher.UIThread.CheckAccess())
+                // Prüfen, ob schon vorhanden
-                    progress.LogAction?.Invoke(logMessage);
+                var existing = allPersons.FirstOrDefault(p => CompareAddresses(p.person, person, comb_type));
+                if (existing.person != null)
+                {
+                    // Falls schon drin → Vorkommen erhöhen
+                    var index = allPersons.IndexOf(existing);
+                    allPersons[index] = (existing.person, existing.count + 1);
+                }
                 else
-                    Dispatcher.UIThread.Post(() => progress.LogAction?.Invoke(logMessage));
+                {
+                    // Neu hinzufügen
+                    allPersons.Add((person, 1));
+                }
+    
+                processed++;
+                var percent = processed / (double)total * 100;
+                var logMessage =
+                    $"{percent:F1}%: Person mit {person.id} verarbeitet (Zwischengröße {allPersons.Count})";
+    
+                if (progress != null)
+                {
+                    if (Dispatcher.UIThread.CheckAccess())
+                        progress.LogAction?.Invoke(logMessage);
+                    else
+                        Dispatcher.UIThread.Post(() => progress.LogAction?.Invoke(logMessage));
+                }
             }
+    
+            // Nur diejenigen übernehmen, die *genau einmal* vorkamen
+            foreach (var (person, count) in allPersons)
+                if (count == 1)
+                    result.KasPersons.Add(person);
+                else
+                    second_result.KasPersons.Add(person);
+    
+            if (return_unused == true) return (result, second_result);
+            return (result, null);
+        }
+        catch (Exception ex)
+        {
+            Logger.Log($"Error while performing symdiff-combining: {ex.Message}", Logger.LogType.Error);
         }
 
-        // Nur diejenigen übernehmen, die *genau einmal* vorkamen
+        return (null,null);
-        foreach (var (person, count) in allPersons)
-            if (count == 1)
-                result.KasPersons.Add(person);
-            else
-                second_result.KasPersons.Add(person);
         
-        if (return_unused == true) return (result, second_result);
-        return (result, null);
     }
     
 }