[chore:] logging for ResultWindow.axaml.cs

DataStore/DataStructures.cs

@@ -60,6 +60,11 @@ public class KasAddressList //Address-Set
         var id = listItemName.Split(" - ")[0];
         return int.Parse(id);
     }
+
+    public override string ToString()
+    {
+        return Name + " (" + KasPersons.Count + " Einträge)";
+    }
 }
 
 public class KasPerson
@@ -188,6 +193,18 @@ public class KasPerson
 
         return highest + base_id + 1;
     }
+
+    public override string ToString()
+    {
+        if (refsid != null && refsid != 0)
+        {
+            return refsid + " - " + name;
+        }
+        else
+        {
+            return id + " - " + name;
+        }
+    }
 }
 
 public class KasPersonError
@@ -212,4 +229,9 @@ public class KasPersonError
 
         return output;
     }
+
+    public string ToString(KasPerson person)
+    {
+        return "ID:"+person.id + "; Name: " +person.name +"; Errors: " + GetString();
+    }
 }

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();
         }
@@ -135,7 +132,7 @@ public class Global
 public class Customers
 {
     public List<Customer> customers = new();
-    public int current { get; set; } = 0;
+    public Customer current { get; set; } = null;
 }
 
 public class Customer
@@ -150,6 +147,11 @@ public class Customer
         ID = highestID + 1;
     }
     
+    public override string ToString()
+    {
+        return name;
+    }
+
     public string name { get; set; } = "";
     public string description { get; set; } = "";
     public string sender_address { get; set; } = "";
@@ -158,11 +160,11 @@ public class Customer
     public int ID { get; }
 
 
-    public static int GetIDByCustomerListItem(string item_content)
+    // public static int GetIDByCustomerListItem(string item_content)
-    {
+    // {
-        var id = item_content.Split(" - ")[0];
+    //     var id = item_content.Split(" - ")[0];
-        return int.Parse(id);
+    //     return int.Parse(id);
-    }
+    // }
 }
 
 public class AddressSets

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

@@ -121,7 +121,7 @@
                                                FontWeight="Bold" />
                                     </StackPanel>
                                 </Button>
-                                <Button Width="250" IsEnabled="False"
+                                <Button Width="250" IsEnabled="False" Click="BtnShorten_OnClick"
                                         HorizontalContentAlignment="Center" x:Name="BtnShorten"
                                         Margin="0,0,0,10">
                                     <StackPanel Orientation="Horizontal">
@@ -220,14 +220,18 @@
                                             </StackPanel>
                                             <Label FontSize="9" Content="Nur Elemente, die NICHT doppelt sind" />
                                         </StackPanel>
-
                                     </Button>
                                 </StackPanel>
-                                <StackPanel Orientation="Vertical" HorizontalAlignment="Center">
+                                <StackPanel Orientation="Horizontal" HorizontalAlignment="Center" Spacing="50">
-                                    <RadioButton Content="Vergleiche nach refsid" IsChecked="True" x:Name="RbComprefsid"></RadioButton>
+                                    <StackPanel Orientation="Vertical" >
-                                    <RadioButton Content="Vergleiche nach finaler Adresse" IsChecked="False" x:Name="RbCompfinAd"></RadioButton>
+                                        <RadioButton Content="Vergleiche nach refsid" IsChecked="True" x:Name="RbComprefsid"></RadioButton>
+                                        <RadioButton Content="Vergleiche nach finaler Adresse" IsChecked="False" x:Name="RbCompfinAd"></RadioButton>
+                                    </StackPanel>
+                                    <StackPanel Orientation="Vertical">
+                                        <CheckBox HorizontalAlignment="Left" x:Name="CbMergeExportUnmerged" IsChecked="False">Speichere Unverarbeitete in neuem Verteiler</CheckBox>
+                                        <CheckBox HorizontalAlignment="Left" x:Name="CbMergeDeleteOld" IsChecked="False">Lösche ursprüngliche Sets</CheckBox>
+                                    </StackPanel>
                                 </StackPanel>
-                                <CheckBox HorizontalAlignment="Center" x:Name="CbMergeExportUnmerged" IsChecked="False">Speichere Unverarbeitete in neuem Verteiler</CheckBox>
                             </StackPanel>
                         </Grid>
                         <Grid Grid.Row="2" Margin="20" ColumnDefinitions="*,5*,*" IsVisible="False" x:Name="GrdExportMarginOptions">
@@ -481,12 +485,20 @@
                                                  TextChanged="TbSettingsCustomerSenderAddress_OnTextChanged"
                                                  x:Name="TbSettingsCustomerSenderAddress" />
                                     </Grid>
-                                    <Grid ColumnDefinitions="150,*">
+                                    <Grid ColumnDefinitions="150,*,Auto">
                                         <Label Content="CSV-Trennzeichen" />
                                         <TextBox Grid.Column="1" Watermark=","
-                                                 HorizontalAlignment="Stretch"
+                                                 HorizontalAlignment="Stretch" Margin="0,0,5,0"
                                                  TextChanged="TbSettingsCustomerCsvSeparator_OnTextChanged"
                                                  x:Name="TbSettingsCustomerCsvSeparator" />
+                                        <StackPanel Grid.Column="2" Orientation="Horizontal" Spacing="5">
+                                            <Button Click="ChangeCSVDivider" Content=","></Button>
+                                            <Button Click="ChangeCSVDivider" Content=";"></Button>
+                                            <Button Click="ChangeCSVDivider" Content="TAB"></Button>
+                                            <Button Click="ChangeCSVDivider" Content="|"></Button>
+                                            <Button Click="ChangeCSVDivider" Content="SPACE"></Button>
+                                            <Button Click="ChangeCSVDivider" Content=":"></Button>
+                                        </StackPanel>
                                     </Grid>
                                     <Grid ColumnDefinitions="150,*">
                                         <Label Content="Address-Patch-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);
@@ -96,27 +104,31 @@ public partial class MainWindow : Window
 
         new ResultWindow(result, addresSetID).Show();
 
+        Settings.Save();
         
         //await MessageBox.Show(_instance, $"{result.Count} Einträge fehlerhaft.", "Fertig");
     }
     
-    // private async void StartAddressRepair(Uri path)
+    private async void StartAddressShortener(KasAddressList set)
-    // {
+    {
-    //     var addresses = DataImport.ImportKasAddressList(path); // Ihr Code hier
+        //var addresses = DataImport.ImportKasAddressList(path); // Ihr Code hier
-    //     var progressWindow = new ProgressWindow();
+
-    //
+        var progressWindow = new ProgressWindow();
-    //     progressWindow.Show(_instance);
+
-    //
+        progressWindow.Show(_instance);
-    //     var processor = new AddressRepair(progressWindow);
+
-    //     //var result = await processor.Perform(addresses.Item2, errors);
+        var processor = new AddressShortener(progressWindow);
-    //
+        await processor.Perform(set);
-    //
+
-    //     progressWindow.Close();
+        // foreach (var item in result)
-    //
+        // {
-    //
+        // }
-    //     //new ResultWindow(result, addresses.Item2).Show();
+
-    //     //await MessageBox.Show(_instance, $"{result.Count} Einträge fehlerhaft.", "Fertig");
+        progressWindow.Close();
-    // }
+        Settings.Save();
+        //await MessageBox.Show(_instance, $"{result.Count} Einträge fehlerhaft.", "Fertig");
+    }
+    
 
     private void MnuExit_OnClick(object? sender, RoutedEventArgs e)
     {
@@ -135,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);
         }
     }
 
@@ -151,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);
         }
     }
 
@@ -167,31 +179,10 @@ 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);
         }
     }
 
-    private async void BtnChooseFile_OnClick(object? sender, RoutedEventArgs e)
-    {
-        var topLevel = GetTopLevel(this);
-        var file = await topLevel!.StorageProvider.OpenFilePickerAsync(new FilePickerOpenOptions
-        {
-            Title = "KAS-CSV-Datei auswählen",
-            AllowMultiple = false,
-            FileTypeFilter = new[]
-            {
-                new FilePickerFileType(".csv-Datei")
-                {
-                    Patterns = new[] { "*.csv" }
-                }
-            }
-        });
-
-        if (file == null) return;
-        //TbFilename.Text = file[0].Path.ToString();
-        filePath = file[0].Path;
-    }
-
     private void BtnCheck_OnClick(object? sender, RoutedEventArgs e)
     {
         MakeCalcManVisible();
@@ -201,7 +192,7 @@ public partial class MainWindow : Window
             return;
         }
 
-        StartAddressCheck(KasAddressList.GetIDByAddressSetListItem(LstCustomerAdressSets.SelectedItem.ToString()));
+        StartAddressCheck(((KasAddressList)LstCustomerAdressSets.SelectedItem).ID);
     }
 
     private void BtnCombine_OnClick(object? sender, RoutedEventArgs e)
@@ -218,41 +209,7 @@ public partial class MainWindow : Window
         GrdExportMarginOptions.IsVisible = false;
     }
 
-    private async void StartCombine(Uri path)
-    {
-        MakeCalcManVisible();
-        var addresses = await DataImport.ImportKasAddressList(path);
-        var progressWindow = new ProgressWindow();
-        var address_list = new List<KasAddressList> { addresses.Item2 };
     
-        var topLevel = GetTopLevel(this);
-        var file = await topLevel!.StorageProvider.OpenFilePickerAsync(new FilePickerOpenOptions
-        {
-            Title = "Weitere KAS-CSV-Dateien auswählen",
-            AllowMultiple = true,
-            FileTypeFilter = new[]
-            {
-                new FilePickerFileType(".csv-Dateien")
-                {
-                    Patterns = new[] { "*.csv" }
-                    //Patterns = new[] { "*" }
-                }
-            }
-        });
-
-        if (file == null) return;
-
-        //filePath = file[0].Path;
-        foreach (var f in file) address_list.Add((await DataImport.ImportKasAddressList(f.Path)).Item2);
-
-        progressWindow.Show(_instance);
-
-        var processor = new CombineAddresses(progressWindow);
-        //var result = await processor.Perform(address_list);
-
-
-        progressWindow.Close();
-    }
 
     private async void NavTree_SelectionChanged(object? sender, SelectionChangedEventArgs e)
     {
@@ -266,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 });
             }
@@ -284,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)
@@ -310,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)
@@ -378,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);
         }
     }
 
@@ -437,39 +408,7 @@ public partial class MainWindow : Window
         MessageBox.Show(this, "Bitte starten Sie das Programm neu, um die Änderungen wirksam zu machen.", "Achtung");
     }
 
-    private async Task<string> OpenSettingsSaveAsDialog()
-    {
-        var settingsFileName = "KAS-Adress-Liste";
     
-        try
-        {
-            var filePicker = new SaveFileDialog
-            {
-                Title = "Datei speichern...",
-                InitialFileName = $"{settingsFileName}.csv",
-                DefaultExtension = ".csv",
-                Filters = new List<FileDialogFilter>
-                {
-                    new() { Name = "CSV-Datei", Extensions = { "csv" } }
-                }
-            };
-
-            var settingsSavePath = await filePicker.ShowAsync(this);
-            if (settingsSavePath == null)
-            {
-            }
-
-            return settingsSavePath;
-            //SettingsManager.Save(settingsSavePath);
-            //Logger.Log("Settings saved at " + settingsSavePath);
-        }
-        catch (Exception ex)
-        {
-            //Logger.Log("Saving file not successful: " + ex.Message, Logger.LogType.Error);
-            await MessageBox.Show(this, ex.Message, "Fehler beim Speichern der Datei");
-            return null;
-        }
-    }
 
     private async void StartCombine(List<KasAddressList> address_lists, int owner_id, string type, CombineAddresses.CombineType comb_type)
     {
@@ -478,7 +417,7 @@ public partial class MainWindow : Window
         progressWindow.Show(_instance);
 
         var processor = new CombineAddresses(progressWindow);
-        var result = await processor.Perform(address_lists, type, comb_type, CbMergeExportUnmerged.IsChecked);
+        var result = await processor.Perform(address_lists, type, comb_type, CbMergeExportUnmerged.IsChecked, CbMergeDeleteOld.IsChecked);
 
         if (result.Item1 != null)
             result.Item1.owner_id = owner_id;
@@ -509,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);
         }
     }
 
