Closed jsjiang closed 6 months ago
I'm still trying to get my local environment set up, so wasn't able to test, but it looks ok, I believe.
@rushirajnenuji @sfisher Hi Rushiraj and Scott, The newer commits are fixes and tests for removing namespace prefixes from XML records. The added code only converts XML records for citation preview. It does not affect other functionalities or record in the database.
You can see the changes by comparing the outputs from stg and dev:
v3.2.3rc0 on ezid-stg without fix for removing namespace prefixes: https://ezid-stg.cdlib.org/id/doi:10.24343/S3690H - citation preview is broken
branch 407_fix_datacite_citation_preview_v1 on ezid-dev with fix for removing namespace prefixes: http://ezid-dev.cdlib.org/id/ark:/99999/fk4zg85c0j - citation preview is not broken
Please review and let me know if you have questions.
Thank you
Jing
@rushirajnenuji @sfisher Hi Rushiraj and Scott, Here are the major changes for issue ticket:
xmltodict
for converting Datacite XML record to Python dictionarydcmsRecordToDict(record)
to convert Datacite XML record to dictbriefDataciteRecord(record)
to convert Datacite XML record to a simplified format with data fields required for citation preview.briefDataciteRecord(record)
The changes are on ezid-dev (http://ezid-dev.cdlib.org/). Please review and let me know if you have questions.
Thank you
Jing