Release v0.0.0.18

New matching logic checks ALL identifying fields for compatibility:
- Empty vs filled field = different contacts (not a match)
- Both empty = compatible (ignored for matching)
- Both filled = must be equal

Fields checked: Email, FirstName, LastName, Company, PhoneWork,
PhoneMobile. Requires at least one strong match (email, name,
or phone) plus no conflicting fields.

Example: Two "Max Mustermann" where one has Company="Firma A"
and the other has empty Company are now correctly identified as
different contacts.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

installer/setup.iss

@@ -2,7 +2,7 @@
 ; Erfordert Inno Setup 6.x (https://jrsoftware.org/isinfo.php)
 
 #define MyAppName "Starface Outlook Sync"
-#define MyAppVersion "0.0.0.15"
+#define MyAppVersion "0.0.0.18"
 #define MyAppPublisher "HackerSoft - Hacker-Net Telekommunikation"
 #define MyAppURL "https://www.hacker-net.de"
 #define MyAppExeName "StarfaceOutlookSync.exe"

src/StarfaceOutlookSync/Services/StarfaceApiClient.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Linq;
 using System.Net;
 using System.Net.Http;
 using System.Security.Cryptography;
@@ -115,43 +116,63 @@ namespace StarfaceOutlookSync.Services
         {
             var books = new List<StarfaceAddressBook>();
 
-            books.Add(new StarfaceAddressBook
-            {
-                Type = "central",
-                Name = "Zentrales Adressbuch"
-            });
-
-            var userId = await GetCurrentUserIdAsync();
-            if (!string.IsNullOrEmpty(userId))
-            {
-                books.Add(new StarfaceAddressBook
-                {
-                    Type = "user",
-                    UserId = userId,
-                    Name = "Persoenliches Adressbuch"
-                });
-            }
-
-            // Tags als virtuelle Adressbuecher
+            // Alle Tags laden - die Starface nutzt Tags als Adressbuch-Zuordnung
+            var allTags = new JArray();
             try
             {
                 var resp = await _http.GetAsync($"{_baseUrl}/contacts/tags");
                 if (resp.IsSuccessStatusCode)
                 {
-                    var tags = JArray.Parse(await resp.Content.ReadAsStringAsync());
-                    foreach (var tag in tags)
-                    {
-                        books.Add(new StarfaceAddressBook
-                        {
-                            Type = "tag",
-                            TagId = tag["id"]?.ToString() ?? "",
-                            Name = $"Tag: {tag["name"]}"
-                        });
-                    }
+                    allTags = JArray.Parse(await resp.Content.ReadAsStringAsync());
+                    OnDebug?.Invoke($"Gefundene Tags: {allTags.Count}");
+                    foreach (var t in allTags)
+                        OnDebug?.Invoke($"  Tag: {t["name"]} (id: {t["id"]}, alias: {t["alias"]}, owner: {t["owner"]})");
                 }
             }
             catch { }
 
+            // Zentrales Adressbuch (folder/all)
+            var allTag = allTags.FirstOrDefault(t => t["name"]?.ToString() == "folder/all"
+                                                  || t["alias"]?.ToString()?.Contains("folder.all") == true);
+            books.Add(new StarfaceAddressBook
+            {
+                Type = "central",
+                TagId = allTag?["id"]?.ToString() ?? "",
+                Name = "Zentrales Adressbuch"
+            });
+
+            // Persoenliches Adressbuch (folder/private mit owner = userId)
+            var userId = await GetCurrentUserIdAsync();
+            if (!string.IsNullOrEmpty(userId))
+            {
+                var privateTag = allTags.FirstOrDefault(t =>
+                    (t["name"]?.ToString() == "folder/private" || t["alias"]?.ToString()?.Contains("folder.private") == true)
+                    && t["owner"]?.ToString() == userId);
+
+                books.Add(new StarfaceAddressBook
+                {
+                    Type = "user",
+                    UserId = userId,
+                    TagId = privateTag?["id"]?.ToString() ?? "",
+                    Name = "Persoenliches Adressbuch"
+                });
+            }
+
+            // Alle weiteren Tags als Adressbuecher anbieten
+            foreach (var tag in allTags)
+            {
+                var tagName = tag["name"]?.ToString() ?? "";
+                // folder/all und folder/private bereits oben erfasst
+                if (tagName == "folder/all" || tagName == "folder/private") continue;
+
+                books.Add(new StarfaceAddressBook
+                {
+                    Type = "tag",
+                    TagId = tag["id"]?.ToString() ?? "",
+                    Name = tagName
+                });
+            }
+
             return books;
         }
 
@@ -248,6 +269,16 @@ namespace StarfaceOutlookSync.Services
         public async Task<UnifiedContact> CreateContactAsync(UnifiedContact contact, StarfaceAddressBook book)
         {
             var sfContact = MapToStarface(contact);
+
+            // Tag zuweisen - die Starface verlangt dass jeder Kontakt einem Tag zugeordnet ist
+            if (!string.IsNullOrEmpty(book.TagId))
+            {
+                sfContact["tags"] = new JArray
+                {
+                    new JObject { ["id"] = book.TagId }
+                };
+            }
+
             var query = "";
             if (book.Type == "user" && !string.IsNullOrEmpty(book.UserId))
                 query = $"?userId={book.UserId}";

src/StarfaceOutlookSync/Services/SyncEngine.cs

@@ -17,61 +17,87 @@ namespace StarfaceOutlookSync.Services
 
         /// <summary>
         /// Findet einen passenden Kontakt in der Kandidatenliste.
-        /// Matching-Reihenfolge: E-Mail, dann Vorname+Nachname+Firma, dann Vorname+Nachname.
+        /// Strenges Matching: Felder die auf einer Seite gefuellt sind muessen
+        /// auf der anderen auch gefuellt (und gleich) sein.
+        /// Ein leeres Feld auf einer Seite und ein gefuelltes auf der anderen
+        /// bedeutet: verschiedene Kontakte.
         /// </summary>
         private static UnifiedContact FindMatch(UnifiedContact contact, List<UnifiedContact> candidates)
         {
             if (candidates == null || candidates.Count == 0) return null;
 
-            // 1. Exakte E-Mail
-            if (!string.IsNullOrEmpty(contact.Email))
+            foreach (var c in candidates)
             {
-                var byEmail = candidates.FirstOrDefault(c =>
-                    !string.IsNullOrEmpty(c.Email) &&
-                    c.Email.Equals(contact.Email, StringComparison.OrdinalIgnoreCase));
-                if (byEmail != null) return byEmail;
-            }
-
-            // 2. Vorname + Nachname + Firma (staerkstes Match ohne E-Mail)
-            if ((!string.IsNullOrEmpty(contact.FirstName) || !string.IsNullOrEmpty(contact.LastName))
-                && !string.IsNullOrEmpty(contact.Company))
-            {
-                var byNameCompany = candidates.FirstOrDefault(c =>
-                    c.FirstName.Equals(contact.FirstName, StringComparison.OrdinalIgnoreCase) &&
-                    c.LastName.Equals(contact.LastName, StringComparison.OrdinalIgnoreCase) &&
-                    c.Company.Equals(contact.Company, StringComparison.OrdinalIgnoreCase));
-                if (byNameCompany != null) return byNameCompany;
-            }
-
-            // 3. Vorname + Nachname (ohne Firma)
-            if (!string.IsNullOrEmpty(contact.FirstName) || !string.IsNullOrEmpty(contact.LastName))
-            {
-                var byName = candidates.FirstOrDefault(c =>
-                    c.FirstName.Equals(contact.FirstName, StringComparison.OrdinalIgnoreCase) &&
-                    c.LastName.Equals(contact.LastName, StringComparison.OrdinalIgnoreCase) &&
-                    (!string.IsNullOrEmpty(c.FirstName) || !string.IsNullOrEmpty(c.LastName)));
-                if (byName != null) return byName;
-            }
-
-            // 4. Telefonnummer (Buero oder Mobil)
-            if (!string.IsNullOrEmpty(contact.PhoneWork))
-            {
-                var byPhone = candidates.FirstOrDefault(c =>
-                    !string.IsNullOrEmpty(c.PhoneWork) &&
-                    NormalizePhone(c.PhoneWork) == NormalizePhone(contact.PhoneWork));
-                if (byPhone != null) return byPhone;
-            }
-            if (!string.IsNullOrEmpty(contact.PhoneMobile))
-            {
-                var byMobile = candidates.FirstOrDefault(c =>
-                    !string.IsNullOrEmpty(c.PhoneMobile) &&
-                    NormalizePhone(c.PhoneMobile) == NormalizePhone(contact.PhoneMobile));
-                if (byMobile != null) return byMobile;
+                if (IsMatch(contact, c))
+                    return c;
             }
 
             return null;
         }
 
+        private static bool IsMatch(UnifiedContact a, UnifiedContact b)
+        {
+            // Mindestens ein identifizierendes Feld muss vorhanden sein
+            bool hasName = !string.IsNullOrEmpty(a.FirstName) || !string.IsNullOrEmpty(a.LastName);
+            bool hasEmail = !string.IsNullOrEmpty(a.Email);
+            bool hasPhone = !string.IsNullOrEmpty(a.PhoneWork) || !string.IsNullOrEmpty(a.PhoneMobile);
+
+            if (!hasName && !hasEmail && !hasPhone) return false;
+
+            // E-Mail: wenn auf beiden Seiten vorhanden, muss sie gleich sein
+            // Wenn nur auf einer Seite vorhanden -> kein Match
+            if (!FieldsCompatible(a.Email, b.Email)) return false;
+
+            // Name: wenn auf einer Seite vorhanden, muss er gleich sein
+            if (!FieldsCompatible(a.FirstName, b.FirstName)) return false;
+            if (!FieldsCompatible(a.LastName, b.LastName)) return false;
+
+            // Firma: wenn auf einer Seite vorhanden, muss sie gleich sein
+            // Leere Firma vs. gefuellte Firma = verschiedene Kontakte
+            if (!FieldsCompatible(a.Company, b.Company)) return false;
+
+            // Telefon: wenn auf einer Seite vorhanden, muss sie gleich sein
+            if (!PhoneFieldsCompatible(a.PhoneWork, b.PhoneWork)) return false;
+            if (!PhoneFieldsCompatible(a.PhoneMobile, b.PhoneMobile)) return false;
+
+            // Mindestens ein starkes Match muss vorhanden sein
+            bool emailMatch = !string.IsNullOrEmpty(a.Email) && !string.IsNullOrEmpty(b.Email)
+                              && a.Email.Equals(b.Email, StringComparison.OrdinalIgnoreCase);
+            bool nameMatch = hasName
+                             && a.FirstName.Equals(b.FirstName, StringComparison.OrdinalIgnoreCase)
+                             && a.LastName.Equals(b.LastName, StringComparison.OrdinalIgnoreCase)
+                             && (!string.IsNullOrEmpty(a.FirstName) || !string.IsNullOrEmpty(a.LastName));
+            bool phoneMatch = !string.IsNullOrEmpty(a.PhoneWork) && !string.IsNullOrEmpty(b.PhoneWork)
+                              && NormalizePhone(a.PhoneWork) == NormalizePhone(b.PhoneWork);
+
+            return emailMatch || nameMatch || phoneMatch;
+        }
+
+        /// <summary>
+        /// Prueft ob zwei Felder kompatibel sind.
+        /// Beide leer = kompatibel. Beide gleich = kompatibel.
+        /// Eins leer, eins gefuellt = NICHT kompatibel (verschiedene Kontakte).
+        /// </summary>
+        private static bool FieldsCompatible(string a, string b)
+        {
+            bool aEmpty = string.IsNullOrEmpty(a);
+            bool bEmpty = string.IsNullOrEmpty(b);
+
+            if (aEmpty && bEmpty) return true;
+            if (aEmpty != bEmpty) return false; // Einer leer, anderer nicht
+            return a.Equals(b, StringComparison.OrdinalIgnoreCase);
+        }
+
+        private static bool PhoneFieldsCompatible(string a, string b)
+        {
+            bool aEmpty = string.IsNullOrEmpty(a);
+            bool bEmpty = string.IsNullOrEmpty(b);
+
+            if (aEmpty && bEmpty) return true;
+            if (aEmpty != bEmpty) return false;
+            return NormalizePhone(a) == NormalizePhone(b);
+        }
+
         private static string NormalizePhone(string phone)
         {
             if (string.IsNullOrEmpty(phone)) return "";
@@ -144,6 +170,45 @@ namespace StarfaceOutlookSync.Services
                         if (oc == null && sc == null)
                         {
                             // Beide Seiten geloescht -> Mapping entfernen
+                            Log($"  Mapping verwaist (beide geloescht), entferne");
+                            continue;
+                        }
+
+                        if (oc == null && sc != null)
+                        {
+                            // In Outlook geloescht -> in Starface auch loeschen
+                            if (profile.SyncDirection == SyncDirection.Both || profile.SyncDirection == SyncDirection.OutlookToStarface)
+                            {
+                                if (await starface.DeleteContactAsync(mapping.StarfaceId))
+                                {
+                                    result.Updated++;
+                                    Log($"  Geloescht (OL->SF): {sc.DisplayName}");
+                                }
+                            }
+                            else
+                            {
+                                // Richtung erlaubt kein Loeschen -> Mapping behalten
+                                newMappings.Add(mapping);
+                            }
+                            continue;
+                        }
+
+                        if (oc != null && sc == null)
+                        {
+                            // In Starface geloescht -> in Outlook auch loeschen
+                            if (profile.SyncDirection == SyncDirection.Both || profile.SyncDirection == SyncDirection.StarfaceToOutlook)
+                            {
+                                if (_outlookService.DeleteContact(mapping.OutlookEntryId))
+                                {
+                                    result.Updated++;
+                                    Log($"  Geloescht (SF->OL): {oc.DisplayName}");
+                                }
+                            }
+                            else
+                            {
+                                // Richtung erlaubt kein Loeschen -> Mapping behalten
+                                newMappings.Add(mapping);
+                            }
                             continue;
                         }
 

src/StarfaceOutlookSync/StarfaceOutlookSync.csproj

@@ -7,9 +7,9 @@
     <AssemblyTitle>Starface Outlook Sync</AssemblyTitle>
     <Company>HackerSoft - Hacker-Net Telekommunikation</Company>
     <Product>Starface Outlook Sync</Product>
-    <Version>0.0.0.15</Version>
-    <AssemblyVersion>0.0.0.15</AssemblyVersion>
-    <FileVersion>0.0.0.15</FileVersion>
+    <Version>0.0.0.18</Version>
+    <AssemblyVersion>0.0.0.18</AssemblyVersion>
+    <FileVersion>0.0.0.18</FileVersion>
     <Description>Synchronisiert Outlook-Kontakte mit Starface Telefonanlage</Description>
     <Copyright>Stefan Hacker - HackerSoft</Copyright>
     <RuntimeIdentifier>win-x64</RuntimeIdentifier>

src/StarfaceOutlookSync/UI/AboutForm.cs

@@ -27,7 +27,7 @@ namespace StarfaceOutlookSync.UI
 
             var lblVersion = new Label
             {
-                Text = "Version 0.0.0.15",
+                Text = "Version 0.0.0.18",
                 Left = 0, Top = 56, Width = 340, Height = 20,
                 TextAlign = ContentAlignment.MiddleCenter,
                 ForeColor = Color.Gray

src/StarfaceOutlookSync/UI/ProfileEditorForm.cs

@@ -333,9 +333,23 @@ namespace StarfaceOutlookSync.UI
             };
 
             if (_isNew)
+            {
                 _pm.AddProfile(profile);
+            }
             else
+            {
+                // Wenn Adressbuch gewechselt wurde, Mappings zuruecksetzen
+                if (_existingProfile.StarfaceAddressBook?.TagId != profile.StarfaceAddressBook?.TagId
+                    || _existingProfile.StarfaceAddressBook?.Type != profile.StarfaceAddressBook?.Type)
+                {
+                    _pm.SaveMappings(profile.Id, new List<SyncMapping>());
+                    profile.LastSync = "";
+                    MessageBox.Show(
+                        "Adressbuch wurde geaendert.\nSync-Zuordnungen wurden automatisch zurueckgesetzt.",
+                        "Adressbuch geaendert", MessageBoxButtons.OK, MessageBoxIcon.Information);
+                }
                 _pm.UpdateProfile(profile);
+            }
 
             DialogResult = DialogResult.OK;
             Close();