when updating a node, you have to save the node twice to see the update in the endpoint. that's because we're passing a nid to rdfx_get_rdf_model(), instead of the node itself in hook_node_update().

Steps to reproduce:
- install SPARQL Endpoint and all its dependencies
- create a node of type article
- check the sparql endpoint and note the value of the title of that note
- edit the node, modify its title, and save the node
- check the sparql endpoint again and note that the title is still the same
- re-edit and save the same node (no change) and see how the title finally changed in the SPARQL endpoint

CommentFileSizeAuthor
#3 error-sparql-ep.png30.56 KBclayball
#1 1237090_1_sparql_ep_node_update.patch1013 bytesscor

Comments

scor’s picture

scor commented
Status: Active » Needs review
StatusFileSize
new 1237090_1_sparql_ep_node_update.patch1013 bytes

this patch also align the hook_node_insert() implementation with #1237078: Align rdfx_get_rdf_model() prototype with entity API naming convention.

Anonymous’s picture

Anonymous (not verified) commented

I can't reproduce the bug, actually. Using the current dev of sparql_endpoint, my endpoint gets updated with every node save.

clayball’s picture

clayball commented
StatusFileSize
new error-sparql-ep.png30.56 KB

The patch produced the attached error on one of my servers.. and the error mentions rdfx.module
??? very strange

clayball’s picture

clayball commented

FYI: the error above still allows the creation of new content and the updating of existing content.
So the error is displyed.. then I got to the home page and can see the new content/the update.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Needs work

Based on your report, I think that the patch might need work.

scor’s picture

scor commented
Status: Needs work » Needs review

sorry, forgot to make it clear that this patch requires #1237078: Align rdfx_get_rdf_model() prototype with entity API naming convention.

scor’s picture

scor commented

updated the issue summary with steps to reproduce. I'm still seeing the same problem. patch #1 fixes the issue.#1237078: Align rdfx_get_rdf_model() prototype with entity API naming convention is fixed some this patch has no dependency.

scor’s picture

scor commented
Priority: Normal » Major

this keep bitting me. will commit this patch in a week unless someone argues otherwise.

scor’s picture

scor commented
Status: Needs review » Fixed

Fixed with http://drupalcode.org/project/sparql.git/commit/3276310

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Updated issue summary.