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

fix missing access service items when parsing dataset #256

Merged

Conversation

SebastianEsp
Copy link

When parsing datasets with a dcat:DataService the "endpoint_url" and "serves_dataset" properties are never added to the resource_dict due to a mislabeled variable name

Fx. a dataset with the following DataService:

<dcat:DataService rdf:about="https://example.com/dataservice1">
  <dcterms:title xml:lang="sv">API exempel</dcterms:title>
  <dcat:keyword xml:lang="sv">bibliotek</dcat:keyword>
  <dcat:keyword xml:lang="en">library</dcat:keyword>
  <dcat:contactPoint rdf:resource="https://example.com/contactpoint1"/>
  <dcterms:license rdf:resource="http://creativecommons.org/publicdomain/zero/1.0/"/>
  <dcterms:accessRights rdf:resource="http://publications.europa.eu/resource/authority/access-right/PUBLIC"/>
  <dcat:endpointURL rdf:resource="http://example.com/api"/>
  <dcat:endpointDescription rdf:resource="http://example.com/apidescription"/>
  <dcat:servesDataset rdf:resource="https://example.com/dataset1" />
</dcat:DataService>

will return the following output when requesting the .rdf representation of the dataset:

<dcat:DataService rdf:about="https://example.com/dataservice1">
  <dct:license rdf:resource="http://creativecommons.org/publicdomain/zero/1.0/"/>
  <dct:accessRights rdf:resource="http://publications.europa.eu/resource/authority/access-right/PUBLIC"/>
  <dct:title>API exempel</dct:title>
  <dcat:endpointDescription>http://example.com/apidescription</dcat:endpointDescription>
</dcat:DataService>

@seitenbau-govdata seitenbau-govdata merged commit 93bf901 into ckan:master Aug 23, 2023
3 checks passed
@seitenbau-govdata
Copy link
Member

Thanks for the fix @SebastianEsp

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