Fix sync duplicates and extreme slowness
Behebt Dubletten auf beiden Seiten und sehr langsame Syncs: - Getrennte Hash-Baselines pro Seite (LastOutlookHash/LastStarfaceHash) statt eines gemeinsamen Hashes. Outlook und Starface stellen denselben Kontakt unterschiedlich dar, wodurch der gemeinsame Hash nie passte und bei jedem Lauf praktisch jeder Kontakt neu geschrieben wurde. - Update-Methoden geben den frisch eingelesenen Stand zurueck, damit die Baseline nach dem Schreiben korrekt gesetzt wird (sauberes Konvergieren). - Unvollstaendig geladene Starface-Liste bricht jetzt mit Fehler ab (inkl. Retry) statt still mit Teil-Liste weiterzuarbeiten - das liess Kontakte faelschlich als geloescht erscheinen und erzeugte Dubletten. - Fehlender Starface-Kontakt (anderes Adressbuch) behaelt das Mapping, statt es zu verwerfen und neu anzulegen. - Lockereres Re-Matching: gleicher E-Mail- oder voller Namens-Treffer reicht; umformatierte Telefonnummern blockieren ihn nicht mehr. - Starface-Kontaktdetails werden parallel geladen (8 gleichzeitig). Bestehende Mappings werden beim ersten Sync automatisch migriert. CHANGELOG.md hinzugefuegt. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Claude Opus 4.8
@@ -37,72 +37,38 @@ namespace StarfaceOutlookSync.Services
|
||||
|
||||
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);
|
||||
bool hasName = (!string.IsNullOrEmpty(a.FirstName) || !string.IsNullOrEmpty(a.LastName))
|
||||
&& (!string.IsNullOrEmpty(b.FirstName) || !string.IsNullOrEmpty(b.LastName));
|
||||
|
||||
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/Fax: wenn auf einer Seite vorhanden, muss es gleich sein
|
||||
if (!PhoneFieldsCompatible(a.PhoneWork, b.PhoneWork)) return false;
|
||||
if (!PhoneFieldsCompatible(a.PhoneMobile, b.PhoneMobile)) return false;
|
||||
if (!PhoneFieldsCompatible(a.PhoneHome, b.PhoneHome)) return false;
|
||||
if (!PhoneFieldsCompatible(a.Fax, b.Fax)) return false;
|
||||
|
||||
// Mindestens ein starkes Match muss vorhanden sein
|
||||
// Starke Identifikatoren
|
||||
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));
|
||||
&& (a.FirstName ?? "").Equals(b.FirstName ?? "", StringComparison.OrdinalIgnoreCase)
|
||||
&& (a.LastName ?? "").Equals(b.LastName ?? "", StringComparison.OrdinalIgnoreCase);
|
||||
bool phoneMatch = (!string.IsNullOrEmpty(a.PhoneWork) && !string.IsNullOrEmpty(b.PhoneWork)
|
||||
&& NormalizePhone(a.PhoneWork) == NormalizePhone(b.PhoneWork))
|
||||
|| (!string.IsNullOrEmpty(a.PhoneMobile) && !string.IsNullOrEmpty(b.PhoneMobile)
|
||||
&& NormalizePhone(a.PhoneMobile) == NormalizePhone(b.PhoneMobile))
|
||||
|| (!string.IsNullOrEmpty(a.Fax) && !string.IsNullOrEmpty(b.Fax)
|
||||
&& NormalizePhone(a.Fax) == NormalizePhone(b.Fax));
|
||||
bool companyMatch = !string.IsNullOrEmpty(a.Company) && !string.IsNullOrEmpty(b.Company)
|
||||
&& a.Company.Equals(b.Company, StringComparison.OrdinalIgnoreCase);
|
||||
|
||||
// Email oder Name reicht. Telefon/Fax nur mit Firma zusammen.
|
||||
return emailMatch || nameMatch || (phoneMatch && companyMatch) || (companyMatch && phoneMatch);
|
||||
}
|
||||
// Widerspruch: beide haben eine E-Mail, aber unterschiedlich -> verschiedene Personen.
|
||||
bool emailContradiction = !string.IsNullOrEmpty(a.Email) && !string.IsNullOrEmpty(b.Email) && !emailMatch;
|
||||
|
||||
/// <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);
|
||||
// Gleiche E-Mail ist der staerkste Identifikator und reicht allein.
|
||||
if (emailMatch) return true;
|
||||
|
||||
if (aEmpty && bEmpty) return true;
|
||||
if (aEmpty != bEmpty) return false; // Einer leer, anderer nicht
|
||||
return a.Equals(b, StringComparison.OrdinalIgnoreCase);
|
||||
}
|
||||
// Gleicher voller Name reicht, solange keine widerspruechliche E-Mail vorliegt.
|
||||
// (Telefon-Umformatierung durch Starface darf einen Namens-Treffer NICHT verhindern.)
|
||||
if (nameMatch && !emailContradiction) return true;
|
||||
|
||||
private static bool PhoneFieldsCompatible(string a, string b)
|
||||
{
|
||||
bool aEmpty = string.IsNullOrEmpty(a);
|
||||
bool bEmpty = string.IsNullOrEmpty(b);
|
||||
// Schwacher Pfad: Telefon/Fax nur zusammen mit gleicher Firma und ohne E-Mail-Widerspruch.
|
||||
if (phoneMatch && companyMatch && !emailContradiction) return true;
|
||||
|
||||
if (aEmpty && bEmpty) return true;
|
||||
if (aEmpty != bEmpty) return false;
|
||||
return NormalizePhone(a) == NormalizePhone(b);
|
||||
return false;
|
||||
}
|
||||
|
||||
private static string NormalizePhone(string phone)
|
||||
@@ -215,29 +181,50 @@ namespace StarfaceOutlookSync.Services
|
||||
|
||||
if (oc != null && sc == null)
|
||||
{
|
||||
// Starface-Kontakt nicht gefunden.
|
||||
// Kann passieren wenn der Kontakt einem anderen Adressbuch gehoert.
|
||||
// NICHT loeschen, nur Mapping entfernen - wird in Phase 2/3 neu verknuepft
|
||||
Log($" Starface-Kontakt nicht in Liste (anderes Adressbuch?): {oc.DisplayName}");
|
||||
// Mapping verwerfen, Outlook-Kontakt als unverarbeitet belassen
|
||||
// damit er in Phase 2 neu zugeordnet oder erstellt werden kann
|
||||
// Starface-Kontakt nicht in der geladenen Liste.
|
||||
// Da unvollstaendige Ladevorgaenge inzwischen abgebrochen
|
||||
// werden (siehe StarfaceApiClient), liegt das hoechstens an
|
||||
// einem anderen Adressbuch. NICHT loeschen und NICHT neu
|
||||
// anlegen - sonst entstehen Dubletten. Mapping behalten,
|
||||
// beim naechsten Sync wird es erneut abgeglichen.
|
||||
Log($" Starface-Kontakt nicht in Liste (anderes Adressbuch?), behalte Mapping: {oc.DisplayName}");
|
||||
newMappings.Add(mapping);
|
||||
continue;
|
||||
}
|
||||
|
||||
if (oc != null && sc != null)
|
||||
{
|
||||
// Beide vorhanden -> auf Aenderungen pruefen
|
||||
// Beide vorhanden -> auf Aenderungen pruefen.
|
||||
// WICHTIG: jede Seite gegen ihre EIGENE Baseline pruefen.
|
||||
// Outlook und Starface stellen denselben Kontakt
|
||||
// unterschiedlich dar, ein gemeinsamer Hash schlaegt nie an.
|
||||
var olHash = oc.GetHash();
|
||||
var sfHash = sc.GetHash();
|
||||
bool olChanged = olHash != mapping.LastSyncHash;
|
||||
bool sfChanged = sfHash != mapping.LastSyncHash;
|
||||
|
||||
// Migration alter Mappings (nur LastSyncHash vorhanden):
|
||||
// aktuellen Stand als Baseline uebernehmen und als synchron
|
||||
// annehmen, damit kein Massen-Update ausgeloest wird.
|
||||
if (string.IsNullOrEmpty(mapping.LastOutlookHash) &&
|
||||
string.IsNullOrEmpty(mapping.LastStarfaceHash))
|
||||
{
|
||||
mapping.LastOutlookHash = olHash;
|
||||
mapping.LastStarfaceHash = sfHash;
|
||||
mapping.LastSyncHash = "";
|
||||
newMappings.Add(mapping);
|
||||
continue;
|
||||
}
|
||||
|
||||
bool olChanged = olHash != mapping.LastOutlookHash;
|
||||
bool sfChanged = sfHash != mapping.LastStarfaceHash;
|
||||
|
||||
if (olChanged && !sfChanged && (profile.SyncDirection == SyncDirection.Both || profile.SyncDirection == SyncDirection.OutlookToStarface))
|
||||
{
|
||||
// Outlook hat sich geaendert -> Starface updaten
|
||||
if (await starface.UpdateContactAsync(mapping.StarfaceId, oc, profile.StarfaceAddressBook))
|
||||
var updated = await starface.UpdateContactAsync(mapping.StarfaceId, oc, profile.StarfaceAddressBook);
|
||||
if (updated != null)
|
||||
{
|
||||
mapping.LastSyncHash = olHash;
|
||||
mapping.LastOutlookHash = olHash;
|
||||
mapping.LastStarfaceHash = updated.GetHash();
|
||||
result.Updated++;
|
||||
Log($" Aktualisiert (OL->SF): {oc.DisplayName}");
|
||||
}
|
||||
@@ -245,30 +232,36 @@ namespace StarfaceOutlookSync.Services
|
||||
else if (sfChanged && !olChanged && (profile.SyncDirection == SyncDirection.Both || profile.SyncDirection == SyncDirection.StarfaceToOutlook))
|
||||
{
|
||||
// Starface hat sich geaendert -> Outlook updaten
|
||||
if (_outlookService.UpdateContact(mapping.OutlookEntryId, sc))
|
||||
var updated = _outlookService.UpdateContact(mapping.OutlookEntryId, sc);
|
||||
if (updated != null)
|
||||
{
|
||||
mapping.LastSyncHash = sfHash;
|
||||
mapping.LastStarfaceHash = sfHash;
|
||||
mapping.LastOutlookHash = updated.GetHash();
|
||||
result.Updated++;
|
||||
Log($" Aktualisiert (SF->OL): {sc.DisplayName}");
|
||||
}
|
||||
}
|
||||
else if (olChanged && sfChanged)
|
||||
{
|
||||
// Beide geaendert -> Konflikt, neuere gewinnt (Outlook bevorzugt)
|
||||
// Beide geaendert -> Konflikt, Outlook bevorzugt
|
||||
if (profile.SyncDirection != SyncDirection.StarfaceToOutlook)
|
||||
{
|
||||
if (await starface.UpdateContactAsync(mapping.StarfaceId, oc, profile.StarfaceAddressBook))
|
||||
var updated = await starface.UpdateContactAsync(mapping.StarfaceId, oc, profile.StarfaceAddressBook);
|
||||
if (updated != null)
|
||||
{
|
||||
mapping.LastSyncHash = olHash;
|
||||
mapping.LastOutlookHash = olHash;
|
||||
mapping.LastStarfaceHash = updated.GetHash();
|
||||
result.Updated++;
|
||||
Log($" Konflikt (OL gewinnt): {oc.DisplayName}");
|
||||
}
|
||||
}
|
||||
else
|
||||
{
|
||||
if (_outlookService.UpdateContact(mapping.OutlookEntryId, sc))
|
||||
var updated = _outlookService.UpdateContact(mapping.OutlookEntryId, sc);
|
||||
if (updated != null)
|
||||
{
|
||||
mapping.LastSyncHash = sfHash;
|
||||
mapping.LastStarfaceHash = sfHash;
|
||||
mapping.LastOutlookHash = updated.GetHash();
|
||||
result.Updated++;
|
||||
Log($" Konflikt (SF gewinnt): {sc.DisplayName}");
|
||||
}
|
||||
@@ -306,14 +299,16 @@ namespace StarfaceOutlookSync.Services
|
||||
if (match != null)
|
||||
{
|
||||
// Existiert schon -> verknuepfen und updaten
|
||||
if (await starface.UpdateContactAsync(match.StarfaceId, oc, profile.StarfaceAddressBook))
|
||||
var updated = await starface.UpdateContactAsync(match.StarfaceId, oc, profile.StarfaceAddressBook);
|
||||
if (updated != null)
|
||||
{
|
||||
newMappings.Add(new SyncMapping
|
||||
{
|
||||
ProfileId = profile.Id,
|
||||
OutlookEntryId = oc.OutlookEntryId,
|
||||
StarfaceId = match.StarfaceId,
|
||||
LastSyncHash = oc.GetHash()
|
||||
LastOutlookHash = oc.GetHash(),
|
||||
LastStarfaceHash = updated.GetHash()
|
||||
});
|
||||
processedStarfaceIds.Add(match.StarfaceId);
|
||||
unmappedStarface.Remove(match);
|
||||
@@ -333,7 +328,8 @@ namespace StarfaceOutlookSync.Services
|
||||
ProfileId = profile.Id,
|
||||
OutlookEntryId = oc.OutlookEntryId,
|
||||
StarfaceId = created.StarfaceId,
|
||||
LastSyncHash = oc.GetHash()
|
||||
LastOutlookHash = oc.GetHash(),
|
||||
LastStarfaceHash = created.GetHash()
|
||||
});
|
||||
result.Created++;
|
||||
Log($" Erstellt (OL->SF): {oc.DisplayName}");
|
||||
@@ -379,14 +375,16 @@ namespace StarfaceOutlookSync.Services
|
||||
if (match != null)
|
||||
{
|
||||
// Existiert schon -> verknuepfen und updaten
|
||||
if (_outlookService.UpdateContact(match.OutlookEntryId, sc))
|
||||
var updated = _outlookService.UpdateContact(match.OutlookEntryId, sc);
|
||||
if (updated != null)
|
||||
{
|
||||
newMappings.Add(new SyncMapping
|
||||
{
|
||||
ProfileId = profile.Id,
|
||||
OutlookEntryId = match.OutlookEntryId,
|
||||
StarfaceId = sc.StarfaceId,
|
||||
LastSyncHash = sc.GetHash()
|
||||
LastStarfaceHash = sc.GetHash(),
|
||||
LastOutlookHash = updated.GetHash()
|
||||
});
|
||||
processedOutlookIds.Add(match.OutlookEntryId);
|
||||
unmappedOutlook.Remove(match);
|
||||
@@ -405,7 +403,8 @@ namespace StarfaceOutlookSync.Services
|
||||
ProfileId = profile.Id,
|
||||
OutlookEntryId = created.OutlookEntryId,
|
||||
StarfaceId = sc.StarfaceId,
|
||||
LastSyncHash = sc.GetHash()
|
||||
LastStarfaceHash = sc.GetHash(),
|
||||
LastOutlookHash = created.GetHash()
|
||||
});
|
||||
result.Created++;
|
||||
Log($" Erstellt (SF->OL): {sc.DisplayName}");
|
||||
|
||||
Reference in New Issue
Block a user