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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tirdadc’s picture

tirdadc CreditAttribution: tirdadc commented
FileSize
rdf-693018.install.patch831 bytes
febbraro’s picture

febbraro CreditAttribution: 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 CreditAttribution: smustgrave at Mobomo commented
Issue summary: View changes
Status: Reviewed & tested by the community » Closed (outdated)