[OMP] Add related ISBN data to ONIX export

[asmecher]

DOAB would like related ISBN data added to the ONIX export.

They recommend the following schema:

RelatedMaterial/RelatedProduct/ProductIdentifier[ProductIDType=15]/IDValue or else RelatedProduct/ProductIdentifier[ProductIDType=15]/IDValue

This would be used to link other editions of the same book.

Related documentation:

• ONIX Application Note: Related products and other product links

[kaitlinnewson]

Which ISBNs within OMP are considered to be for other editions of the same book? In the Onix export metadata, when there are multiple publication formats with different ISBNs these are all included as separate products with their own ISBN if it's included in the metadata. Is there another location for this type of ISBN?

[asmecher]

Hmm, probably a good question for DOAB. I'll get you into the appropriate Mattermost channel to ask them directly.

[asmecher]

...but as for where you can enter ISBNs in OMP: there's the publication format form...

...and the "Product Identification" grid on the Metadata tab of the same area:

When you enter ISBNs in the publication format edit form, they show up in the grid, i.e. they aren't different data sources.

(screenshots from pre-3.5.0 -- 3.3.0 and 3.4.0 will be a little different)

[kaitlinnewson]

Thanks Alec - I found the place for ISBNs, but they already seem to be in the Onix export, so I'm trying to understand which ISBNs are appropriate for this "RelatedMaterial" field.

[kaitlinnewson]

PR ready for review and will forward-port once approved:

• OMP 3.3: pkp/pkp-lib#9928 add related identifiers to onix exports omp#1756
• OMP 3.4: pkp/pkp-lib#9928 add related identifiers to onix exports omp#1764
• OMP main: pkp/pkp-lib#9928 add related identifiers to onix exports omp#1765

This PR collects the identifiers for all publication formats being exported, and creates a <RelatedMaterial> element which contains the identifiers for all other publication formats that belong to the publication. This may be an ISBN, or any other identifier.

@bozana are you able to review?

[kaitlinnewson]

@bozana forward-ported and PRs are listed above.

[bozana]

All merged, thus closing...
Thanks a lot!