@@ -518,7 +458,7 @@ public partial class MainWindow : Window
     {
         if (LstSettingsCustomers.SelectedIndex == null || LstSettingsCustomers.SelectedIndex == -1) return;
         foreach (var customer in Settings._instance.customers.customers)
-            if (customer.ID == Settings._instance.customers.current)
+            if (customer.ID == Settings._instance.customers.current.ID)
                 customer.name = TbSettingsCustomerName.Text;
         //Settings.Save();
         //RefreshCustomerItems();
@@ -528,19 +468,19 @@ public partial class MainWindow : Window
     {
         if (LstSettingsCustomers.SelectedIndex < 0) return;
         Settings._instance.customers.current =
-            Customer.GetIDByCustomerListItem(LstSettingsCustomers.SelectedItems[0].ToString());
+            ((Customer)LstSettingsCustomers.SelectedItems[0]);
-        foreach (var customer in Settings._instance.customers.customers)
+        //foreach (var customer in Settings._instance.customers.customers)
-            if (customer.ID == Settings._instance.customers.current)
+            //if (customer.ID == Settings._instance.customers.current.ID)
-            {
+            //{
-                TbSettingsCustomerDescription.Text = customer.description;
+                TbSettingsCustomerDescription.Text = Settings._instance.customers.current.description;
-                TbSettingsCustomerName.Text = customer.name;
+                TbSettingsCustomerName.Text = Settings._instance.customers.current.name;
-                TbSettingsCustomerSenderAddress.Text = customer.sender_address;
+                TbSettingsCustomerSenderAddress.Text = Settings._instance.customers.current.sender_address;
-                TbSettingsCustomerCsvSeparator.Text = customer.separator.ToString();
+                TbSettingsCustomerCsvSeparator.Text = Settings._instance.customers.current.separator.ToString();
-                if (customer.patch != null)
+                if (Settings._instance.customers.current.patch != null)
-                    TbSettingsCustomerPatchInfo.Text = customer.patch.ToString();
+                    TbSettingsCustomerPatchInfo.Text = Settings._instance.customers.current.patch.ToString();
                 else
                     TbSettingsCustomerPatchInfo.Text = "";
-            }
+            //}
     }
 
     private async void EditButton_Click(object? sender, RoutedEventArgs e)
@@ -573,25 +513,33 @@ public partial class MainWindow : Window
         LstSettingsCustomers.Items.Clear();
         foreach (var customer in Settings._instance.customers.customers)
         {
-            LstCustomers.Items.Add(customer.ID + " - " + customer.name);
+            LstCustomers.Items.Add(customer);
-            LstSettingsCustomers.Items.Add(customer.ID + " - " + customer.name);
+            LstSettingsCustomers.Items.Add(customer);
         }
 
         try
         {
             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)
+            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();
     }
 
@@ -605,69 +553,78 @@ public partial class MainWindow : Window
     {
         MakeCalcManVisible();
         if (LstCustomers.SelectedItems == null || LstCustomers.SelectedIndex == -1) return;
-        var customer_id = int.Parse(LstCustomers.SelectedItem.ToString().Split(" - ")[0]);
+        var customer_id = ((Customer)LstCustomers.SelectedItem).ID;
         RefreshAddressSetListItems(customer_id);
     }
 
     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.ID == Customer.GetIDByCustomerListItem(LstCustomers.SelectedItems[0].ToString()))
+                 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)
@@ -685,7 +642,7 @@ public partial class MainWindow : Window
         foreach (var k in Settings._instance.addressSets.addresses)
         foreach (var customer in Settings._instance.customers.customers)
             if (customer.ID == k.owner_id && customer.ID == customer_id)
-                LstCustomerAdressSets.Items.Add(k.ID + " - " + k.Name + " (" + k.KasPersons.Count + " Einträge)");
+                LstCustomerAdressSets.Items.Add(k);
     }
 
     private void LstCustomerAdressSets_OnSelectionChanged(object? sender, SelectionChangedEventArgs e)
@@ -704,6 +661,7 @@ public partial class MainWindow : Window
             BtnCheck.IsEnabled = true;
             BtnCombine.IsEnabled = true;
             BtnGenerateLabels.IsEnabled = true;
+            BtnShorten.IsEnabled = true;
 
             // BtnRepair.IsEnabled = true;
             // BtnShorten.IsEnabled = true;
@@ -712,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.GetIDByCustomerListItem(LstSettingsCustomers.SelectedItems[0].ToString()))
+                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)
@@ -746,7 +711,7 @@ public partial class MainWindow : Window
             ? TbSettingsCustomerCsvSeparator.Text
             : ",";
         foreach (var customer in Settings._instance.customers.customers)
