I'm not sure whether to file this here or as a bug for the domain access module (and other modules using drupal_goto in hook_init).

nodequeue.views.inc carries a hook_views_tables and is included in nodequeue_init. Unfortunately, this reliance means that whenever a hook_init implementation running before nodequeue_init, issues a drupal_goto, nodequeue_views_tables is not loaded.

This in turn means that views will not know about nodequeue tables (and build broken queries) when certain conditions are met:

1. A visitor causes cache_views to be emptied by creating a new taxonomy term.
2. Another visitor's request reaches domain_init; it issues a drupal_goto before reaching nodequeue_init
3. drupal_goto invokes hook_exit implementations.
4. statistics_exit attempts to fetch a title for the current page when it is set to log page access.
5. as no title has been set explicitly menu_get_active_title eventually initiates call to hook_menu implementations.
6. views_menu eventually triggers a rebuild of table information in cache_views.

A more robust, but uglier approach to including nodequeue.views.inc would be to implement the hooks as wrappers in nodequeue.module.

Comments

ezra-g’s picture

Status: Active » Closed (won't fix)

Thanks for the thorough bug report.

Marking WF now that Drupal 5 is no longer a supported core version.