hook_rdf_mapping got removed by #1869600: Refactor RDF mappings to be inline with the new Entity Field API but node_rdf_mapping is still in node module. Lets remove it.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webflo’s picture

Status: Active » Needs review
FileSize
1.69 KB

Fixed the comment from node_load_multiple().

linclark’s picture

Status: Needs review » Reviewed & tested by the community

Thanks! This is related to #2030467: Remove mentions of hook_rdf_mapping, which I filed earlier today. I'll refocus that issue to updating the documentation.

Dries’s picture

I believe this needs a re-roll as the patch no longer applies. Asking for re-test.

Dries’s picture

#1: remove-node_rdf_mapping.patch queued for re-testing.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, remove-node_rdf_mapping.patch, failed testing.

webflo’s picture

Status: Needs work » Needs review
FileSize
1.69 KB

Rerolled.

linclark’s picture

Status: Needs review » Reviewed & tested by the community
YesCT’s picture

Issue tags: +RTBC July 1

This issue was RTBC and passing tests on July 1, the beginning of API freeze.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

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

Anonymous’s picture

Issue summary: View changes

Fixed issue link.