Fix for #619. don't report property deletes as changes when they have… (

#621)

* Fix for #619. don't report property deletes as changes when they have already happened

* Add fix for #620 - don't recreate items that are deleted.

uSync.Core/Tracking/SyncRootMergerHelper.cs

@@ -4,6 +4,8 @@
 using System.Xml.Linq;
 using System.Xml.XPath;
 
+using Serilog.Core;
+
 namespace uSync.Core.Tracking;
 
 public class SyncRootMergerHelper
@@ -26,11 +28,19 @@ public static (XElement combined, XElement differences) CompareNodes(List<XEleme
         var differences = XElement.Parse(nodes[^1].ToString());
         var combined = XElement.Parse(nodes[^1].ToString());
 
+        // latest is a blank one, that is a delete so should 
+        // be marked as one.
+        if (combined.IsEmptyItem())
+            return (combined, BlankNode(differences));
+
         foreach (var node in nodes[..^1])
         {
+            // 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 +162,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();
             }
         }