From 7d1860e608496ad28cadc173bc7eea47e653600a Mon Sep 17 00:00:00 2001
From: Kevin Jump <kevin@thejumps.co.uk>
Date: Tue, 16 Apr 2024 16:29:18 +0100
Subject: [PATCH] Fix for #619. don't report property deletes as changes when
 they have already happened

---
 uSync.Core/Tracking/SyncRootMergerHelper.cs | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/uSync.Core/Tracking/SyncRootMergerHelper.cs b/uSync.Core/Tracking/SyncRootMergerHelper.cs
index cb52b0e3..8ab418be 100644
--- a/uSync.Core/Tracking/SyncRootMergerHelper.cs
+++ b/uSync.Core/Tracking/SyncRootMergerHelper.cs
@@ -28,9 +28,17 @@ public static (XElement combined, XElement differences) CompareNodes(List<XEleme
 
         foreach (var node in nodes[..^1])
         {
+            // if the later node is a delete/rename, it always wins
+            // there is nothing to merge
+            if (node.IsEmptyItem())
+                return (node, BlankNode(differences));
+
+            // if this node is the same as the differences we already have, 
+            // return it. 
             if (node.ToString() == differences.ToString())
                 return (combined, BlankNode(differences));
 
+            // workout any merged diffrences, 
             (combined, differences) = GetTrackedNodeDifferences(node, combined, trackedNodes);
         }
         return (combined, differences);
@@ -152,8 +160,13 @@ private static (XElement combined, XElement diffrence) GetMultipleChanges(Tracki
             else
             {
                 var replacement = FindByKey(combinedCollection, element, item.Keys, key);
-                replacement?.AddAfterSelf(targetElement);
-                replacement?.Remove();
+
+				// only add this if its not a delete
+				if (targetElement.Attribute("deleted").ValueOrDefault(false) is false)
+                {
+					replacement?.AddAfterSelf(targetElement);
+				}
+				replacement?.Remove();
             }
         }