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.

CommentFileSizeAuthor
#6 remove-node_rdf_mapping-6.patch1.69 KBwebflo
#1 remove-node_rdf_mapping.patch1.69 KBwebflo
remove-node_rdf_mapping.patch1.64 KBwebflo

Comments

webflo’s picture

webflo commented
Status: Active » Needs review
StatusFileSize
new remove-node_rdf_mapping.patch1.69 KB

Fixed the comment from node_load_multiple().

linclark’s picture

linclark commented
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

dries commented

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

dries’s picture

dries commented

#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

webflo commented
Status: Needs work » Needs review
StatusFileSize
new remove-node_rdf_mapping-6.patch1.69 KB

Rerolled.

linclark’s picture

linclark commented
Status: Needs review » Reviewed & tested by the community
yesct’s picture

yesct commented
Issue tags: +RTBC July 1

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

catch’s picture

catch
he/him
Primary language English
commented
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

(not verified) commented
Issue summary: View changes

Fixed issue link.