Serialization of attributes

invisibleXML / ixml

Invisible XML

GNU General Public License v3.0

51 stars 7 forks source link

Serialization of attributes #263

Open ndw opened 3 months ago

ndw commented 3 months ago

The spec says:

Attribute: the node is serialized as an XML attribute whose:

name is the alias of the node if present, or the alias of the referred-to rule, if it has one, and otherwise the name of the node,

value is the serialization of all non-deleted terminal descendants of the node (regardless of the marking of intermediate nonterminals), if any, in order.

Where "all non-deleted terminal descendants" does not appear to cover the case of insertions.

GuntherRademacher commented 3 months ago

Looks like it. But there seems to be consensus that the insertions should be covered: I tried CoffeePot, ixampl, jωiXML, and Markup Blitz, and everyone includes them in attribute content.

Also, these tests will fail, if we don't:

Insertion tests/insert-attribute/iAtt-1
Insertion tests/insert-example/iEx-1
Insertion tests/insert-multiple-attribute/iMultAtt-1
Insertion tests/insert-special/insert-special
Character class tests/chars.astral/chars.astral.1

ndw commented 3 months ago

Yes, I don't think there's likely to be any actual dispute that they should, it's just not technically what the spec says at the moment.

ndw commented 2 months ago

The CG agreed this is a bug on 2024-08-06, SP took the action to make a PR.