Problem writing a changed DeepCopied document

Hello! I have a problem writing a changed DeepCopied document.

Description

I have an original SVG file, that looks like that I open it and make a deep copy of it:

var sampleDoc = SvgDocument.Open(Path.Combine(AppDomain.CurrentDomain.BaseDirectory, @"../../../sample_closed.svg"));
var newDoc = (SvgDocument)sampleDoc.DeepCopy<SvgDocument>();
SvgPath newPath = (SvgPath)newDoc.Children[1].Children[1];

Then I make some changes to the path elements of the deep copied document. For example:

newPath.PathData[1] = new PointF(newPath.PathData[1].End.X - 1000, newPath.PathData[1].End.Y - 1000);

After that, I call Draw method to get changed png file as a result:

newDoc.Draw().Save(Path.Combine(AppDomain.CurrentDomain.BaseDirectory, @"result.png"));

And it works fine:

Then I try to save changed svg, calling Write method:

newDoc.Write(Path.Combine(AppDomain.CurrentDomain.BaseDirectory, @"result.svg"));

But I get a svg file without any change as a result

Is it a bug or I do something wrong?

Used Versions

Target framework: .net 6.0

[mrbean-bremen]

I had a look at this, and there seem to be two problems:

• the writing is done based on the Nodes of the object, that are not created on adding an element to Children
• DeepCopy does not check for equal references, so the same object referenced both as a child and as a node will end up as two different objects after the deep copy - updating the nodes will not have the expected effect

I think that at least the second one is a bug, and the first one is either a bug, or some missing convienence method, or at least documentation.

[mrbean-bremen]

I would rather not have Nodes because they are complicating things.

Fair point.

However, only SvgTextBase is designed based on Nodes, and it is difficult to redesign it…

Well, the problem happens for other SvgElements too, in this case SvgPath, because Nodes are used for writing documents.