xproc / 3.0-steps

Repository for change requests to the standard step library and for official extension steps
10 stars 7 forks source link

The description of serialization properties in validate-with-dtd could be improved #637

Open ndw opened 6 days ago

ndw commented 6 days ago

Over the weekend, I read this:

If the document to be stored has a “serialization” property, the serialization is controlled by the merger of the two maps where the entries in the “serialization” property take precedence.

and interpreted it incorrectly. I think something like this might be better:

If the document to be stored has a “serialization” property, the document property map and the serialization option map are combined. Where there are keys in both maps, the value in the document's serialization property takes precedence over the value in the serialization option.

Conal-Tuohy commented 5 days ago

Perhaps it could be glossed in terms of a particular call to the standard map merge function?

e.g. "as if by a call to: map:merge(($document-properties, $serialization), map{'duplicates': 'use-last'})"

xatapult commented 5 days ago

See also #640

ndw commented 4 days ago

Perhaps, but we don't usually define things with XPath code fragments. I'll be inclined to just use prose. Though a note with the XPath expression might serve to clarify.