The logic in _aggregator_get_variables() is incorrect: It thinks that if the aggregator's processor is turned off in the UI, then aggregator.processor.inc doesn't need to be loaded any more. But it's not true: If you turn off that processor, then the whole settings UI goes away, because it's not there :-)

The correct implementation here is to use hook_hook_info() to inform Drupal about the availability of these hooks. This patch adds that. (Thanks, @davereid.)

CommentFileSizeAuthor
drupal.aggregator_hook_hook_info.patch1.75 KBrfay

Comments

Status: Needs review » Needs work

The last submitted patch, drupal.aggregator_hook_hook_info.patch, failed testing.

jody lynn’s picture

jody lynn
she/her
Primary language English
commented
Version: 7.x-dev » 8.x-dev
rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Version: 8.x-dev » 7.x-dev

Actually, this is a bug, not a feature, so may still need to be addressed in 7.x. I have to admit it's an unusual case, and that the bug has probably been there through 3-4 versions of Drupal.

jody lynn’s picture

jody lynn
she/her
Primary language English
commented

What's hook_hook_info?

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.