ascmitc / mhl

ASC Media Hash List
MIT License
58 stars 8 forks source link

Fix author info to conform to structure in spec #150

Closed ptrpfn closed 2 months ago

ptrpfn commented 2 months ago

Currently the ascmhl create command writes wrong syntax for author info (as part of <creatorinfo> tag):

    <author>
      <name>John Doe</name>
      <role>Data wrangler</role>
      <email>john@mail.com</email>
      <phone>+1 123 456789</phone>
    </author>

Changed to updated syntax:

    <author role="Data wrangler" email="john@mail.com" phone="+1 123 456789">John Doe</author>

Please not that backwards compatibility is maintained in the ascmhl commands, so ASC MHL histories with the wrong/old syntax above can still be read. This avoids that any previously written histories get unusable.

Also the order of tags within the <creatorinfo> is fixed to conform to the XSD.

ptrpfn commented 2 months ago

Did some more testing, looks good so far.