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

CODE: Change from href to resource where CURIEs are used #47

Closed
wants to merge 1 commit into from

Conversation

niklasl
Copy link

@niklasl niklasl commented Jun 17, 2014

Plain href attributes only work with regular URIs. To get prefix expansion within CURIEs, resource (and other RDFa specific attributes) are required.

@danbri
Copy link
Contributor

danbri commented Jun 17, 2014

Thanks. I need to check our built-in pseudo-rdfa parser can read this, and also fix up the code that outputs this data too.

@danbri danbri changed the title Change from href to resource where CURIEs are used CODE: Change from href to resource where CURIEs are used Jun 17, 2014
@danbri
Copy link
Contributor

danbri commented Jul 8, 2014

I mis-spoke - our internal parser for rdfa/*.rdfa is irrelevant. This is just a fix in api.py for published per-term RDFa. I've gone ahead and made that change - thanks.

One note. I rely currently on only using prefixes that are in the RDFa Initial Context.

However http://www.w3.org/2012/pyRdfa/extract?uri=http%3A%2F%2Fsdopending.appspot.com%2FDataset&format=turtle&rdfagraph=output&vocab_expansion=false&rdfa_lite=false&embedded_rdf=true&space_preserve=true&vocab_cache=true&vocab_cache_report=false&vocab_cache_refresh=false suggests Ivan's parser isn't recognising 'dcat' prefix yet. I've followed up with Ivan.

@danbri danbri closed this Jul 8, 2014
@niklasl
Copy link
Author

niklasl commented Jul 8, 2014

Great! And thanks for reminding me about the pyRdfa service. I fixed this in RDFLib (RDFLib/rdflib@3182efb), but forgot to ping Ivan. I've sent him a reference to this fix now.

@danbri
Copy link
Contributor

danbri commented Jul 8, 2014

Thanks again for spotting this! :)

danbri pushed a commit that referenced this pull request Apr 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
site tools + python code Infrastructural issues around schema.org site. Most can ignore this!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants