While running Strongarm, RDF and a module that uses RDF hooks to store data, we ran into recurrent user warnings about errors in the SQL syntax when saving / updating nodes. It turned out that if a module with lower weight than RDF calls drupal_get_schema() from its hook_init(), the user defined rdf_data tables never get properly loaded in the schema, resulting in a mess when saving RDF data through the API for the custom module.

This patch insures that rdf_db_get_repository_tables() is available to rdf_schema_alter().

CommentFileSizeAuthor
#1 rdf-693018.install.patch831 bytestirdadc

Comments

tirdadc’s picture

tirdadc commented
StatusFileSize
new rdf-693018.install.patch831 bytes
febbraro’s picture

febbraro commented
Status: Active » Reviewed & tested by the community

This looks good and works for me.

Other RDF maintainers, was there a specific reason for the previous implementation, meaning a reason for not just including the file that was needed, but instead skipping the addition if it was not previously included? Any objections to committing this?

smustgrave’s picture

smustgrave commented
Issue summary: View changes
Status: Reviewed & tested by the community » Closed (outdated)