dataverse icon indicating copy to clipboard operation
dataverse copied to clipboard

OAIDC export use publication date if prodcution date is not set

Open tcoupin opened this issue 2 years ago • 2 comments

What this PR does / why we need it:

Production date is used as date tag when export dataset to oai_dc. This PR use the publication date if production is not set.

Which issue(s) this PR closes:

Related to #8103 and #8732

Special notes for your reviewer:

Suggestions on how to test this:

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?:

Additional documentation:

tcoupin avatar May 24 '22 16:05 tcoupin

Hi @tcoupin, for this next sprint we are catching up on Community PRs. Would you mind updating/ refreshing thiis PR from develop? Thanks!

scolapasta avatar Aug 04 '22 19:08 scolapasta

I rebase on develop branch

tcoupin avatar Aug 04 '22 20:08 tcoupin

Coverage Status

Coverage decreased (-0.0005%) to 19.88% when pulling 97cf6bc5445ebf8daab3cb02029361d1e89a4abb on tcoupin:oaidc-date into 9686ab86f933b794543c2ae80fc74694f05608ab on IQSS:develop.

coveralls avatar Sep 13 '22 05:09 coveralls

Looks good! Merged! Thanks @sekmiller @jggautier and @landreev for the discussion.

At release time, @jggautier plans to update our crosswalk doc to explain the new date handling. I linked to the doc in the release note snippet I added: 97cf6bc

We also discussed a bit how the oai_dc format is not available in the dropdown as an export. However you can hack the URL to get to it: https://dataverse.harvard.edu/api/datasets/export?exporter=oai_dc&persistentId=doi%3A10.7910/DVN/TJCLKP . If we ever want to change this, it's just a matter of flipping the isAvailableToUsers boolean in DublinCoreExporter:

https://github.com/IQSS/dataverse/blob/v5.12/src/main/java/edu/harvard/iq/dataverse/export/DublinCoreExporter.java#L52-L54

Anyway, thanks, @tcoupin! Please keep the pull requests coming! 🎉

pdurbin avatar Oct 13 '22 15:10 pdurbin