Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle MI_Metadata from CSW record XML #210

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

benjwadams
Copy link

Handles MI_Metadata elements returned from CSW XML records, rather than
returning None and causing an error later. Raises an exception
when neither MD_Metadata nor MI_Metadata are found. Also sets encoding
argument to 'unicode' rather than unicode (type), so that Python 3 does
not break.

Handles MI_Metadata elements returned from CSW XML records, rather than
returning None and causing an error later.  Raises an exception
when neither MD_Metadata nor MI_Metadata are found.  Also sets encoding
argument to 'unicode' rather than unicode (type), so that Python 3 does
not break.
@amercader
Copy link
Member

This looks good. I'll wait until the tests are fixed on master to double check is not breaking anything and I'll merge it.

ccancellieri added a commit to ccancellieri/ckanext-spatial that referenced this pull request Oct 22, 2021
…mespace and tag name so we can now harvest any kind of metadata (if validation is provided or ignored) ckan#209 ckan#210 ckan#219 ckan#258
ccancellieri added a commit to ccancellieri/ckanext-spatial that referenced this pull request Oct 22, 2021
…mespace and tag name so we can now harvest any kind of metadata (if validation is provided or ignored) ckan#209 ckan#210 ckan#219 ckan#258
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants