I was working in some contrib hook_boot code recently, and noticed that I had to perform a full bootstrap just to use node_load/node_load_multiple with drupal_load('module', 'node') because there is a call to drupal_schema_fields_sql(), which is in common.inc. The same applies to taxonomy_term_load/taxonomy_term_load_multiple. I suggest moving drupal_schema_fields_sql() to bootstrap.inc.

CommentFileSizeAuthor
#1 345497-mov-drupal-schema-fields-sql-D7.patch2.33 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 345497-mov-drupal-schema-fields-sql-D7.patch2.33 KB
damien tournoud’s picture

damien tournoud commented

It sounds weird to move that function to bootstrap.inc. Why not database.inc, where it can make new database friends?

Status: Needs review » Needs work

The last submitted patch failed testing.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Needs review

Testing failure on slave #8.

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Needs work

Review #2 still stands.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

My main reason for moving it to bootstrap.inc is because we have a group of schema API functions there already, like drupal_get_schema().

Jooblay.net’s picture

Jooblay.net commented
Issue summary: View changes

What is the status of this ticket:) Can we close this...

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.