Text gets appended into <Server></Server>

[Sworddragon]

After upgrading FileZilla from version 3.12.0.2 to version 3.14.1 I'm noticing that sitemanager.xml gets malformed a bit:

• Within a <Server></Server> entry at the end the text that is in <Name></Name> gets appended.
• </Server> gets not indented correctly.

[Tim Kosse]

The XML is well-formed.

[Sworddragon]

Maybe, but the text doesn't belong there.

[Tim Kosse]

There is nothing wrong with the XML. It's in the format FileZilla understands and every standards-compliant XML parser in the whole universe can parse that document just fine.

[Sworddragon]

It seems you don't understand what I'm meaning. Between those 2 FileZilla versions the file got changed with replacing 4 spaces with a tab. But somewhere between those 2 versions a mistake was made. The first is just a minor formation issue by placing a closing tag in the same line of another closing tag. It may not make the file invalid but since the file was designed to be intended to be well-formated this issue shouldn't exist there. The second issue is more problematic as between any <Server></Server> entry besides elements like <Host> or <Port> a non-whitespace text entry is written at the end of it. If there would be a DTD (I wonder why the file isn't linked against one) the XML would be even invalid. I hope it is now clear what I'm meaning.

[Tim Kosse]

Indendation and well-formedness are independent. Indentation has no semantic meaning and is thus irrelevant.

Attached is an XML file with inline DTD that gets parses as completely valid despite having the perceived flaws you mention.

[Tim Kosse]

A completely valid XML file, with DTD.

[Sworddragon]

If this all is no issue can you tell me why <Server></Server> contains additional character data that matches what in <Name></Name> is? This makes no sense and because of the indentation issue at the same time it looks very obviously like an accident. Despite the fact that the indentation issue is still a valid bug, even if only a minor one.

[Tim Kosse]

Backwards compatibility with older version, not all versions use the Name element.

The indentation issue is no issue because the indentation is completely irrelevant. It could just as well be completely absent.

[Sworddragon]

With FileZilla 3.33 I did notice that the compatibility appending does not exist anymore. Also the wrong indentation does not appear anymore too so this issue is fixed.