Media type is being inserted into ./otherEntity/entityType element

[gothub]

The media type for data files newly added via the editor is being inserted into the EML //otherEntity/entityType element.

The media type could be entered instead into the EML element //otherEntity/physical/dataFormat/externallyDefinedFormat/formatName

For an example, see

• dataset: https://dev.nceas.ucsb.edu/view/urn%3Auuid%3A369a5ef6-ed98-4187-8800-c504d18a5639
• metadata: https://dev.nceas.ucsb.edu/knb/d1/mn/v2/object/urn:uuid:369a5ef6-ed98-4187-8800-c504d18a5639

Here is the section inserted by MetacatUI:

<otherEntity id="urn-uuid-bb476d98-f762-41d9-9509-3fca430887a7">
  <entityName>Scooter_blanket.jpg</entityName>
  <entityDescription>Cute dog photo</entityDescription>
  <entityType>image/jpeg</entityType>
</otherEntity>

which instead could be written as:

<otherEntity id="urn-uuid-bb476d98-f762-41d9-9509-3fca430887a7>
  <entityName>Scooter_blanket.jpg</entityName>
  <entityDescription>Cute dog photo</entityDescription>
  <entityType>JPEG Image</entityType>
  <physical>
    <objectName>Scooter_blanket.jpg</objectName>
    <size unit="byte">87616</size>
    <dataFormat>
      <externallyDefinedFormat>
        <formatName>image/jpeg</formatName>
      </externallyDefinedFormat>
    </dataFormat>
  </physical>
</otherEntity>

For further context on how this issue originated, see NCEAS/metadig-checks#64 (comment)

[jeanetteclark]

I believe entityType is required for otherEntity elements so something has to go there.

A more broad issue to create physical sections is #450, which I would still love to see implemented

[vchendrix]

@vchendrix adding the <physcial> element seems fine but to make it backwards compatible <entityType/> should still be the mime type. and not a human friendly description.