Entity API uses the following prototype for entity_metadata_wrapper():

entity_metadata_wrapper($type, $data = NULL, array $info = array())

in other words it accepts any kind of data as $data. rdfx_get_rdf_model() requires a entity ID but that's not convenient in certain conditions, for example see #1237090: SPARQL Endpoint index not immediately sync'ed on node update. I propose to align rdfx_get_rdf_model() to accept $data as second parameter instead of $id.

CommentFileSizeAuthor
#1 1237078_1_rdfx_get_rdf_model.patch680 bytesscor

Comments

scor’s picture

Status: Active » Needs review
StatusFileSize
new680 bytes
clayball’s picture

Status: Needs review » Reviewed & tested by the community

This patch applied cleanly and looks good to me. I reviewed this a two separate servers.

scor’s picture

Status: Reviewed & tested by the community » Fixed

Thanks Clay as usual for the review :)

Tests passed with #1229026: Test RDF serialization for Nodes no longer works applied (this one still needs review).

Fixed with http://drupalcode.org/project/rdfx.git/commit/1a65843

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

add issue number