fix(XMLPatch): fixed replace operation for elements

refactor(XMLPatch): enhance logging for add, replace, and remove operations in XMLPatch
docs: updated Changelog in README's

Author: chemodun

Docs/README.egosoft

@@ -245,6 +245,40 @@ To create such debug file please use the [b]--log-to-file[/b] option.
 
 There is a MIT license for this tool. You can find it in the [url=https://github.com/chemodun/x4_XMLDiffAndPatch/raw/main//LICENSE]LICENSE[/url] file.
 
+[size=130][u]Changelog[/u][/size]
+
+[size=125][u][0.2.16] - 2025-02-25[/u][/size]
+[list]
+[*] Fixed
+[list]
+[*] Fixed issue with element replacements
+[/list]
+[*] Improved
+[list]
+[*] Logging information
+[/list]
+[/list]
+[size=125][u][0.2.15] - 2025-02-24[/u][/size]
+[list]
+[*] Fixed
+[list]
+[*] Fixed loading the diff.xsd
+[*] Fixed issue, if resulting file has to be located in current folder
+[/list]
+[*] Improved
+[/list]  Logging information, especially about wrong sel value. More info logged about processed XML elements.
+[list][*] Added
+[list]
+[*] Possibility to append into existing debug log
+[/list]
+[/list]
+[size=125][u][0.2.14] - 2025-01-17[/u][/size]
+[list]
+[*] Added
+[list]
+[*] First public version coded in C#.
+[/list]
+[/list]
 [size=130][u]Additional links[/u][/size]
 
 There is a topic on the [url=https://forum.egosoft.com/viewtopic.php?t=468623]EGOSOFT forum[/url], related to this toolset.

Docs/README.nexus

@@ -245,6 +245,40 @@ To create such debug file please use the [b]--log-to-file[/b] option.
 
 There is a MIT license for this tool. You can find it in the [url=https://github.com/chemodun/x4_XMLDiffAndPatch/raw/main//LICENSE]LICENSE[/url] file.
 
+[size=3][u][b]Changelog[/b][/u][/size]
+
+[size=3][u][i][0.2.16] - 2025-02-25[/i][/u][/size]
+[list]
+[*] Fixed
+[list]
+[*] Fixed issue with element replacements
+[/list]
+[*] Improved
+[list]
+[*] Logging information
+[/list]
+[/list]
+[size=3][u][i][0.2.15] - 2025-02-24[/i][/u][/size]
+[list]
+[*] Fixed
+[list]
+[*] Fixed loading the diff.xsd
+[*] Fixed issue, if resulting file has to be located in current folder
+[/list]
+[*] Improved
+[/list]  Logging information, especially about wrong sel value. More info logged about processed XML elements.
+[list][*] Added
+[list]
+[*] Possibility to append into existing debug log
+[/list]
+[/list]
+[size=3][u][i][0.2.14] - 2025-01-17[/i][/u][/size]
+[list]
+[*] Added
+[list]
+[*] First public version coded in C#.
+[/list]
+[/list]
 [size=3][u][b]Additional links[/b][/u][/size]
 
 There is a topic on the [url=https://forum.egosoft.com/viewtopic.php?t=468623]EGOSOFT forum[/url], related to this toolset.

README.md

@@ -243,6 +243,30 @@ To create such debug file please use the `--log-to-file` option.
 
 There is a MIT license for this tool. You can find it in the [LICENSE](LICENSE) file.
 
+## Changelog
+
+### [0.2.16] - 2025-02-25
+
+- Fixed
+  - Fixed issue with element replacements
+- Improved
+  - Logging information
+
+### [0.2.15] - 2025-02-24
+
+- Fixed
+  - Fixed loading the diff.xsd
+  - Fixed issue, if resulting file has to be located in current folder
+- Improved
+  Logging information, especially about wrong sel value. More info logged about processed XML elements.
+- Added
+  - Possibility to append into existing debug log
+
+### [0.2.14] - 2025-01-17
+
+- Added
+  - First public version coded in C#.
+
 ## Additional links
 
 There is a topic on the [EGOSOFT forum](https://forum.egosoft.com/viewtopic.php?t=468623), related to this toolset.

XMLPatch/Program.cs

@@ -345,6 +345,7 @@ private static int DetectIndentation(string xmlPath)
     private static void ApplyAdd(XElement addElement, XElement originalRoot)
     {
       string? sel = addElement.Attribute("sel")?.Value;
+      Logger.Debug($"Applying add operation: '{sel}'");
       if (sel == null)
       {
         Logger.Warn("Add operation missing 'sel' attribute! Skipping operation.");
@@ -357,7 +358,7 @@ private static void ApplyAdd(XElement addElement, XElement originalRoot)
         pos = "append";
       }
 
-      Logger.Debug($"Applying add operation: '{sel}' at {pos!}");
+      Logger.Info($"Applying add operation: '{sel}' at {pos!}");
 
       var targetElements = originalRoot.XPathSelectElements(sel);
       if (targetElements == null || !targetElements.Any())
@@ -428,6 +429,7 @@ private static void ApplyAdd(XElement addElement, XElement originalRoot)
     private static void ApplyReplace(XElement replaceElement, XElement originalRoot)
     {
       string? sel = replaceElement.Attribute("sel")?.Value;
+      Logger.Info($"Applying replace operation: '{sel}'");
       if (sel == null)
       {
         Logger.Warn("Replace operation missing 'sel' attribute! Skipping operation.");
@@ -447,38 +449,39 @@ private static void ApplyReplace(XElement replaceElement, XElement originalRoot)
       {
         if (targetObj is XElement target)
         {
-          var newContent = replaceElement.Value;
+          string targetName = target.Name.LocalName;
+          XElement? replaceSubElement = replaceElement.Element(targetName);
+          XElement? parent = target.Parent;
           string targetInfo = GetElementInfo(target);
-          if (!string.IsNullOrEmpty(newContent))
+          string parentInfo = GetElementInfo(parent);
+          if (replaceSubElement != null)
           {
-            target.Value = newContent;
-            Logger.Debug($"Replaced text of element '{targetInfo}' with '{newContent}'.");
+            string replaceInfo = GetElementInfo(replaceSubElement);
+            target.ReplaceWith(replaceSubElement);
+            Logger.Info($"Replaced element '{targetInfo}' with '{replaceInfo}' in '{parentInfo}'.");
           }
-
-          var newElement = replaceElement.Element("new");
-          if (newElement != null)
+          else
           {
-            XElement replacement = new XElement(newElement);
-            target.ReplaceWith(replacement);
-            Logger.Info($"Replaced element '{targetInfo}' with '{GetElementInfo(replacement)}'.");
+            Logger.Warn($"Can't process replacement for '{targetInfo}' in '{parentInfo}'. Skipping operation.");
           }
         }
         else if (targetObj is XText textNode)
         {
           textNode.Value = replaceElement.Value;
-          Logger.Debug("Replaced text node.");
+          Logger.Info("Replaced text node.");
         }
         else if (targetObj is XAttribute attr)
         {
           attr.Value = replaceElement.Value;
-          Logger.Debug($"Replaced value of attribute '{attr.Name}' with '{replaceElement.Value}'.");
+          Logger.Info($"Replaced value of attribute '{attr.Name}' with '{replaceElement.Value}'.");
         }
       }
     }
 
     private static void ApplyRemove(XElement removeElement, XElement originalRoot)
     {
       string? sel = removeElement.Attribute("sel")?.Value;
+      Logger.Debug($"Applying remove operation: '{sel}'");
       if (sel == null)
       {
         Logger.Warn("Remove operation missing 'sel' attribute! Skipping operation.");
@@ -499,14 +502,15 @@ private static void ApplyRemove(XElement removeElement, XElement originalRoot)
         if (targetObj is XElement target)
         {
           XElement? parent = target.Parent;
+          string parentInfo = GetElementInfo(parent);
           string targetInfo = GetElementInfo(target);
           if (parent == null)
           {
             Logger.Warn($"Element '{targetInfo}' has no parent. Cannot remove.");
             continue;
           }
           target.Remove();
-          Logger.Debug($"Removed element '{targetInfo}' from '{parent.Name}'.");
+          Logger.Info($"Removed element '{targetInfo}' from '{parentInfo}'.");
         }
         else if (targetObj is XAttribute attr)
         {
@@ -518,12 +522,12 @@ private static void ApplyRemove(XElement removeElement, XElement originalRoot)
             continue;
           }
           attr.Remove();
-          Logger.Debug($"Removed attribute '{attr.Name}' from '{parentInfo}'.");
+          Logger.Info($"Removed attribute '{attr.Name}' from '{parentInfo}'.");
         }
         else if (targetObj is XText textNode)
         {
           textNode.Remove();
-          Logger.Debug("Removed text node.");
+          Logger.Info("Removed text node.");
         }
       }
     }