This is a catch all issue for the various modules which implement hook_rdf_mapping(). The choice of the right RDF mappings for the core module was postponed during the work on the RDF Mapping API, since designing the API was

Changing these mappings is neither a new feature nor a UI or API change. However, since these mappings are fixed in the rdf_mapping table upon installation, we should agree on the right predicates before we start requiring an upgrade path for changes which affects the database (not sure whether this is after the first alpha or beta). The RDF API will provide a way to change the mappings, but core on its own won't.

These changes come from random discussions on semantic web mailing lists and/or from people who tried Drupal 7 and had feedback on the predicates it uses.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

scor’s picture

Status: Active » Needs review
FileSize
2.46 KB

starter patch

scor’s picture

According to the RDFa deployment advice, the official GRDDL profile URI is now http://www.w3.org/1999/xhtml/vocab (this URI was not available when we initially implemented RDFa in core).

sioct:Post does not exist.

scor’s picture

cleaned up the patch. added mapping for the 'changed' field in comment.module and fixed the changed mapping in node.module. Improved the mapping of taxonomy term and vocabulary using SKOS terms, add a mapping for the parent term.

Anonymous’s picture

Status: Needs review » Reviewed & tested by the community

These are pretty straight forward corrections and additions to the RDF mappings that are set by core modules, so they don't affect the UI or API.

You're right, since these are fixed in the database at install, this should be settled before an upgrade path is required.

RTBC

scor’s picture

forgot to update the tests, due to the renaming of sioct:Post to sioc:Post. All RDF tests are green now.

testing bot: we miss you!

Status: Reviewed & tested by the community » Needs work

The last submitted patch failed testing.

scor’s picture

Status: Needs work » Needs review
FileSize
9.55 KB

I'm not able to reproduce this failures. rerolling.

Anonymous’s picture

Status: Needs review » Reviewed & tested by the community

Since this passed testing, I'm resetting it to RTBC.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -RDF

Automatically closed -- issue fixed for 2 weeks with no activity.