-            if (customer.ID == Settings._instance.customers.current)
+            if (customer.ID == Settings._instance.customers.current.ID)
                 try
                 {
                     customer.separator = Convert.ToChar(sep);
@@ -754,26 +719,27 @@ 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)
     {
         var list = new List<KasAddressList>();
         foreach (var item in LstCustomerAdressSets.SelectedItems)
-            list.Add(Settings._instance.addressSets.GetAddressSetByID(
+            list.Add((KasAddressList)item);
-                Convert.ToInt32(item.ToString().Split(" - ")[0])));
         try
         {
-            StartCombine(list, Convert.ToInt32(LstCustomers.SelectedItem.ToString().Split(" - ")[0]), "difference", GetCombiningTyp());
+            StartCombine(list, Convert.ToInt32(LstCustomers.SelectedItem), "difference", GetCombiningTyp());
-        }
+        }catch (Exception ex)
-        catch
         {
-            
+            Logger.Log($"Error while combining (difference): {ex.Message}", Logger.LogType.Error);
         }
     }
 
@@ -781,15 +747,14 @@ public partial class MainWindow : Window
     {
         var list = new List<KasAddressList>();
         foreach (var item in LstCustomerAdressSets.SelectedItems)
-            list.Add(Settings._instance.addressSets.GetAddressSetByID(
+            list.Add((KasAddressList)item);
-                Convert.ToInt32(item.ToString().Split(" - ")[0])));
         try
         {
-            StartCombine(list, Convert.ToInt32(LstCustomers.SelectedItem.ToString().Split(" - ")[0]), "union", GetCombiningTyp());
+            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);
         }
         
     }
@@ -798,15 +763,14 @@ public partial class MainWindow : Window
     {
         var list = new List<KasAddressList>();
         foreach (var item in LstCustomerAdressSets.SelectedItems)
-            list.Add(Settings._instance.addressSets.GetAddressSetByID(
+            list.Add((KasAddressList)item);
-                Convert.ToInt32(item.ToString().Split(" - ")[0])));
         try
         {
             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);
         }
     }
 
@@ -827,8 +791,7 @@ public partial class MainWindow : Window
     {
         var list = new List<KasAddressList>();
         foreach (var item in LstCustomerAdressSets.SelectedItems)
-            list.Add(Settings._instance.addressSets.GetAddressSetByID(
+            list.Add((KasAddressList)item);
-                Convert.ToInt32(item.ToString().Split(" - ")[0])));
 
         StartCombine(list, Convert.ToInt32(LstCustomers.SelectedItem.ToString().Split(" - ")[0]), "symdiff", GetCombiningTyp());
     }
@@ -843,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(
-                Convert.ToInt32(LstCustomerAdressSets.SelectedItems[0].ToString().Split(" - ")[0]),
-                "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()
@@ -950,7 +955,7 @@ public partial class MainWindow : Window
     {
         if (LstSettingsCustomers.SelectedIndex == null || LstSettingsCustomers.SelectedIndex == -1) return;
         foreach (var customer in Settings._instance.customers.customers)
-            if (customer.ID == Settings._instance.customers.current)
+            if (customer.ID == Settings._instance.customers.current.ID)
                 customer.sender_address = TbSettingsCustomerSenderAddress.Text;
     }
 
@@ -969,6 +974,7 @@ public partial class MainWindow : Window
 
     public void RefreshCountryView()
     {
+        
         if (_suppressCountryRefresh) return;
         try
         {
@@ -983,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);
@@ -992,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;
@@ -1016,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
@@ -1033,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();
@@ -1110,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);
         }
     }
     
@@ -1133,51 +1144,83 @@ 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.GetIDByCustomerListItem(LstCustomers.SelectedItems[0].ToString());
+                    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);
             }
     }
 
     private void BtnDeleteCustomer_OnClick(object? sender, RoutedEventArgs e)
     {
         if (LstSettingsCustomers.SelectedIndex == null || LstSettingsCustomers.SelectedIndex == -1) return;
-        foreach (var customer in Settings._instance.customers.customers)
+        try
-            if (customer.ID == Settings._instance.customers.current)
+        {
-                try
+            Settings._instance.customers.customers.Remove((Customer)LstSettingsCustomers.SelectedItem);
-                {
+        }
-                    Settings._instance.customers.customers.Remove(customer);
+        catch (Exception ex)
-                    break;
+        {
-                }
+            MessageBox.Show(this, "Unknown Error: " + ex.Message, "Error");
-                catch (Exception ex)
+            Logger.Log($"Error while deleting customer: {ex.Message}", Logger.LogType.Error);
-                {
+        }
-                    MessageBox.Show(this, "Unknown Error: " + ex.Message, "Error");
-                }
         RefreshCustomerItems();
     }
+
+    private void BtnShorten_OnClick(object? sender, RoutedEventArgs e)
+    {
+        MakeCalcManVisible();
+        if (LstCustomerAdressSets.SelectedIndex == -1)
+        {
+            MessageBox.Show(null, "Bitte zunächst ein Adress-Set auswählen", "Kein Adress-Set ausgewählt");
+            return;
+        }
+
+        StartAddressShortener((KasAddressList)LstCustomerAdressSets.SelectedItem);
+    }
+
+    private void ChangeCSVDivider(object sender, RoutedEventArgs e)
+    {
+        try
+        {
+            string div = (sender as Button).Content as string;
+            if (div == "TAB") div = "\t";
+            if (div == "SPACE") div = " ";
+            TbSettingsCustomerCsvSeparator.Text = div;
+            Settings.Save();
+        }
+        catch (Exception ex)
+        {
+            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!);
         }
     }

ProgressWindow.axaml.cs

@@ -16,9 +16,9 @@ public partial class ProgressWindow : Window
         PbProgress.Value = percentage;
     }
 
-    public void AddToLog(string message)
+    public void AddToLog(string message, string percent)
     {
-        TbLog.Text = message;
+        TbLog.Text = message + $"\n{percent}%";
         //ScvLog.ScrollToEnd();
     }
 }

ResultWindow.axaml

@@ -20,7 +20,9 @@
                     Margin="10,10,10,10" />
         </Grid>
         <ScrollViewer Grid.Column="1" HorizontalScrollBarVisibility="Auto" VerticalScrollBarVisibility="Auto">
-            <StackPanel x:Name="StkResults" Orientation="Vertical" Margin="10" />
+            <!-- <TextBlock x:Name="TbResults"></TextBlock> -->
+            <ListBox x:Name="LbResults"></ListBox>
+            <!-- <StackPanel x:Name="StkResults" Orientation="Vertical" Margin="10" /> -->
         </ScrollViewer>
 
     </Grid>

ResultWindow.axaml.cs

@@ -28,12 +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";
 
-        StkResults.Children.Clear();
+
-        foreach (var person in result_with_errors) StkResults.Children.Add(CreatePersonGrid(person));
+            // Filter to only show persons with errors
+            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";
+            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)
@@ -102,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)
@@ -148,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>();
@@ -201,8 +224,17 @@ public partial class ResultWindow : Window
 
         LblResultCount.Content = $"{temp_result.Count}/{ur_result.Count} Ergebnisse";
 
-        StkResults.Children.Clear();
+        LbResults.Items.Clear();
-        foreach (var person in temp_result) StkResults.Children.Add(CreatePersonGrid(person));
+        foreach (var person in temp_result) LbResults.Items.Add(person.PersonError.ToString(person));
+        // TbResults.Text = "";
+        // 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

@@ -1,3 +1,4 @@
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Threading;
@@ -38,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");
+                        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

@@ -0,0 +1,84 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Threading.Tasks;
+using System.Threading.Tasks.Dataflow;
+
+namespace Logof_Client;
+
+public class AddressShortener(ProgressWindow progressWindow)
+{
+    private readonly ProgressWindow _progress = progressWindow;
+
+    public async Task Perform(KasAddressList list)
+    {
+        try
+        {
+            List<int> doubled_ids = new List<int>();
+                    for (int i = 0; i < list.KasPersons.Count; i++)
+                    {
+                        var address =  list.KasPersons[i];
+                        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)
+                            {
+                                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);
+        }
+        
+    }
+}

Tasks/CombineAddresses.cs

@@ -24,14 +24,50 @@ public class CombineAddresses
     }
 
     public async Task<(KasAddressList, KasAddressList)> Perform(List<KasAddressList> address_lists, string type, CombineType comb_type,
+        bool? exportUnused, bool? deleteOld)
+    {
+        try
+        {
+            var result = await Execute(address_lists,type,comb_type,exportUnused);
+            
+                    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 (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);
+            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);
         
-        return (null, null);
     }
 
 
@@ -44,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;
@@ -80,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);
+        
     }
 
 
@@ -166,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);
     }
     
 }